- Fix listupdate event doesn't trigger on search response (#1486708)

alecpl

2010-05-06 c833ed4053106c9df58b84a441cc4509cda45a38

 program/js/app.js

@@ -87,7 +87,7 @@
    if (sel) button_prop.sel = sel;
    if (over) button_prop.over = over;

    this.buttons[command][this.buttons[command].length] = button_prop;    
    this.buttons[command][this.buttons[command].length] = button_prop;
  };

  // register a specific gui object
@@ -165,8 +165,10 @@

        if (this.gui_objects.messagelist) {

          this.message_list = new rcube_list_widget(this.gui_objects.messagelist,
            {multiselect:true, multiexpand:true, draggable:true, keyboard:true, dblclick_time:this.dblclick_time});
          this.message_list = new rcube_list_widget(this.gui_objects.messagelist, {
            multiselect:true, multiexpand:true, draggable:true, keyboard:true,
            column_movable:this.env.col_movable, column_fixed:0, dblclick_time:this.dblclick_time
            });
          this.message_list.row_init = function(o){ p.init_message_row(o); };
          this.message_list.addEventListener('dblclick', function(o){ p.msglist_dbl_click(o); });
          this.message_list.addEventListener('click', function(o){ p.msglist_click(o); });
@@ -176,17 +178,16 @@
          this.message_list.addEventListener('dragmove', function(e){ p.drag_move(e); });
          this.message_list.addEventListener('dragend', function(e){ p.drag_end(e); });
          this.message_list.addEventListener('expandcollapse', function(e){ p.msglist_expand(e); });
          this.message_list.addEventListener('column_replace', function(e){ p.msglist_set_coltypes(e); });

          document.onmouseup = function(e){ return p.doc_mouse_up(e); };
          this.gui_objects.messagelist.parentNode.onmousedown = function(e){ return p.click_on_list(e); };

          this.set_message_coltypes(this.env.coltypes);
          this.message_list.init();
          this.enable_command('toggle_status', 'toggle_flag', 'menu-open', 'menu-save', true);

          // load messages
          if (this.env.messagecount)
            this.command('list');
          this.command('list');
        }

        if (this.gui_objects.qsearchbox) {
@@ -245,16 +246,6 @@
        // show printing dialog
        else if (this.env.action == 'print')
          window.print();

        if (this.env.messagecount) {
          this.enable_command('select-all', 'select-none', 'expunge', true);
          this.enable_command('expand-all', 'expand-unread', 'collapse-all', this.env.threading);
        }

        if (this.purge_mailbox_test())
          this.enable_command('purge', true);

        this.set_page_buttons();

        // get unread count for each mailbox
        if (this.gui_objects.mailboxlist) {
@@ -369,6 +360,11 @@

        // detect client timezone
        $('#rcmlogintz').val(new Date().getTimezoneOffset() / -60);

        // display 'loading' message on form submit
        $('form').submit(function () { 
          rcmail.display_message(rcmail.get_label('loading'), 'loading', true); 
        }); 

        this.enable_command('login', true);
        break;
@@ -1403,15 +1399,13 @@
    var selected = list.get_single_selection() != null;

    // Hide certain command buttons when Drafts folder is selected
    if (this.env.mailbox == this.env.drafts_mailbox) {
      this.enable_command('reply', 'reply-all', 'forward', false);
      this.enable_command('show', 'print', 'open', 'edit', 'download', 'viewsource', selected);
      this.enable_command('delete', 'moveto', 'copy', 'mark', (list.selection.length > 0 ? true : false));
    }
    else {
      this.enable_command('show', 'reply', 'reply-all', 'forward', 'print', 'edit', 'open', 'download', 'viewsource', selected);
      this.enable_command('delete', 'moveto', 'copy', 'mark', (list.selection.length > 0 ? true : false));
    }
    this.enable_command('reply', 'reply-all', 'forward', this.env.mailbox == this.env.drafts_mailbox ? false : selected);
    this.enable_command('show', 'print', 'open', 'edit', 'download', 'viewsource', selected);
    this.enable_command('delete', 'moveto', 'copy', 'mark', (list.selection.length > 0 ? true : false));

    // reset all-pages-selection
    if (selected || list.selection.length != list.rowcount)
      this.select_all_mode = false;

    // start timer for message preview (wait for double click)
    if (selected && this.env.contentframe && !list.multi_selecting)
@@ -1482,6 +1476,24 @@
    if (this.env.messages[row.uid])
      this.env.messages[row.uid].expanded = row.expanded;
  };
  
  this.msglist_set_coltypes = function(list)
  {
    var i, found, name, cols = list.list.tHead.rows[0].cells;
    
    this.env.coltypes = [];
    
    for (i=0; i<cols.length; i++)
      if (cols[i].id && cols[i].id.match(/^rcm/)) {
        name = cols[i].id.replace(/^rcm/, '');
        this.env.coltypes[this.env.coltypes.length] = name == 'to' ? 'from' : name;
      }

    if ((found = $.inArray('flag', this.env.coltypes)) >= 0)
      this.set_env('flagged_col', found);

    this.http_post('save-pref', { '_name':'list_cols', '_value':this.env.coltypes });
  };

  this.check_droptarget = function(id)
  {
@@ -1502,7 +1514,7 @@

  this.init_message_row = function(row)
  {
    var self = this, uid = row.uid;
    var expando, self = this, uid = row.uid;

    if (uid && this.env.messages[uid])
      $.extend(row, this.env.messages[uid]);
@@ -1519,7 +1531,6 @@
      row.flagged_icon.onmousedown = function(e) { self.command('toggle_flag', this); };
    }

    var expando;
    if (!row.depth && row.has_children && (expando = document.getElementById('rcmexpando'+row.uid))) {
      expando.onmousedown = function(e) { return self.expand_message_row(e, uid); };
    }
@@ -1554,7 +1565,7 @@
      parent_uid: flags.parent_uid
    });

    var tree = expando = '',
    var c, tree = expando = '',
      rows = this.message_list.rows,
      rowcount = tbody.rows.length,
      even = rowcount%2,
