From 163a13c70f0268a5ea61ca8da9ad1d47f155fc2f Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Mon, 06 Oct 2008 05:37:07 -0400
Subject: [PATCH] - added 'flagged' class handling in set_message()

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

diff --git a/program/js/app.js b/program/js/app.js
index 2e0a1ea..85dd42f 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)
@@ -339,7 +337,6 @@
         break;
       }
 
-
     // enable basic commands
     this.enable_command('logout', true);
 
@@ -353,7 +350,6 @@
     // start keep-alive interval
     this.start_keepalive();
     
-    
     // execute all foreign onload scripts
     for (var i=0; i<this.onloads.length; i++)
       {
@@ -364,7 +360,6 @@
       }
     };
 
-
   // start interval for keep-alive/recent_check signal
   this.start_keepalive = function()
     {
@@ -373,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)
   {
@@ -414,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()
@@ -477,11 +470,9 @@
     };
 
 
-
   /*********************************************************/
   /*********       client command interface        *********/
   /*********************************************************/
-
 
   // execute a specific command on the web client
   this.command = function(command, props, obj)
@@ -501,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'))
@@ -758,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';
@@ -832,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();
@@ -1073,7 +1067,6 @@
     return obj ? false : true;
     };
 
-
   // set command enabled or disabled
   this.enable_command = function()
     {
@@ -1091,7 +1084,6 @@
       }
       return true;
     };
-
 
   // lock/unlock interface
   this.set_busy = function(a, message)
@@ -1122,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)
     {
@@ -1131,7 +1122,6 @@
     else
       return name;
     };
-
 
   // switch to another application task
   this.switch_task = function(task)
@@ -1146,7 +1136,6 @@
     this.redirect(url);
     };
 
-
   this.get_task_url = function(task, url)
     {
     if (!url)
@@ -1155,7 +1144,6 @@
     return url.replace(/_task=[a-z]+/, '_task='+task);
     };
     
-  
   // called when a request timed out
   this.request_timed_out = function()
     {
@@ -1167,7 +1155,6 @@
   /*********************************************************/
   /*********        event handling methods         *********/
   /*********************************************************/
-
 
   this.doc_mouse_up = function(e)
   {
@@ -1265,7 +1252,6 @@
     return rcube_event.get_button(e) == 2 ? true : rcube_event.cancel(e);
     };
 
-
   this.msglist_select = function(list)
     {
     if (this.preview_timer)
@@ -1293,7 +1279,6 @@
       this.show_contentframe(false);
     };
 
-
   this.msglist_dbl_click = function(list)
     {
       if (this.preview_timer)
@@ -1305,7 +1290,6 @@
     else if (uid)
       this.show_message(uid, false, false);
     };
-
 
   this.msglist_keypress = function(list)
     {
@@ -1319,7 +1303,6 @@
       list.shiftkey = false;
     };
 
-
   this.msglist_get_preview = function()
   {
     var uid = this.get_single_uid();
@@ -1328,7 +1311,6 @@
     else if (this.env.contentframe)
       this.show_contentframe(false);
   };
-  
   
   this.check_droptarget = function(id)
   {
@@ -1345,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];
@@ -1364,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)
     {
@@ -1396,7 +1385,6 @@
     if (!show && this.busy)
       this.set_busy(false);
     };
-
 
   // list a specific page
   this.list_page = function(page)
@@ -1420,7 +1408,6 @@
         this.list_contacts(this.env.source, page);
       }
     };
-
 
   // list messages of a specific mailbox
   this.list_mailbox = function(mbox, page, sort)
@@ -1477,7 +1464,6 @@
       }
     };
 
-
   // send remote request to load message list
   this.list_mailbox_remote = function(mbox, page, add_url)
     {
@@ -1489,7 +1475,6 @@
     this.set_busy(true, 'loading');
     this.http_request('list', url+add_url, true);
     };
-
 
   this.expunge_mailbox = function(mbox)
     {
@@ -1508,7 +1493,6 @@
     var url = '_mbox='+urlencode(mbox);
     this.http_post('expunge', url+add_url, lock);
     };
-
 
   this.purge_mailbox = function(mbox)
     {
@@ -1540,6 +1524,109 @@
       || 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);
+      }
+
+    if (rows[uid].flagged && rows[uid].classname.indexOf('flagged')<0)
+      {
+      rows[uid].classname += ' flagged';
+      this.set_classname(rows[uid].obj, 'flagged', true);
+      }
+    else if (!rows[uid].flagged && rows[uid].classname.indexOf('flagged')>=0)
+      {
+      rows[uid].classname = rows[uid].classname.replace(/\s*flagged/, '');
+      this.set_classname(rows[uid].obj, 'flagged', false);
+      }
+
+    this.set_message_icon(uid);
+    }
 
   // move selected messages to the specified mailbox
   this.move_messages = function(mbox)
@@ -1604,7 +1691,6 @@
       this.permanently_remove_messages();
   };
 
-
   // delete the selected messages permanently
   this.permanently_remove_messages = function()
     {
@@ -1639,20 +1725,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);
         }
       }
     }
