From a164a2e64b65fbe0729f6d4326b0219f1914c139 Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Mon, 06 Oct 2008 05:32:09 -0400
Subject: [PATCH] - #1485463: fixed css classes setting in messages list

---
 program/js/app.js |  465 +++++++++++++++++++++------------------------------------
 1 files changed, 174 insertions(+), 291 deletions(-)

diff --git a/program/js/app.js b/program/js/app.js
index 0f074f3..bf6e8bc 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -64,13 +64,11 @@
       this.env[p] = value;
     };
 
-
   // add a localized label to the client environment
   this.add_label = function(key, value)
     {
     this.labels[key] = value;
     };
-
 
   // add a button to the button list
   this.register_button = function(command, id, type, act, sel, over)
@@ -214,12 +212,6 @@
 
         this.set_page_buttons();
 
-        // focus main window
-        if (this.env.framed && window.parent)
-          window.parent.focus();
-        else
-          window.focus();
-
         // init message compose form
         if (this.env.action=='compose')
           this.init_messageform();
@@ -345,7 +337,6 @@
         break;
       }
 
-
     // enable basic commands
     this.enable_command('logout', true);
 
@@ -359,7 +350,6 @@
     // start keep-alive interval
     this.start_keepalive();
     
-    
     // execute all foreign onload scripts
     for (var i=0; i<this.onloads.length; i++)
       {
@@ -370,7 +360,6 @@
       }
     };
 
-
   // start interval for keep-alive/recent_check signal
   this.start_keepalive = function()
     {
@@ -379,7 +368,6 @@
     else if (this.env.keep_alive && !this.env.framed && this.task!='login')
       this._int = setInterval(function(){ ref.send_keep_alive(); }, this.env.keep_alive * 1000);    
     }
-
 
   this.init_message_row = function(row)
   {
@@ -420,7 +408,6 @@
       row.flagged_icon.onmousedown = function(e) { p.command('toggle_flag', this); };
       }
   };
-
 
   // init message compose form: set focus and eventhandlers
   this.init_messageform = function()
@@ -483,11 +470,9 @@
     };
 
 
-
   /*********************************************************/
   /*********       client command interface        *********/
   /*********************************************************/
-
 
   // execute a specific command on the web client
   this.command = function(command, props, obj)
@@ -507,7 +492,6 @@
 
       return false;
       }
-      
       
    // check input before leaving compose step
    if (this.task=='mail' && this.env.action=='compose' && (command=='list' || command=='mail' || command=='addressbook' || command=='settings'))
@@ -764,7 +748,6 @@
         if (props._row.uid)
           {
           uid = props._row.uid;
-          this.message_list.dont_select = true;
           // toggle flagged/unflagged
           if (this.message_list.rows[uid].flagged)
             flag = 'unflagged';
@@ -838,22 +821,27 @@
       case 'compose':
         var url = this.env.comm_path+'&_action=compose';
        
-        if (this.task=='mail' && this.env.mailbox==this.env.drafts_mailbox)
+        if (this.task=='mail')
           {
-          var uid;
-          if (uid = this.get_single_uid())
-            url += '&_draft_uid='+uid+'&_mbox='+urlencode(this.env.mailbox);
-          }
+    	  url += '&_mbox='+urlencode(this.env.mailbox);
+          
+	  if (this.env.mailbox==this.env.drafts_mailbox)
+	    {
+	    var uid;
+            if (uid = this.get_single_uid())
+              url += '&_draft_uid='+uid;
+            }
+	  }
         // modify url if we're in addressbook
         else if (this.task=='addressbook')
           {
           // switch to mail compose step directly
           if (props && props.indexOf('@') > 0)
-          {
+            {
             url = this.get_task_url('mail', url);
             this.redirect(url + '&_to='+urlencode(props));
             break;
-          }
+            }
           
           // use contact_id passed as command parameter
           var a_cids = new Array();
@@ -1079,7 +1067,6 @@
     return obj ? false : true;
     };
 
-
   // set command enabled or disabled
   this.enable_command = function()
     {
@@ -1097,7 +1084,6 @@
       }
       return true;
     };
-
 
   // lock/unlock interface
   this.set_busy = function(a, message)
@@ -1128,7 +1114,6 @@
       this.request_timer = window.setTimeout(function(){ ref.request_timed_out(); }, this.env.request_timeout * 1000);
     };
 
-
   // return a localized string
   this.get_label = function(name)
     {
@@ -1137,7 +1122,6 @@
     else
       return name;
     };
-
 
   // switch to another application task
   this.switch_task = function(task)
@@ -1152,7 +1136,6 @@
     this.redirect(url);
     };
 
-
   this.get_task_url = function(task, url)
     {
     if (!url)
@@ -1161,7 +1144,6 @@
     return url.replace(/_task=[a-z]+/, '_task='+task);
     };
     
