summaryrefslogtreecommitdiff
path: root/program/steps
diff options
context:
space:
mode:
authorThomas Bruederli <thomas@roundcube.net>2014-04-08 10:04:53 +0200
committerThomas Bruederli <thomas@roundcube.net>2014-04-08 10:04:53 +0200
commitf0c94a3dc308d22f3e79fd4df3f1b2016c14e1c0 (patch)
treefea1456415f76d53ba65180907c0859f60c75ec8 /program/steps
parent0f48e6e660cb5aa0d7bfdf11967b648c78cb64f9 (diff)
Don't display error when moving a message to the same folder it already resides
Diffstat (limited to 'program/steps')
-rw-r--r--program/steps/mail/move_del.inc16
1 files changed, 12 insertions, 4 deletions
diff --git a/program/steps/mail/move_del.inc b/program/steps/mail/move_del.inc
index 7df22590a..ae367b4f7 100644
--- a/program/steps/mail/move_del.inc
+++ b/program/steps/mail/move_del.inc
@@ -5,7 +5,7 @@
| program/steps/mail/move_del.inc |
| |
| This file is part of the Roundcube Webmail client |
- | Copyright (C) 2005-2013, The Roundcube Dev Team |
+ | Copyright (C) 2005-2014, The Roundcube Dev Team |
| |
| Licensed under the GNU General Public License version 3 or |
| any later version with exceptions for skins & plugins. |
@@ -35,12 +35,20 @@ if ($RCMAIL->action == 'move' && !empty($_POST['_uid']) && strlen($_POST['_targe
$target = rcube_utils::get_input_value('_target_mbox', rcube_utils::INPUT_POST, true);
$trash = $RCMAIL->config->get('trash_mbox');
+ $success = true;
foreach (rcmail::get_uids() as $mbox => $uids) {
- $moved += (int)$RCMAIL->storage->move_message($uids, $target, $mbox);
- $count += count($uids);
+ if ($mbox == $target) {
+ $count += count($uids);
+ }
+ else if ($RCMAIL->storage->move_message($uids, $target, $mbox)) {
+ $count += count($uids);
+ }
+ else {
+ $success = false;
+ }
}
- if (!$moved) {
+ if (!$success) {
// send error message
if ($_POST['_from'] != 'show')
$OUTPUT->command('list_mailbox');