summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAleksander Machniak <alec@alec.pl>2014-04-18 11:26:07 +0200
committerAleksander Machniak <alec@alec.pl>2014-04-18 11:27:03 +0200
commite1e65c49803e5236a7b6e9527b90e5a894b02281 (patch)
treee5635baa3b1fe891f886138cedfe3bdf2df026d2
parent701d7a369b36813fd0673e0f4e373df6951e1c78 (diff)
Fix redundant alert message on over-size uploads (#1489817)
-rw-r--r--CHANGELOG1
-rw-r--r--program/js/app.js20
2 files changed, 12 insertions, 9 deletions
diff --git a/CHANGELOG b/CHANGELOG
index aea6a8481..43867e5c9 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -9,6 +9,7 @@ CHANGELOG Roundcube Webmail
- Fix lack of translation of special folders in some configurations (#1489799)
- Fix XSS issue in plain text spellchecker (#1489806)
- Fix invalid page title for some folders (1489804)
+- Fix redundant alert message on over-size uploads (#1489817)
RELEASE 1.0.0
-------------
diff --git a/program/js/app.js b/program/js/app.js
index 49a74d2cb..a72f49e53 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -1054,8 +1054,9 @@ function rcube_webmail()
// Reset the auto-save timer
clearTimeout(this.save_timer);
- if (!this.upload_file(props || this.gui_objects.uploadform, 'upload')) {
- alert(this.get_label('selectimportfile'));
+ if (!(flag = this.upload_file(props || this.gui_objects.uploadform, 'upload'))) {
+ if (flag !== false)
+ alert(this.get_label('selectimportfile'));
aborted = true;
}
break;
@@ -1179,12 +1180,15 @@ function rcube_webmail()
break;
case 'import-messages':
- var form = props || this.gui_objects.importform;
- var importlock = this.set_busy(true, 'importwait');
+ var form = props || this.gui_objects.importform,
+ importlock = this.set_busy(true, 'importwait');
+
$('input[name="_unlock"]', form).val(importlock);
- if (!this.upload_file(form, 'import')) {
+
+ if (!(flag = this.upload_file(form, 'import'))) {
this.set_busy(false, null, importlock);
- alert(this.get_label('selectimportfile'));
+ if (flag !== false)
+ alert(this.get_label('selectimportfile'));
aborted = true;
}
break;
@@ -3973,7 +3977,7 @@ function rcube_webmail()
this.upload_file = function(form, action)
{
if (!form)
- return false;
+ return;
// count files and size on capable browser
var size = 0, numfiles = 0;
@@ -4033,8 +4037,6 @@ function rcube_webmail()
this.gui_objects.attachmentform = form;
return true;
}
-
- return false;
};
// add file name to attachment list