-  
   // called when a request timed out
   this.request_timed_out = function()
     {
@@ -1173,7 +1155,6 @@
   /*********************************************************/
   /*********        event handling methods         *********/
   /*********************************************************/
-
 
   this.doc_mouse_up = function(e)
   {
@@ -1271,7 +1252,6 @@
     return rcube_event.get_button(e) == 2 ? true : rcube_event.cancel(e);
     };
 
-
   this.msglist_select = function(list)
     {
     if (this.preview_timer)
@@ -1299,7 +1279,6 @@
       this.show_contentframe(false);
     };
 
-
   this.msglist_dbl_click = function(list)
     {
       if (this.preview_timer)
@@ -1311,7 +1290,6 @@
     else if (uid)
       this.show_message(uid, false, false);
     };
-
 
   this.msglist_keypress = function(list)
     {
@@ -1325,7 +1303,6 @@
       list.shiftkey = false;
     };
 
-
   this.msglist_get_preview = function()
   {
     var uid = this.get_single_uid();
@@ -1334,7 +1311,6 @@
     else if (this.env.contentframe)
       this.show_contentframe(false);
   };
-  
   
   this.check_droptarget = function(id)
   {
@@ -1351,13 +1327,15 @@
   /*********     (message) list functionality      *********/
   /*********************************************************/
 
-
   // when user doble-clicks on a row
   this.show_message = function(id, safe, preview)
     {
+    if (!id) return;
+    
     var add_url = '';
     var action = preview ? 'preview': 'show';
     var target = window;
+    
     if (preview && this.env.contentframe && window.frames && window.frames[this.env.contentframe])
       {
       target = window.frames[this.env.contentframe];
@@ -1370,20 +1348,25 @@
     // also send search request to get the right messages
     if (this.env.search_request)
       add_url += '&_search='+this.env.search_request;
-
-    if (id)
+    var url = '&_action='+action+'&_uid='+id+'&_mbox='+urlencode(this.env.mailbox)+add_url;
+    if (action == 'preview' && String(target.location.href).indexOf(url) >= 0)
+      this.show_contentframe(true);
+    else
       {
-      var url = '&_action='+action+'&_uid='+id+'&_mbox='+urlencode(this.env.mailbox)+add_url;
-      if (action == 'preview' && String(target.location.href).indexOf(url) >= 0)
-        this.show_contentframe(true);
-      else
+      this.set_busy(true, 'loading');
+      target.location.href = this.env.comm_path+url;
+      // mark as read and change mbox unread counter
+      if (action == 'preview' && this.message_list && this.message_list.rows[id] && this.message_list.rows[id].unread)
         {
-        this.set_busy(true, 'loading');
-        target.location.href = this.env.comm_path+url;
-        }
+        this.set_message(id, 'unread', false);
+	if (this.env.unread_counts[this.env.mailbox])
+	  {
+	  this.env.unread_counts[this.env.mailbox] -= 1;
+	  this.set_unread_count(this.env.mailbox, this.env.unread_counts[this.env.mailbox], this.env.mailbox == 'INBOX');
+	  }
+	}
       }
     };
-
 
   this.show_contentframe = function(show)
     {
@@ -1402,7 +1385,6 @@
     if (!show && this.busy)
       this.set_busy(false);
     };
-
 
   // list a specific page
   this.list_page = function(page)
@@ -1426,7 +1408,6 @@
         this.list_contacts(this.env.source, page);
       }
     };
-
 
   // list messages of a specific mailbox
   this.list_mailbox = function(mbox, page, sort)
@@ -1483,7 +1464,6 @@
       }
     };
 
-
   // send remote request to load message list
   this.list_mailbox_remote = function(mbox, page, add_url)
     {
@@ -1495,7 +1475,6 @@
     this.set_busy(true, 'loading');
     this.http_request('list', url+add_url, true);
     };
-
 
   this.expunge_mailbox = function(mbox)
     {
@@ -1514,7 +1493,6 @@
     var url = '_mbox='+urlencode(mbox);
     this.http_post('expunge', url+add_url, lock);
     };
-
 
   this.purge_mailbox = function(mbox)
     {
@@ -1546,6 +1524,98 @@
       || this.env.mailbox.match('^' + RegExp.escape(this.env.junk_mailbox) + RegExp.escape(this.env.delimiter))));
   };
 
