Changeset 291 in subversion

Timestamp:

Aug 2, 2006 6:05:42 PM (7 years ago)

Author:

thomasb

Message:

Message sorting: added patch by Eric; fixed check-for-recent confusion

Location:

trunk/roundcubemail/program

Files:

• include/rcube_imap.inc (modified) (3 diffs)
• steps/mail/check_recent.inc (modified) (1 diff)

trunk/roundcubemail/program/include/rcube_imap.inc

@@ -517 +517 @@
       $headers_sorted = TRUE;
       }
+    // cache is dirty, sync it
+    else if ($this->caching_enabled && $cache_status==-1 && !$recursive)
+      {
+      $this->sync_header_index($mailbox);
+      return $this->_list_headers($mailbox, $page, $this->sort_field, $this->sort_order, TRUE);
+      }
     else
       {
-      $sorter = new rcube_header_sorter();
-
       // retrieve headers from IMAP
       if ($this->get_capability('sort') && ($msg_index = iil_C_Sort($this->conn, $mailbox, $this->sort_field, $this->skip_deleted ? 'UNDELETED' : '')))
-        {
-        $sorter->set_sequence_numbers($msg_index);
-        
+        {        
         $msgs = $msg_index[$begin];
         for ($i=$begin+1; $i < $end; $i++)
@@ -532 +534 @@
       else
         {
-        $msgs = sprintf("%d:%d", $begin+1, $end);        
-        }
-
-
-      // cache is dirty, sync it
-      if ($this->caching_enabled && $cache_status==-1 && !$recursive)
-        {
-        $this->sync_header_index($mailbox);
-        return $this->_list_headers($mailbox, $page, $this->sort_field, $this->sort_order, TRUE);
-        }      
-
+        $msgs = sprintf("%d:%d", $begin+1, $end);
+
+        $i = 0;
+        for ($msg_seqnum = $begin; $msg_seqnum <= $end; $msg_seqnum++)
+          $msg_index[$i++] = $msg_seqnum;
+        }
+
+      // use this class for message sorting
+      $sorter = new rcube_header_sorter();
+      $sorter->set_sequence_numbers($msg_index);
 
       // fetch reuested headers from server
@@ -574 +575 @@
     return array_values($a_msg_headers);
     }
-
-
-
-
-
-function gethdrids($hdr)
-{
-   return $hdr->uid . ',' . $hdr->id;
-}
-
 
 

trunk/roundcubemail/program/steps/mail/check_recent.inc

@@ -41 +41 @@
       $a_headers = array();
       for ($i=$recent_count, $id=$count-$recent_count+1; $i>0; $i--, $id++)
-        $a_headers[] = $IMAP->get_headers($id, NULL, FALSE);
+        {
+        $header = $IMAP->get_headers($id, NULL, FALSE);
+        if ($header->recent)
+          $a_headers[] = $header;
+        }
 
       $commands .= rcmail_js_message_list($a_headers, TRUE);