Don't remove messages from list when moving to another folder in multi-fold...

Thomas Bruederli

2014-04-08 1e9a59ab89ed69d4bf3cd2d306e1c3f8821f4018

Don't remove messages from list when moving to another folder in multi-folder search mode, just update the list

 program/js/app.js

@@ -707,7 +707,7 @@
        break;

      case 'list':
        // re-send for the selected folder
        // re-send search query for the selected folder
        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
@@ -1713,7 +1713,9 @@
  {
    switch (this.task) {
      case 'mail':
        return (this.env.mailboxes[id] && this.env.mailboxes[id].id != this.env.mailbox && !this.env.mailboxes[id].virtual) ? 1 : 0;
        return (this.env.mailboxes[id]
            && !this.env.mailboxes[id].virtual
            && (this.env.mailboxes[id].id != this.env.mailbox || this.is_multifolder_search())) ? 1 : 0;

      case 'settings':
        return id != this.env.mailbox ? 1 : 0;
@@ -2191,8 +2193,16 @@
    this.http_request('search', this.search_params(false, filter), lock);
  };

  // reload the current message listing
  this.refresh_list = function()
  {
    this.list_mailbox(this.env.mailbox, this.env.current_page || 1, null, { _clear:1 }, true);
    if (this.message_list)
      this.message_list.clear_selection();
  };

  // list messages of a specific mailbox
  this.list_mailbox = function(mbox, page, sort, url)
  this.list_mailbox = function(mbox, page, sort, url, update_only)
  {
    var win, target = window;

@@ -2217,15 +2227,17 @@
      this.select_all_mode = false;
    }

    // unselect selected messages and clear the list and message data
    this.clear_message_list();
    if (!update_only) {
      // unselect selected messages and clear the list and message data
      this.clear_message_list();

    if (mbox != this.env.mailbox || (mbox == this.env.mailbox && !page && !sort))
      url._refresh = 1;
      if (mbox != this.env.mailbox || (mbox == this.env.mailbox && !page && !sort))
        url._refresh = 1;

    this.select_folder(mbox, '', true);
    this.unmark_folder(mbox, 'recent', '', true);
    this.env.mailbox = mbox;
      this.select_folder(mbox, '', true);
      this.unmark_folder(mbox, 'recent', '', true);
      this.env.mailbox = mbox;
    }

    // load message list remotely
    if (this.gui_objects.messagelist) {
@@ -2259,20 +2271,17 @@
  };

  // send remote request to load message list
  this.list_mailbox_remote = function(mbox, page, post_data)
  this.list_mailbox_remote = function(mbox, page, url)
  {
    // clear message list first
    this.message_list.clear();

    var lock = this.set_busy(true, 'loading');

    if (typeof post_data != 'object')
      post_data = {};
    post_data._mbox = mbox;
    if (typeof url != 'object')
      url = {};
    url._mbox = mbox;
    if (page)
      post_data._page = page;
      url._page = page;

    this.http_request('list', post_data, lock);
    this.http_request('list', url, lock);
  };

  // removes messages that doesn't exists from list selection array
@@ -2689,7 +2698,7 @@
      return this.folder_selector(obj, function(folder) { ref.command('move', folder); });

    // exit if current or no mailbox specified
    if (!mbox || (mbox == this.env.mailbox && (!this.env.search_request || this.env.search_scope == 'base')))
    if (!mbox || (mbox == this.env.mailbox && !this.is_multifolder_search()))
      return;

    var lock = false, post_data = this.selection_post_data({_target_mbox: mbox});
@@ -2757,7 +2766,8 @@
  // @private
  this._with_selected_messages = function(action, post_data, lock)
  {
    var count = 0, msg;
    var count = 0, msg,
      remove = (action == 'delete' || !this.is_multifolder_search());

    // update the list (remove rows, clear selection)
    if (this.message_list) {
@@ -2774,10 +2784,11 @@
            roots.push(root);
          }
        }
        this.message_list.remove_row(id, (this.env.display_next && n == selection.length-1));
        if (remove)
          this.message_list.remove_row(id, (this.env.display_next && n == selection.length-1));
      }
      // make sure there are no selected rows
      if (!this.env.display_next)
      if (!this.env.display_next && remove)
        this.message_list.clear_selection();
      // update thread tree icons
      for (n=0, len=roots.length; n<len; n++) {
@@ -2788,8 +2799,11 @@
    if (count < 0)
      post_data._count = (count*-1);
    // remove threads from the end of the list
    else if (count > 0)
    else if (count > 0 && remove)
      this.delete_excessive_thread_rows();

    if (!remove)
      post_data._refresh = 1;

    if (!lock) {
      msg = action == 'move' ? 'movingmessage' : 'deletingmessage';
@@ -4259,6 +4273,11 @@
    this.env.search_mods[mbox] = mods;
  };

  this.is_multifolder_search = function()
  {
    return this.env.search_request && (this.env.search_scope || 'base') != 'base';
  }

  this.sent_successfully = function(type, msg, folders)
  {
    this.display_message(msg, type);

 program/lib/Roundcube/rcube_ldap.php

@@ -377,10 +377,11 @@
                // replace placeholders in filter settings
                if (!empty($this->prop['filter']))
                    $this->prop['filter'] = strtr($this->prop['filter'], $replaces);
                if (!empty($this->prop['groups']['filter']))
                    $this->prop['groups']['filter'] = strtr($this->prop['groups']['filter'], $replaces);
                if (!empty($this->prop['groups']['member_filter']))
                    $this->prop['groups']['member_filter'] = strtr($this->prop['groups']['member_filter'], $replaces);

                foreach (array('base_dn','filter','member_filter') as $k) {
                    if (!empty($this->prop['groups'][$k]))
                        $this->prop['groups'][$k] = strtr($this->prop['groups'][$k], $replaces);
                }

                if (!empty($this->prop['group_filters'])) {
                    foreach ($this->prop['group_filters'] as $i => $gf) {

 program/steps/mail/list.inc

@@ -101,6 +101,11 @@
$OUTPUT->set_env('exists', $exists);
$OUTPUT->command('set_rowcount', rcmail_get_messagecount_text($count), $mbox_name);

// remove old message rows if commanded by the client
if (!empty($_REQUEST['_clear'])) {
  $OUTPUT->command('clear_message_list');
}

// add message rows
rcmail_js_message_list($a_headers, false, $cols);


 program/steps/mail/move_del.inc

@@ -60,7 +60,13 @@
        $OUTPUT->show_message('messagemoved', 'confirmation');
    }

    $addrows = true;
    if (!empty($_POST['_refresh'])) {
        // FIXME: send updated message rows instead of releading the entire list
        $OUTPUT->command('refresh_list');
    }
    else {
        $addrows = true;
    }
}
// delete messages 
else if ($RCMAIL->action=='delete' && !empty($_POST['_uid'])) {