Don't display error when moving a message to the same folder it already res...

Thomas Bruederli

2014-04-08 f0c94a3dc308d22f3e79fd4df3f1b2016c14e1c0

Don't display error when moving a message to the same folder it already resides

 program/include/rcmail.php

@@ -2022,8 +2022,8 @@
        // message UID (or comma-separated list of IDs) is provided in
        // the form of <ID>-<MBOX>[,<ID>-<MBOX>]*

        $_uid  = $uids ?: get_input_value('_uid', RCUBE_INPUT_GPC);
        $_mbox = $mbox ?: (string)get_input_value('_mbox', RCUBE_INPUT_GPC);
        $_uid  = $uids ?: rcube_utils::get_input_value('_uid', RCUBE_INPUT_GPC);
        $_mbox = $mbox ?: (string)rcube_utils::get_input_value('_mbox', RCUBE_INPUT_GPC);

        if (is_array($uid)) {
            return $uid;

 program/js/app.js

@@ -708,7 +708,7 @@

      case 'list':
        // re-send for the selected folder
        if (props && props != '' && this.env.search_request) {
        if (props && props != '' && this.env.search_request && this.gui_objects.qsearchbox.value) {
          var oldmbox = this.env.search_scope == 'all' ? '*' : this.env.mailbox;
          this.env.search_mods[props] = this.env.search_mods[oldmbox];  // copy search mods from active search
          this.env.mailbox = props;

 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 @@
    $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');