@@ -1664,7 +1739,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)
@@ -1727,96 +1801,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);
   };
@@ -1828,7 +1824,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);
@@ -1841,7 +1837,7 @@
     {
       uid = a_uids[i];
       if (rows[uid]) {
-        if (rows[uid].classname.indexOf('deleted')<0)
+        if (!rows[uid].deleted)
         {
           all_deleted = false;
           break;
@@ -1857,48 +1853,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 = '';
@@ -1910,22 +1873,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;
         }
@@ -1937,7 +1885,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
@@ -1957,12 +1904,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);
         }
       }
   };
@@ -1991,7 +1933,6 @@
   /*********        message compose methods        *********/
   /*********************************************************/
   
-  
   // checks the input fields before sending a message
   this.check_compose_input = function()
     {
@@ -1999,8 +1940,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);
@@ -2046,6 +1996,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';
     }
@@ -2056,7 +2013,6 @@
     this.spellcheck_ready = (s=='check_spelling' || s=='ready');
     this.enable_command('spellcheck', this.spellcheck_ready);
     };
-
 
   this.set_draft_id = function(id)
     {
@@ -2073,7 +2029,6 @@
     // Unlock interface now that saving is complete
     this.busy = false;
     };
-
 
   this.compose_field_hash = function(save)
     {
@@ -2108,7 +2063,6 @@
     return str;
     };
     
-  
   this.change_identity = function(obj)
     {
     if (!obj || !obj.options)
@@ -2192,7 +2146,6 @@
     return true;
     };
 
-
   this.show_attachment_form = function(a)
     {
     if (!this.gui_objects.uploadbox)
@@ -2223,7 +2176,6 @@
     
     return true;  
     };
-
 
   // upload attachment file
   this.upload_file = function(form)
@@ -2274,7 +2226,6 @@
     this.gui_objects.attachmentform = form;
     return true;
     };
-
 
   // add file name to attachment list
   // called from upload page
@@ -2348,7 +2299,6 @@
     return true;
     };
 
-
   this.sent_successfully = function(type, msg)
     {
     this.list_mailbox();
@@ -2359,7 +2309,6 @@
   /*********************************************************/
   /*********     keyboard live-search methods      *********/
   /*********************************************************/
-
 
   // handler for keyboard events on address-fields
   this.ksearch_keypress = function(e, obj)
@@ -2430,7 +2379,6 @@
     return true;
     };
 
-
   this.insert_recipient = function(id)
   {
     if (!this.env.contacts[id] || !this.ksearch_input)
@@ -2452,7 +2400,6 @@
     if (this.ksearch_input.setSelectionRange)
       this.ksearch_input.setSelectionRange(cpos, cpos);
   };
-
 
   // address search processor
   this.ksearch_get_results = function()
@@ -2554,7 +2501,6 @@
       this.ksearch_hide();
     };
 
-
   this.ksearch_blur = function(e, obj)
     {
     if (this.ksearch_timer)
@@ -2576,18 +2522,15 @@
     };
 
 
-
   /*********************************************************/
   /*********         address book methods          *********/
   /*********************************************************/
-
 
   this.contactlist_keypress = function(list)
     {
       if (list.key_pressed == list.DELETE_KEY)
         this.command('delete');
     };
-
 
   this.contactlist_select = function(list)
     {
@@ -2606,7 +2549,6 @@
 
       return false;
     };
-
 
   this.list_contacts = function(src, page)
     {
@@ -2650,7 +2592,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)
     {
@@ -2670,7 +2611,6 @@
     this.set_busy(true, 'loading');
     this.http_request('list', url, true);
     };
-
 
   // load contact record
   this.load_contact = function(cid, action, framed)
@@ -2696,13 +2636,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()
@@ -2740,7 +2680,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)
@@ -2813,7 +2752,6 @@
     return true;
     };
 
-
   this.delete_identity = function(id)
     {
     // exit if no mailbox specified or if selection is empty
@@ -2828,7 +2766,6 @@
     this.goto_url('delete-identity', '_iid='+id, true);
     return true;
     };
-
 
   this.focus_subscription = function(id)
     {
@@ -2855,7 +2792,6 @@
         }
     }
 
-
   this.unfocus_subscription = function(id)
     {
       var row;
@@ -2866,7 +2802,6 @@
       else
         this.set_classname(this.subscription_list.frame, 'droptarget', false);
     }
-
 
   this.subscription_select = function(list)
     {
@@ -2882,7 +2817,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)+']+$');
@@ -2897,7 +2831,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)
@@ -2925,7 +2858,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
@@ -2961,7 +2893,6 @@
       }
     };
 
-
   // remove the input field and write the current mailbox name to the table cell
   this.reset_folder_rename = function()
     {
@@ -2972,7 +2903,6 @@
       
     this.edit_folder = null;
     };
-
 
   // handler for keyboard events on the input field
   this.name_input_keypress = function(e)
@@ -3003,7 +2933,6 @@
       this.reset_folder_rename();
     };
 