@@ -1633,20 +1644,20 @@

    // add each submitted col
    for (var n = 0; n < this.env.coltypes.length; n++) {
      var c = this.env.coltypes[n];
      c = this.env.coltypes[n];
      col = document.createElement('td');
      col.className = String(c).toLowerCase();

      var html;
      if (c=='flag') {
      if (c == 'flag') {
        if (flags.flagged && this.env.flaggedicon)
          html = '<img id="flaggedicn'+uid+'" src="'+this.env.flaggedicon+'" class="flagicon" alt="" />';
        else if(!flags.flagged && this.env.unflaggedicon)
          html = '<img id="flaggedicn'+uid+'" src="'+this.env.unflaggedicon+'" class="flagicon" alt="" />';
      }
      else if (c=='attachment')
      else if (c == 'attachment')
        html = flags.attachment && this.env.attachmenticon ? '<img src="'+this.env.attachmenticon+'" alt="" />' : '&nbsp;';
      else if (c=='subject')
      else if (c == 'subject')
        html = tree + cols[c];
      else
        html = cols[c];
@@ -1688,6 +1699,9 @@
  {
    var update, add_url = '';

    if (!sort_col) sort_col = this.env.sort_col;
    if (!sort_order) sort_order = this.env.sort_order;

    if (this.env.sort_col != sort_col || this.env.sort_order != sort_order) {
      update = 1;
      this.set_list_sorting(sort_col, sort_order);
@@ -1698,9 +1712,25 @@
      add_url += '&_threads=' + threads;     
    }

    if (cols.join() != this.env.coltypes.join()) {
      update = 1;
      add_url += '&_cols=' + cols.join(',');
    if (cols && cols.length) {
      // make sure new columns are added at the end of the list
      var i, idx, name, newcols = [], oldcols = this.env.coltypes;
      for (i=0; i<oldcols.length; i++) {
        name = oldcols[i] == 'to' ? 'from' : oldcols[i];
        idx = $.inArray(name, cols);
        if (idx != -1) {
          newcols[newcols.length] = name;
          delete cols[idx];
        }
      }
      for (i=0; i<cols.length; i++)
        if (cols[i])
          newcols[newcols.length] = cols[i];
      
      if (newcols.join() != this.env.coltypes.join()) {
        update = 1;
        add_url += '&_cols=' + newcols.join(',');
      }
    }

    if (update)
@@ -1710,11 +1740,12 @@
  // when user doble-clicks on a row
  this.show_message = function(id, safe, preview)
  {
    if (!id) return;
    if (!id)
      return;

    var add_url = '';
    var action = preview ? 'preview': 'show';
    var target = window;
    var add_url = '',
      target = window,
      action = preview ? 'preview': 'show';

    if (preview && this.env.contentframe && window.frames && window.frames[this.env.contentframe]) {
      target = window.frames[this.env.contentframe];
@@ -1770,21 +1801,21 @@
  // list a specific page
  this.list_page = function(page)
  {
    if (page=='next')
    if (page == 'next')
      page = this.env.current_page+1;
    if (page=='last')
    if (page == 'last')
      page = this.env.pagecount;
    if (page=='prev' && this.env.current_page>1)
    if (page == 'prev' && this.env.current_page > 1)
      page = this.env.current_page-1;
    if (page=='first' && this.env.current_page>1)
    if (page == 'first' && this.env.current_page > 1)
      page = 1;

    if (page > 0 && page <= this.env.pagecount) {
      this.env.current_page = page;

      if (this.task=='mail')
      if (this.task == 'mail')
        this.list_mailbox(this.env.mailbox, page);
      else if (this.task=='addressbook')
      else if (this.task == 'addressbook')
        this.list_contacts(this.env.source, null, page);
    }
  };
@@ -1809,8 +1840,7 @@
  // list messages of a specific mailbox
  this.list_mailbox = function(mbox, page, sort, add_url)
  {
    var url = '';
    var target = window;
    var url = '', target = window;

    if (!mbox)
      mbox = this.env.mailbox;
@@ -1878,18 +1908,18 @@
  // expand all threads with unread children
  this.expand_unread = function()
  {
    var tbody = this.gui_objects.messagelist.tBodies[0];
    var new_row = tbody.firstChild;
    var r;
    var r, expando,
      tbody = this.gui_objects.messagelist.tBodies[0],
      new_row = tbody.firstChild;

    while (new_row) {
      if (new_row.nodeType == 1 && (r = this.message_list.rows[new_row.uid])
       && r.unread_children) {
       this.message_list.expand_all(r);
       var expando = document.getElementById('rcmexpando' + r.uid);
       expando = document.getElementById('rcmexpando' + r.uid);
       if (expando)
         expando.className = 'expanded';
         this.set_unread_children(r.uid);
       this.set_unread_children(r.uid);
      }
      new_row = new_row.nextSibling;
    }
@@ -1919,7 +1949,6 @@
      case 2: this.expand_unread(); break;
      case 1: this.message_list.expand_all(); break;
    }
    //  this.message_list.expand(null);
  };

  // update parent in a thread
@@ -1937,10 +1966,12 @@

    if (flag == 'read' && p.unread_children) {
      p.unread_children--;
    } else if (flag == 'unread' && p.has_children) {
    }
    else if (flag == 'unread' && p.has_children) {
      // unread_children may be undefined
      p.unread_children = p.unread_children ? p.unread_children + 1 : 1;
    } else {
    }
    else {
      return;
    }

@@ -1955,17 +1986,17 @@
    if (!this.env.threading)
      return 0;

    var rows = this.message_list.rows;
    var row = rows[uid]
    var depth = rows[uid].depth;
    var r, parent, count = 0;
    var roots = [];
    var r, parent, count = 0,
      rows = this.message_list.rows,
      row = rows[uid],
      depth = rows[uid].depth,
      roots = [];

    if (!row.depth) // root message: decrease roots count
      count--;
    else if (row.unread) {
      // update unread_children for thread root
      var parent = this.message_list.find_root(uid);
      parent = this.message_list.find_root(uid);
      rows[parent].unread_children--;
      this.set_unread_children(parent);
    }
@@ -2018,10 +2049,10 @@

  this.delete_excessive_thread_rows = function()
  {
    var rows = this.message_list.rows;    
    var tbody = this.message_list.list.tBodies[0];
    var row = tbody.firstChild;
    var cnt = this.env.pagesize + 1;
    var rows = this.message_list.rows,
      tbody = this.message_list.list.tBodies[0],
      row = tbody.firstChild,
      cnt = this.env.pagesize + 1;

    while (row) {
      if (row.nodeType == 1 && (r = rows[row.uid])) {
@@ -2038,8 +2069,8 @@
  // set message icon
  this.set_message_icon = function(uid)
  {
    var icn_src;
    var rows = this.message_list.rows;
    var icn_src,
      rows = this.message_list.rows;

    if (!rows[uid])
      return false;
@@ -2578,8 +2609,10 @@
      input_to.focus();
    else if (input_subject.val() == '')
      input_subject.focus();
    else if (input_message && !html_mode)
    else if (input_message)
      input_message.focus();

    this.env.compose_focus_elem = document.activeElement;

    // get summary of all field values
    this.compose_field_hash(true);
@@ -2746,12 +2779,12 @@
    if (!show_sig)
      show_sig = this.env.show_sig;

    var id = obj.options[obj.selectedIndex].value;
    var input_message = $("[name='_message']");
    var message = input_message.val();
    var is_html = ($("input[name='_is_html']").val() == '1');
    var sig_separator = this.env.sig_above && (this.env.compose_mode == 'reply' || this.env.compose_mode == 'forward') ? '---' : '-- ';
    var sig, cursor_pos, p = -1;
    var sig, cursor_pos, p = -1,
      id = obj.options[obj.selectedIndex].value,
      input_message = $("[name='_message']"),
      message = input_message.val(),
      is_html = ($("input[name='_is_html']").val() == '1'),
      sig_separator = this.env.sig_above && (this.env.compose_mode == 'reply' || this.env.compose_mode == 'forward') ? '---' : '-- ';

    if (!this.env.identity)
      this.env.identity = id
@@ -2787,7 +2820,7 @@
          if (p >= 0) { // in place of removed signature
            message = message.substring(0, p) + sig + message.substring(p, message.length);
            cursor_pos = p - 1;
          } 
          }
          else if (pos = this.get_caret_pos(input_message.get(0))) { // at cursor position
            message = message.substring(0, pos) + '\n' + sig + '\n\n' + message.substring(pos, message.length);
            cursor_pos = pos;
@@ -2795,7 +2828,7 @@
          else { // on top
            cursor_pos = 0;
            message = '\n\n' + sig + '\n\n' + message.replace(/^[\r\n]+/, '');
          } 
          }
        }
        else {
          message = message.replace(/[\r\n]+$/, '');
@@ -2812,20 +2845,20 @@
      this.set_caret_pos(input_message.get(0), cursor_pos);
    }
    else if (show_sig && this.env.signatures) {  // html
      var editor = tinyMCE.get(this.env.composebody);
      var sigElem = editor.dom.get('_rc_sig');
      var editor = tinyMCE.get(this.env.composebody),
        sigElem = editor.dom.get('_rc_sig');

      // Append the signature as a div within the body
      if (!sigElem) {
        var body = editor.getBody();
        var doc = editor.getDoc();
        var body = editor.getBody(),
          doc = editor.getDoc();

        sigElem = doc.createElement('div');
        sigElem.setAttribute('id', '_rc_sig');

        if (this.env.sig_above) {
          // if no existing sig and top posting then insert at caret pos
          editor.getWin().focus(); // correct focus in IE
          editor.getWin().focus(); // correct focus in IE & Chrome

          var node = editor.selection.getNode();
          if (node.nodeName == 'BODY') {
@@ -3431,8 +3464,8 @@
  // load contact record
  this.load_contact = function(cid, action, framed)
  {
    var add_url = '';
    var target = window;
    var add_url = '', target = window;

    if (this.env.contentframe && window.frames && window.frames[this.env.contentframe]) {
      add_url = '&_framed=1';
      target = window.frames[this.env.contentframe];
@@ -3442,8 +3475,11 @@
      return false;

    if (action && (cid || action=='add') && !this.drag_active) {
      if (this.env.group)
        add_url += '&_gid='+urlencode(this.env.group);

      this.set_busy(true);
      target.location.href = this.env.comm_path+'&_action='+action+'&_source='+urlencode(this.env.source)+'&_gid='+urlencode(this.env.group)+'&_cid='+urlencode(cid) + add_url;
      target.location.href = this.env.comm_path+'&_action='+action+'&_source='+urlencode(this.env.source)+'&_cid='+urlencode(cid) + add_url;
    }
    return true;
  };
@@ -4201,13 +4237,13 @@

      // set image according to button state
      if (obj && button.type=='image' && button[state]) {
        button.status = state;        
        button.status = state;
        obj.src = button[state];
      }
      // set class name according to button state
      else if (obj && typeof(button[state])!='undefined') {
        button.status = state;        
        obj.className = button[state];        
        button.status = state;
        obj.className = button[state];
      }
      // disable/enable input buttons
      if (obj && button.type=='input') {
@@ -4431,13 +4467,15 @@
    this.env.flagged_col = null;

    var found;
    if((found = $.inArray('subject', this.env.coltypes)) >= 0) {
    if ((found = $.inArray('subject', this.env.coltypes)) >= 0) {
      this.set_env('subject_col', found);
      if (this.message_list)
        this.message_list.subject_col = found+1;
    }
    if((found = $.inArray('flag', this.env.coltypes)) >= 0)
    if ((found = $.inArray('flag', this.env.coltypes)) >= 0)
      this.set_env('flagged_col', found);

    this.message_list.init_header();
  };

  // replace content of row count display
@@ -4481,6 +4519,7 @@
  this.set_unread_count_display = function(mbox, set_title)
  {
    var reg, text_obj, item, mycount, childcount, div;

    if (item = this.get_folder_li(mbox)) {
      mycount = this.env.unread_counts[mbox] ? this.env.unread_counts[mbox] : 0;
      text_obj = item.getElementsByTagName('a')[0];
@@ -4517,8 +4556,8 @@
    // set unread count to window title
    reg = /^\([0-9]+\)\s+/i;
    if (set_title && document.title) {
      var doc_title = String(document.title);
      var new_title = "";
      var new_title = '',
        doc_title = String(document.title);

      if (mycount && doc_title.match(reg))
        new_title = doc_title.replace(reg, '('+mycount+') ');
@@ -4543,16 +4582,16 @@

  this.toggle_prefer_html = function(checkbox)
  {
    var addrbook_show_images;
    if (addrbook_show_images = document.getElementById('rcmfd_addrbook_show_images'))
      addrbook_show_images.disabled = !checkbox.checked;
    var elem;
    if (elem = document.getElementById('rcmfd_addrbook_show_images'))
      elem.disabled = !checkbox.checked;
  };

  this.toggle_preview_pane = function(checkbox)
  {
    var preview_pane_mark_read;
    if (preview_pane_mark_read = document.getElementById('rcmfd_preview_pane_mark_read'))
      preview_pane_mark_read.disabled = !checkbox.checked;
    var elem;
    if (elem = document.getElementById('rcmfd_preview_pane_mark_read'))
      elem.disabled = !checkbox.checked;
  };

  // display fetched raw headers
@@ -4602,13 +4641,13 @@
    if (!data || !obj)
      return false;

    var limit_high = 80;
    var limit_mid  = 55;
    var width = data.width ? data.width : this.env.indicator_width ? this.env.indicator_width : 100;
    var height = data.height ? data.height : this.env.indicator_height ? this.env.indicator_height : 14;
    var quota = data.percent ? Math.abs(parseInt(data.percent)) : 0;
    var quota_width = parseInt(quota / 100 * width);
    var pos = $(obj).position();
    var limit_high = 80,
      limit_mid  = 55,
      width = data.width ? data.width : this.env.indicator_width ? this.env.indicator_width : 100,
      height = data.height ? data.height : this.env.indicator_height ? this.env.indicator_height : 14,
      quota = data.percent ? Math.abs(parseInt(data.percent)) : 0,
      quota_width = parseInt(quota / 100 * width),
      pos = $(obj).position();

    this.env.indicator_width = width;
    this.env.indicator_height = height;
@@ -4658,11 +4697,11 @@

  this.html2plain = function(htmlText, id)
  {
    var url = this.env.bin_path+'html2text.php';
    var rcmail = this;
    var rcmail = this,
      url = this.env.bin_path + 'html2text.php';

    this.set_busy(true, 'converting');
    console.log('HTTP POST: '+url);
    console.log('HTTP POST: ' + url);

    $.ajax({ type: 'POST', url: url, data: htmlText, contentType: 'application/octet-stream',
      error: function(o) { rcmail.http_error(o); },
@@ -4730,8 +4769,6 @@
  // handle HTTP response
  this.http_response = function(response)
  {
    var console_msg = '';

    if (response.unlock)
      this.set_busy(false);

@@ -4772,7 +4809,8 @@
        if (this.env.action == 'show') {
          // re-enable commands on move/delete error
          this.enable_command('reply', 'reply-all', 'forward', 'delete', 'mark', 'print', 'open', 'edit', 'viewsource', 'download', true);
        } else if (this.message_list)
        }
        else if (this.message_list)
          this.message_list.init();
        break;

@@ -4795,22 +4833,20 @@
      case 'search':
      case 'list':
        if (this.task == 'mail') {
          if (this.message_list && (response.action == 'list' || response.action == 'search')) {
            this.msglist_select(this.message_list);
            this.expand_threads();
          }
          this.enable_command('show', 'expunge', 'select-all', 'select-none', 'sort', (this.env.messagecount > 0));
          this.enable_command('purge', this.purge_mailbox_test());

          this.enable_command('expand-all', 'expand-unread', 'collapse-all', this.env.threading && this.env.messagecount);

          if (response.action == 'list')
          if (response.action == 'list' || response.action == 'search') {
            this.msglist_select(this.message_list);
            this.expand_threads();
            this.triggerEvent('listupdate', { folder:this.env.mailbox, rowcount:this.message_list.rowcount });
          }
        }
        else if (this.task == 'addressbook') {
          this.enable_command('export', (this.contact_list && this.contact_list.rowcount > 0));

          if (response.action == 'list') {
          if (response.action == 'list' || response.action == 'search') {
            this.enable_command('group-create', this.env.address_sources[this.env.source].groups);
            this.enable_command('group-rename', 'group-delete', this.env.address_sources[this.env.source].groups && this.env.group);
            this.triggerEvent('listupdate', { folder:this.env.source, rowcount:this.contact_list.rowcount });
@@ -4937,7 +4973,7 @@
    var type;
    for (var n=0; n<form.elements.length; n++) {
      type = form.elements[n];
      if (type=='hidden')
      if (type == 'hidden')
        continue;

      form.elements[n].disabled = lock;