+  // set message icon
+  this.set_message_icon = function(uid)
+  {
+    var icn_src;
+    var rows = this.message_list.rows;
+
+    if (!rows[uid])
+      return false;
+
+    if (rows[uid].deleted && this.env.deletedicon)
+      icn_src = this.env.deletedicon;
+    else if (rows[uid].replied && this.env.repliedicon)
+      {
+      if (rows[uid].forwarded && this.env.forwardedrepliedicon)
+        icn_src = this.env.forwardedrepliedicon;
+      else
+        icn_src = this.env.repliedicon;
+      }
+    else if (rows[uid].forwarded && this.env.forwardedicon)
+      icn_src = this.env.forwardedicon;
+    else if (rows[uid].unread && this.env.unreadicon)
+      icn_src = this.env.unreadicon;
+    else if (this.env.messageicon)
+      icn_src = this.env.messageicon;
+      
+    if (icn_src && rows[uid].icon)
+      rows[uid].icon.src = icn_src;
+
+    icn_src = '';
+    
+    if (rows[uid].flagged && this.env.flaggedicon)
+      icn_src = this.env.flaggedicon;
+    else if (this.env.unflaggedicon)
+      icn_src = this.env.unflaggedicon;
+
+    if (rows[uid].flagged_icon && icn_src)
+      rows[uid].flagged_icon.src = icn_src;
+  }
+
+  // set message status
+  this.set_message_status = function(uid, flag, status)
+    {
+    var rows = this.message_list.rows;
+
+    if (!rows[uid]) return false;
+
+    if (flag == 'unread')
+      rows[uid].unread = status;
+    else if(flag == 'deleted')
+      rows[uid].deleted = status;
+    else if (flag == 'replied')
+      rows[uid].replied = status;
+    else if (flag == 'forwarded')
+      rows[uid].forwarded = status;
+    else if (flag == 'flagged')
+      rows[uid].flagged = status;
+    }
+
+  // set message row status, class and icon
+  this.set_message = function(uid, flag, status)
+    {
+    var rows = this.message_list.rows;
+
+    if (!rows[uid]) return false;
+    
+    if (flag)
+      this.set_message_status(uid, flag, status);
+    
+    if (rows[uid].unread && rows[uid].classname.indexOf('unread')<0)
+      {
+      rows[uid].classname += ' unread';
+      this.set_classname(rows[uid].obj, 'unread', true);
+      }
+    else if (!rows[uid].unread && rows[uid].classname.indexOf('unread')>=0)
+      {
+      rows[uid].classname = rows[uid].classname.replace(/\s*unread/, '');
+      this.set_classname(rows[uid].obj, 'unread', false);
+      }
+    
+    if (rows[uid].deleted && rows[uid].classname.indexOf('deleted')<0)
+      {
+      rows[uid].classname += ' deleted';
+      this.set_classname(rows[uid].obj, 'deleted', true);
+      }
+    else if (!rows[uid].deleted && rows[uid].classname.indexOf('deleted')>=0)
+      {
+      rows[uid].classname = rows[uid].classname.replace(/\s*deleted/, '');
+      this.set_classname(rows[uid].obj, 'deleted', false);
+      }
+
+    this.set_message_icon(uid);
+    }
 
   // move selected messages to the specified mailbox
   this.move_messages = function(mbox)
@@ -1610,7 +1680,6 @@
       this.permanently_remove_messages();
   };
 
-
   // delete the selected messages permanently
   this.permanently_remove_messages = function()
     {
@@ -1645,20 +1714,9 @@
         else
         {
           rows[id].deleted = true;
-        
-          if (rows[id].classname.indexOf('deleted')<0)
-          {
-            rows[id].classname += ' deleted';
-            this.set_classname(rows[id].obj, 'deleted', true);
-          }
           if (this.env.read_when_deleted)
-          {
-            rows[id].classname = rows[id].classname.replace(/\s*unread/, '');
-            this.set_classname(rows[id].obj, 'unread', false);
-          }
-        
-          if (rows[id].icon && this.env.deletedicon)
-            rows[id].icon.src = this.env.deletedicon;
+    	    rows[id].unread = false;
+	  this.set_message(id);
         }
       }
     }
@@ -1670,7 +1728,6 @@
     // send request to server
     this.http_post(action, '_uid='+a_uids.join(',')+'&_mbox='+urlencode(this.env.mailbox)+add_url, lock);
   };
-
 
   // set a specific flag to one or more messages
   this.mark_message = function(flag, uid)
@@ -1733,96 +1790,18 @@
   this.toggle_read_status = function(flag, a_uids)
   {
     // mark all message rows as read/unread
-    var icn_src;
-    var rows = this.message_list.rows;
     for (var i=0; i<a_uids.length; i++)
-      {
-      uid = a_uids[i];
-      if (rows[uid])
-        {
-        rows[uid].unread = (flag=='unread' ? true : false);
-        
-        if (rows[uid].classname.indexOf('unread')<0 && rows[uid].unread)
-          {
-          rows[uid].classname += ' unread';
-          this.set_classname(rows[uid].obj, 'unread', true);
-
-          if (this.env.unreadicon)
-            icn_src = this.env.unreadicon;
-          }
-        else if (!rows[uid].unread)
-          {
-          rows[uid].classname = rows[uid].classname.replace(/\s*unread/, '');
-          this.set_classname(rows[uid].obj, 'unread', false);
-
-          if (this.env.messageicon)
-            icn_src = this.env.messageicon;
-          }
-
-        if (rows[uid].icon && icn_src 
-            && !(rows[uid].replied && this.env.repliedicon)
-	    && !(rows[uid].forwarded && this.env.forwardedicon)
-            && !(rows[uid].deleted && this.env.deletedicon))
-          rows[uid].icon.src = icn_src;
-        }
-      }
+      this.set_message(a_uids[i], 'unread', (flag=='unread' ? true : false));
 
     this.http_post('mark', '_uid='+a_uids.join(',')+'&_flag='+flag);
   };
 