-
   // delete a specific mailbox with all its messages
   this.delete_folder = function(id)
     {
@@ -3022,7 +2951,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)
@@ -3099,7 +3027,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)
     {
@@ -3126,7 +3053,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)
@@ -3155,13 +3081,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)
     {
@@ -3169,7 +3093,6 @@
       this.http_post('unsubscribe', '_mbox='+urlencode(folder));
     };
     
-
   // helper method to find a specific mailbox row ID
   this.get_folder_row_id = function(folder)
     {
@@ -3207,7 +3130,6 @@
   /*********           GUI functionality           *********/
   /*********************************************************/
 
-
   // eable/disable buttons for page shifting
   this.set_page_buttons = function()
     {
@@ -3216,7 +3138,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)
@@ -3350,7 +3271,6 @@
       }
     };
 
-
   // set/unset a specific class name
   this.set_classname = function(obj, classname, set)
     {
@@ -3361,14 +3281,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)
@@ -3404,7 +3322,6 @@
       this.message_timer = window.setTimeout(function(){ ref.hide_message(); }, this.message_time);
     };
 
-
   // make a message row disapear
   this.hide_message = function()
     {
@@ -3414,7 +3331,6 @@
       this.gui_objects.message.onmousedown = null;
       }
     };
-
 
   // mark a mailbox as selected and set environment variable
   this.select_folder = function(name, old)
@@ -3448,7 +3364,6 @@
 
     return null;
   };
-
 
   // for reordering column array, Konqueror workaround
   this.set_message_coltypes = function(coltypes) 
@@ -3502,7 +3417,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';
@@ -3510,8 +3429,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)
@@ -3521,6 +3438,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';
@@ -3563,7 +3482,6 @@
       }
     };
 
-
   // replace content of row count display
   this.set_rowcount = function(text)
     {
@@ -3573,7 +3491,6 @@
     // update page navigation buttons
     this.set_page_buttons();
     };
-
 
   // replace content of mailboxname display
   this.set_mailboxname = function(content)
@@ -3589,7 +3506,6 @@
       this.gui_objects.quotadisplay.innerHTML = content;
     };
 
-
   // update the mailboxlist
   this.set_unread_count = function(mbox, count, set_title)
     {
@@ -3600,14 +3516,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;
 
@@ -3616,10 +3531,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))
@@ -3630,7 +3545,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,17 +3569,8 @@
         
       this.set_pagetitle(new_title);
       }
-
-      // update page buttons
-      this.set_page_buttons();
     };
 
-  // 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)
     {
@@ -3695,7 +3601,6 @@
     this.enable_command('export', (this.contact_list.rowcount > 0));
     };
 
-
   this.toggle_editor = function(checkbox, textAreaId)
     {
     var ischecked = checkbox.checked;
@@ -3717,14 +3622,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)
@@ -3741,7 +3644,6 @@
         this.set_busy(false);
       }
     };
-
 
   // display all-headers row and fetch raw message headers
   this.load_headers = function(elem)
@@ -3761,7 +3663,6 @@
       this.http_post('headers', '_uid='+this.env.uid);
       }
     }
-
 
   // hide all-headers row
   this.hide_headers = function(elem)
@@ -3823,7 +3724,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
@@ -3842,7 +3742,6 @@
     return this.http_sockets[i];
     };
   
-
   // send a http request to the server
   this.http_request = function(action, querystring, lock)
     {
@@ -3900,11 +3799,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);
@@ -3948,25 +3848,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);
 
@@ -3975,7 +3874,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()
     {
@@ -3983,7 +3881,6 @@
     this.http_request('keep-alive', '_t='+d.getTime());
     };
 
-    
   // send periodic request to check for recent messages
   this.check_for_recent = function()
     {
@@ -4040,7 +3937,6 @@
       return obj.value.length;
     };
 
-
   this.set_caret2start = function(obj)
     {
     if (obj.createTextRange)
@@ -4054,7 +3950,6 @@
 
     obj.focus();
     };
-
 
   // set all fields of a form disabled
   this.lock_form = function(form, lock)
@@ -4076,7 +3971,6 @@
   }  // end object rcube_webmail
 
 
-
 /**
  * Class for sending HTTP requests
  * @constructor
@@ -4086,7 +3980,6 @@
   this.url = '';
   this.busy = false;
   this.xmlhttp = null;
-
 
   // reset object properties
   this.reset = function()
@@ -4103,7 +3996,6 @@
     this.busy = false;
     this.xmlhttp = null;
     }
-
 
   // create HTMLHTTP object
   this.build = function()
@@ -4142,7 +4034,6 @@
     this.xmlhttp.send(null);
     };
 
-
   this.POST = function(url, body, contentType)
     {
     // default value for contentType if not provided
@@ -4175,7 +4066,6 @@
     this.xmlhttp.setRequestHeader('X-RoundCube-Referer', bw.get_cookie('roundcube_sessid'));
     this.xmlhttp.send(req_body);
     };
-
 
   // handle onreadystatechange event
   this.xmlhttp_onreadystatechange = function()
@@ -4236,4 +4126,3 @@
     window.setTimeout('if (window[\''+o+'\'] && window[\''+o+'\'].init) { '+o+'.init(); }',
                       bw.win ? 500 : 200);
   }
-

--
Gitblit v1.9.1