-  // set class to read/unread
-  this.mark_as_read_from_preview = function(uid)
-  {
-    var icn_src;
-    var rows = parent.rcmail.message_list.rows;
-    if(rows[uid].unread)
-      {
-        rows[uid].unread = false;
-        rows[uid].classname = rows[uid].classname.replace(/\s*unread/, '');
-        parent.rcmail.set_classname(rows[uid].obj, 'unread', false);
-
-        if (rows[uid].deleted && parent.rcmail.env.deletedicon)
-          icn_src = parent.rcmail.env.deletedicon;
-        else if (rows[uid].replied && parent.rcmail.env.repliedicon)
-	  {
-	  if (rows[uid].forwarded && parent.rcmail.env.forwardedrepliedicon)
-	    icn_src = parent.rcmail.env.forwardedrepliedicon;
-          else
-	    icn_src = parent.rcmail.env.repliedicon;
-          }
-	else if (rows[uid].forwarded && parent.rcmail.env.forwardedicon)
-          icn_src = parent.rcmail.env.forwardedicon;
-        else if (parent.rcmail.env.messageicon)
-          icn_src = parent.rcmail.env.messageicon;
-      
-        if (rows[uid].icon && icn_src)
-          rows[uid].icon.src = icn_src;
-      }
-  }
-  
-  
   // set image to flagged or unflagged
   this.toggle_flagged_status = function(flag, a_uids)
   {
     // mark all message rows as flagged/unflagged
-    var icn_src;
-    var rows = this.message_list.rows;
     for (var i=0; i<a_uids.length; i++)
-      {
-      uid = a_uids[i];
-      if (rows[uid])
-        {
-        rows[uid].flagged = (flag=='flagged' ? true : false);
-
-        if (rows[uid].flagged && this.env.flaggedicon)
-          icn_src = this.env.flaggedicon;
-        else if (this.env.unflaggedicon)
-          icn_src = this.env.unflaggedicon;
-
-        if (rows[uid].flagged_icon && icn_src)
-          rows[uid].flagged_icon.src = icn_src;
-        }
-      }
+      this.set_message(a_uids[i], 'flagged', (flag=='flagged' ? true : false));
 
     this.http_post('mark', '_uid='+a_uids.join(',')+'&_flag='+flag);
   };
@@ -1834,7 +1813,7 @@
     
     if (a_uids.length==1)
     {
-      if (!rows.length || (rows[a_uids[0]] && rows[a_uids[0]].classname.indexOf('deleted') < 0))
+      if (!rows.length || (rows[a_uids[0]] && !rows[a_uids[0]].deleted))
         this.flag_as_deleted(a_uids);
       else
         this.flag_as_undeleted(a_uids);
@@ -1847,7 +1826,7 @@
     {
       uid = a_uids[i];
       if (rows[uid]) {
-        if (rows[uid].classname.indexOf('deleted')<0)
+        if (!rows[uid].deleted)
         {
           all_deleted = false;
           break;
@@ -1863,48 +1842,15 @@
     return true;
   };
 
-
   this.flag_as_undeleted = function(a_uids)
   {
-    var icn_src;
-    var rows = this.message_list ? this.message_list.rows : new Array();
-      
     for (var i=0; i<a_uids.length; i++)
-    {
-      uid = a_uids[i];
-      if (rows[uid]) {
-        rows[uid].deleted = false;
-        
-        if (rows[uid].classname.indexOf('deleted') > 0)
-        {
-          rows[uid].classname = rows[uid].classname.replace(/\s*deleted/, '');
-          this.set_classname(rows[uid].obj, 'deleted', false);
-        }
-
-        if (rows[uid].unread && this.env.unreadicon)
-          icn_src = this.env.unreadicon;
-        else if (rows[uid].replied && this.env.repliedicon)
-	  {
-	  if (rows[uid].forwarded && this.env.forwardedrepliedicon)
-            icn_src = this.env.forwardedrepliedicon;
-          else
-	    icn_src = this.env.repliedicon;
-          }
-	else if (rows[uid].forwarded && this.env.forwardedicon)
-          icn_src = this.env.forwardedicon;
-        else if (this.env.messageicon)
-          icn_src = this.env.messageicon;
-
-        if (rows[uid].icon && icn_src)
-          rows[uid].icon.src = icn_src;
-      }
-    }
+      this.set_message(a_uids[i], 'deleted', false);
 
     this.http_post('mark', '_uid='+a_uids.join(',')+'&_flag=undelete');
     return true;
   };
 
-  
   this.flag_as_deleted = function(a_uids)
   {
     var add_url = '';
@@ -1916,22 +1862,7 @@
       uid = a_uids[i];
       if (rows[uid])
         {
-        rows[uid].deleted = true;
-        
-        if (rows[uid].classname.indexOf('deleted')<0)
-          {
-          rows[uid].classname += ' deleted';
-          this.set_classname(rows[uid].obj, 'deleted', true);
-          }
-        if (this.env.read_when_deleted)
-        {
-          rows[uid].classname = rows[uid].classname.replace(/\s*unread/, '');
-          this.set_classname(rows[uid].obj, 'unread', false);
-        }
-          
-        if (rows[uid].icon && this.env.deletedicon)
-          rows[uid].icon.src = this.env.deletedicon;
-
+	this.set_message(uid, 'deleted', true);
         if (rows[uid].unread)
           r_uids[r_uids.length] = uid;
         }
@@ -1943,7 +1874,6 @@
     this.http_post('mark', '_uid='+a_uids.join(',')+'&_flag=delete'+add_url);
     return true;  
   };
-
 
   // flag as read without mark request (called from backend)
   // argument should be a coma-separated list of uids
@@ -1963,12 +1893,7 @@
         {
         rows[uid].unread = false;
         rows[uid].read = true;
-        
-        rows[uid].classname = rows[uid].classname.replace(/\s*unread/, '');
-        this.set_classname(rows[uid].obj, 'unread', false);
-
-        if (rows[uid].icon)
-          rows[uid].icon.src = this.env.deletedicon;
+	this.set_message(uid);
         }
       }
   };
@@ -1997,7 +1922,6 @@
   /*********        message compose methods        *********/
   /*********************************************************/
   
-  
   // checks the input fields before sending a message
   this.check_compose_input = function()
     {
@@ -2005,8 +1929,17 @@
     var input_to = rcube_find_object('_to');
     var input_cc = rcube_find_object('_cc');
     var input_bcc = rcube_find_object('_bcc');
+    var input_from = rcube_find_object('_from');
     var input_subject = rcube_find_object('_subject');
     var input_message = rcube_find_object('_message');
+
+    // check sender (if have no identities)
+    if (input_from.type == 'text' && !rcube_check_email(input_from.value, true))
+      {
+      alert(this.get_label('nosenderwarning'));
+      input_from.focus();
+      return false;
+      }
 
     // check for empty recipient
     var recipients = input_to.value ? input_to.value : (input_cc.value ? input_cc.value : input_bcc.value);
@@ -2052,6 +1985,13 @@
   this.display_spellcheck_controls = function(vis)
   {
     if (this.env.spellcheck) {
+      // stop spellchecking process
+      if (!vis && !this.spellcheck_ready) 
+        {
+	exec_event(this.env.spellcheck.check_link, 'click');
+	this.set_spellcheck_state('ready');
+	}
+			      
       this.env.spellcheck.check_link.style.visibility = vis ? 'visible' : 'hidden';
       this.env.spellcheck.switch_lan_pic.style.visibility = vis ? 'visible' : 'hidden';
     }
@@ -2062,7 +2002,6 @@
     this.spellcheck_ready = (s=='check_spelling' || s=='ready');
     this.enable_command('spellcheck', this.spellcheck_ready);
     };
-
 
   this.set_draft_id = function(id)
     {
@@ -2079,7 +2018,6 @@
     // Unlock interface now that saving is complete
     this.busy = false;
     };
-
 
   this.compose_field_hash = function(save)
     {
@@ -2114,7 +2052,6 @@
     return str;
     };
     
-  
   this.change_identity = function(obj)
     {
     if (!obj || !obj.options)
@@ -2198,7 +2135,6 @@
     return true;
     };
 
-
   this.show_attachment_form = function(a)
     {
     if (!this.gui_objects.uploadbox)
@@ -2229,7 +2165,6 @@
     
     return true;  
     };
-
 
   // upload attachment file
   this.upload_file = function(form)
@@ -2280,7 +2215,6 @@
     this.gui_objects.attachmentform = form;
     return true;
     };
-
 
   // add file name to attachment list
   // called from upload page
@@ -2354,7 +2288,6 @@
     return true;
     };
 
-
   this.sent_successfully = function(type, msg)
     {
     this.list_mailbox();
@@ -2365,7 +2298,6 @@
   /*********************************************************/
   /*********     keyboard live-search methods      *********/
   /*********************************************************/
-
 
   // handler for keyboard events on address-fields
   this.ksearch_keypress = function(e, obj)
@@ -2436,7 +2368,6 @@
     return true;
     };
 
-
   this.insert_recipient = function(id)
   {
     if (!this.env.contacts[id] || !this.ksearch_input)
@@ -2458,7 +2389,6 @@
     if (this.ksearch_input.setSelectionRange)
       this.ksearch_input.setSelectionRange(cpos, cpos);
   };
-
 
   // address search processor
   this.ksearch_get_results = function()
@@ -2560,7 +2490,6 @@
       this.ksearch_hide();
     };
 
-
   this.ksearch_blur = function(e, obj)
     {
     if (this.ksearch_timer)
@@ -2582,18 +2511,15 @@
     };
 
 
-
   /*********************************************************/
   /*********         address book methods          *********/
   /*********************************************************/
-
 
   this.contactlist_keypress = function(list)
     {
       if (list.key_pressed == list.DELETE_KEY)
         this.command('delete');
     };
-
 
   this.contactlist_select = function(list)
     {
@@ -2612,7 +2538,6 @@
 
       return false;
     };
-
 
   this.list_contacts = function(src, page)
     {
@@ -2656,7 +2581,6 @@
     target.location.href = this.env.comm_path+(src ? '&_source='+urlencode(src) : '')+(page ? '&_page='+page : '')+add_url;
     };
 
-
   // send remote request to load contacts list
   this.list_contacts_remote = function(src, page)
     {
@@ -2676,7 +2600,6 @@
     this.set_busy(true, 'loading');
     this.http_request('list', url, true);
     };
-
 
   // load contact record
   this.load_contact = function(cid, action, framed)
@@ -2702,13 +2625,13 @@
 
   // copy a contact to the specified target (group or directory)
   this.copy_contact = function(cid, to)
-  {
+    {
     if (!cid)
       cid = this.contact_list.get_selection().join(',');
 
     if (to != this.env.source && cid && this.env.address_sources[to] && !this.env.address_sources[to].readonly)
       this.http_post('copy', '_cid='+urlencode(cid)+'&_source='+urlencode(this.env.source)+'&_to='+urlencode(to));
-  };
+    };
 
 
   this.delete_contacts = function()
@@ -2746,7 +2669,6 @@
     this.http_post('delete', '_cid='+urlencode(a_cids.join(','))+'&_source='+urlencode(this.env.source)+'&_from='+(this.env.action ? this.env.action : '')+qs);
     return true;
     };
-
 
   // update a contact record in the list
   this.update_contact_row = function(cid, cols_arr)
@@ -2819,7 +2741,6 @@
     return true;
     };
 
-
   this.delete_identity = function(id)
     {
     // exit if no mailbox specified or if selection is empty
@@ -2834,7 +2755,6 @@
     this.goto_url('delete-identity', '_iid='+id, true);
     return true;
     };
-
 
   this.focus_subscription = function(id)
     {
@@ -2861,7 +2781,6 @@
         }
     }
 
-
   this.unfocus_subscription = function(id)
     {
       var row;
@@ -2872,7 +2791,6 @@
       else
         this.set_classname(this.subscription_list.frame, 'droptarget', false);
     }
-
 
   this.subscription_select = function(list)
     {
@@ -2888,7 +2806,6 @@
       this.gui_objects.createfolderhint.innerHTML = this.env.folder ? this.get_label('addsubfolderhint') : '';
     };
 
-
   this.subscription_move_folder = function(list)
     {
     var reg = RegExp('['+RegExp.escape(this.env.delimiter)+']?[^'+RegExp.escape(this.env.delimiter)+']+$');
@@ -2903,7 +2820,6 @@
     this.drag_active = false;
     this.unfocus_subscription(this.get_folder_row_id(this.env.dstfolder));
     };
-
 
   // tell server to create and subscribe a new mailbox
   this.create_folder = function(name)
@@ -2931,7 +2847,6 @@
     else if (form.elements['_folder_name'])
       form.elements['_folder_name'].focus();
     };
-
 
   // start renaming the mailbox name.
   // this will replace the name string with an input field
@@ -2967,7 +2882,6 @@
       }
     };
 
-
   // remove the input field and write the current mailbox name to the table cell
   this.reset_folder_rename = function()
     {
@@ -2978,7 +2892,6 @@
       
     this.edit_folder = null;
     };
-
 
   // handler for keyboard events on the input field
   this.name_input_keypress = function(e)
@@ -3009,7 +2922,6 @@
       this.reset_folder_rename();
     };
 
-
   // delete a specific mailbox with all its messages
   this.delete_folder = function(id)
     {
@@ -3028,7 +2940,6 @@
         this.gui_objects.createfolderhint.innerHTML = '';
       }
     };
-
 
   // add a new folder to the subscription list by cloning a folder row
   this.add_folder_row = function(name, display_name, replace, before)
@@ -3105,7 +3016,6 @@
       document.getElementById(id).scrollIntoView();
     };
 
-
   // replace an existing table row with a new folder line
   this.replace_folder_row = function(oldfolder, newfolder, display_name, before)
     {
@@ -3132,7 +3042,6 @@
       form.elements['_folder_newname'].value = '';
       }
     };
-
 
   // remove the table row of a specific mailbox from the table
   // (the row will not be removed, just hidden)
@@ -3161,13 +3070,11 @@
       form.elements['_folder_newname'].value = '';
     };
 
-
   this.subscribe_folder = function(folder)
     {
     if (folder)
       this.http_post('subscribe', '_mbox='+urlencode(folder));
     };
-
 
   this.unsubscribe_folder = function(folder)
     {
@@ -3175,7 +3082,6 @@
       this.http_post('unsubscribe', '_mbox='+urlencode(folder));
     };
     
-
   // helper method to find a specific mailbox row ID
   this.get_folder_row_id = function(folder)
     {
@@ -3213,7 +3119,6 @@
   /*********           GUI functionality           *********/
   /*********************************************************/
 
-
   // eable/disable buttons for page shifting
   this.set_page_buttons = function()
     {
@@ -3222,7 +3127,6 @@
     this.enable_command('previouspage', (this.env.current_page > 1));
     this.enable_command('firstpage', (this.env.current_page > 1));
     }
-
 
   // set button to a specific state
   this.set_button = function(command, state)
@@ -3356,7 +3260,6 @@
       }
     };
 
-
   // set/unset a specific class name
   this.set_classname = function(obj, classname, set)
     {
@@ -3367,14 +3270,12 @@
       obj.className += ' '+classname;
     };
 
-
   // write to the document/window title
   this.set_pagetitle = function(title)
   {
     if (title && document.title)
       document.title = title;
   }
-
 
   // display a system message
   this.display_message = function(msg, type, hold)
@@ -3410,7 +3311,6 @@
       this.message_timer = window.setTimeout(function(){ ref.hide_message(); }, this.message_time);
     };
 
-
   // make a message row disapear
   this.hide_message = function()
     {
@@ -3420,7 +3320,6 @@
       this.gui_objects.message.onmousedown = null;
       }
     };
-
 
   // mark a mailbox as selected and set environment variable
   this.select_folder = function(name, old)
@@ -3454,7 +3353,6 @@
 
     return null;
   };
-
 
   // for reordering column array, Konqueror workaround
   this.set_message_coltypes = function(coltypes) 
@@ -3508,7 +3406,11 @@
     
     var row = document.createElement('TR');
     row.id = 'rcmrow'+uid;
-    row.className = 'message '+(even ? 'even' : 'odd')+(flags.unread ? ' unread' : '')+(flags.deleted ? ' deleted' : '');
+    row.className = 'message'
+	+ (even ? ' even' : ' odd')
+        + (flags.unread ? ' unread' : '')
+	+ (flags.deleted ? ' deleted' : '')
+	+ (flags.flagged ? ' flagged' : '');		    
 
     if (this.message_list.in_selection(uid))
       row.className += ' selected';
@@ -3516,8 +3418,6 @@
     var icon = this.env.messageicon;
     if (flags.deleted && this.env.deletedicon)
       icon = this.env.deletedicon;
-    else if(flags.unread && this.env.unreadicon)
-      icon = this.env.unreadicon;
     else if (flags.replied && this.env.repliedicon)
       {
       if (flags.forwarded && this.env.forwardedrepliedicon)
@@ -3527,6 +3427,8 @@
       }
     else if (flags.forwarded && this.env.forwardedicon)
       icon = this.env.forwardedicon;
+    else if(flags.unread && this.env.unreadicon)
+      icon = this.env.unreadicon;
     
     var col = document.createElement('TD');
     col.className = 'icon';
@@ -3559,8 +3461,15 @@
     row.appendChild(col);
 
     this.message_list.insert_row(row, attop);
-    };
 
+    // remove 'old' row
+    if (attop && this.env.pagesize && this.message_list.rowcount > this.env.pagesize)
+      {
+	var uid = this.message_list.get_last_row();
+        this.message_list.remove_row(uid);
+	this.message_list.clear_selection(uid);
+      }
+    };
 
   // replace content of row count display
   this.set_rowcount = function(text)
@@ -3571,7 +3480,6 @@
     // update page navigation buttons
     this.set_page_buttons();
     };
-
 
   // replace content of mailboxname display
   this.set_mailboxname = function(content)
@@ -3587,7 +3495,6 @@
       this.gui_objects.quotadisplay.innerHTML = content;
     };
 
-
   // update the mailboxlist
   this.set_unread_count = function(mbox, count, set_title)
     {
@@ -3598,14 +3505,13 @@
     this.set_unread_count_display(mbox, set_title);
     }
 
-
   // update the mailbox count display
   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];
+      mycount = this.env.unread_counts[mbox] ? this.env.unread_counts[mbox] : 0;
       text_obj = item.getElementsByTagName('a')[0];
       reg = /\s+\([0-9]+\)$/i;
 
@@ -3614,10 +3520,10 @@
           div.className.match(/collapsed/))
         {
         // add children's counters
-        for (var k in this.env.unread_counts)
+        for (var k in this.env.unread_counts) 
           if (k.indexOf(mbox + this.env.delimiter) == 0) {
             childcount += this.env.unread_counts[k];
-          }
+	  }
         }
 
       if (mycount && text_obj.innerHTML.match(reg))
@@ -3628,7 +3534,7 @@
         text_obj.innerHTML = text_obj.innerHTML.replace(reg, '');
 
       // set parent's display
-      reg = new RegExp(RegExp.escape(this.env.delimiter) + '[^' + RegExp.escape(this.env.delimiter) + ']+');
+      reg = new RegExp(RegExp.escape(this.env.delimiter) + '[^' + RegExp.escape(this.env.delimiter) + ']+$');
       if (mbox.match(reg))
         this.set_unread_count_display(mbox.replace(reg, ''), false);
 
@@ -3654,12 +3560,6 @@
       }
     };
 
-  // update parent's mailboxlist (from preview)
-  this.set_unread_count_from_preview = function(mbox, count, set_title)
-  {
-    parent.rcmail.set_unread_count(mbox, count, set_title);
-  }
-  
   // add row to contacts list
   this.add_contact_row = function(cid, cols, select)
     {
@@ -3690,7 +3590,6 @@
     this.enable_command('export', (this.contact_list.rowcount > 0));
     };
 
-
   this.toggle_editor = function(checkbox, textAreaId)
     {
     var ischecked = checkbox.checked;
@@ -3712,14 +3611,12 @@
       }
     };
 
-
   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;
     }
-
 
   // display fetched raw headers
   this.set_headers = function(content)
@@ -3736,7 +3633,6 @@
         this.set_busy(false);
       }
     };
-
 
   // display all-headers row and fetch raw message headers
   this.load_headers = function(elem)
@@ -3756,7 +3652,6 @@
       this.http_post('headers', '_uid='+this.env.uid);
       }
     }
-
 
   // hide all-headers row
   this.hide_headers = function(elem)
@@ -3818,7 +3713,6 @@
     this.redirect(this.env.comm_path+'&_action='+action+querystring, lock);
     };
 
-
   this.http_sockets = new Array();
   
   // find a non-busy socket or create a new one
@@ -3837,7 +3731,6 @@
     return this.http_sockets[i];
     };
   
-
   // send a http request to the server
   this.http_request = function(action, querystring, lock)
     {
@@ -3895,11 +3788,12 @@
   this.http_response = function(request_obj)
     {
     var ctype = request_obj.get_header('Content-Type');
-    if (ctype){
+    if (ctype)
+      {
       ctype = String(ctype).toLowerCase();
       var ctype_array=ctype.split(";");
       ctype = ctype_array[0];
-    }
+      }
 
     if (request_obj.__lock)
       this.set_busy(false);
@@ -3943,25 +3837,24 @@
       case 'getunread':
       case 'list':
         if (this.task == 'mail') {
-          if (this.message_list)
+          if (this.message_list && request_obj.__action == 'list')
             this.msglist_select(this.message_list);
           this.enable_command('show', 'expunge', 'select-all', 'select-none', 'sort', (this.env.messagecount > 0));
           this.enable_command('purge', this.purge_mailbox_test());
         }
         else if (this.task == 'addressbook')
           this.enable_command('export', (this.contact_list && this.contact_list.rowcount > 0));
+
         break;
-    }
+      }
 
     request_obj.reset();
     };
-
 
   // handle HTTP request errors
   this.http_error = function(request_obj)
     {
     //alert('Error sending request: '+request_obj.url+' => HTTP '+request_obj.xmlhttp.status);
-
     if (request_obj.__lock)
       this.set_busy(false);
 
@@ -3970,7 +3863,6 @@
     this.display_message('Unknown Server Error!', 'error');
     };
 
-
   // use an image to send a keep-alive siganl to the server
   this.send_keep_alive = function()
     {
@@ -3978,7 +3870,6 @@
     this.http_request('keep-alive', '_t='+d.getTime());
     };
 
-    
   // send periodic request to check for recent messages
   this.check_for_recent = function()
     {
@@ -4035,7 +3926,6 @@
       return obj.value.length;
     };
 
-
   this.set_caret2start = function(obj)
     {
     if (obj.createTextRange)
@@ -4049,7 +3939,6 @@
 
     obj.focus();
     };
-
 
   // set all fields of a form disabled
   this.lock_form = function(form, lock)
@@ -4071,7 +3960,6 @@
   }  // end object rcube_webmail
 
 
-
 /**
  * Class for sending HTTP requests
  * @constructor
@@ -4081,7 +3969,6 @@
   this.url = '';
   this.busy = false;
   this.xmlhttp = null;
-
 
   // reset object properties
   this.reset = function()
@@ -4098,7 +3985,6 @@
     this.busy = false;
     this.xmlhttp = null;
     }
-
 
   // create HTMLHTTP object
   this.build = function()
@@ -4137,7 +4023,6 @@
     this.xmlhttp.send(null);
     };
 
-
   this.POST = function(url, body, contentType)
     {
     // default value for contentType if not provided
@@ -4170,7 +4055,6 @@
     this.xmlhttp.setRequestHeader('X-RoundCube-Referer', bw.get_cookie('roundcube_sessid'));
     this.xmlhttp.send(req_body);
     };
-
 
   // handle onreadystatechange event
   this.xmlhttp_onreadystatechange = function()
@@ -4231,4 +4115,3 @@
     window.setTimeout('if (window[\''+o+'\'] && window[\''+o+'\'].init) { '+o+'.init(); }',
                       bw.win ? 500 : 200);
   }
-

--
Gitblit v1.9.1