Changeset 4014 in subversion

Timestamp:

Sep 29, 2010 9:46:30 AM (3 years ago)

Author:

alec

Message:

• Use custom sorting when SORT is disabled by IMAP admin (#1486959)

Location:

trunk/roundcubemail

Files:

• CHANGELOG (modified) (1 diff)
• program/include/rcube_imap.php (modified) (13 diffs)

trunk/roundcubemail/CHANGELOG

@@ -8 +8 @@
 - Add option to "Return receipt" will be always checked (#1486352)
 - Fix HTML to plain text conversion doesn't handle citation blocks (#1486921)
+- Use custom sorting when SORT is disabled by IMAP admin (#1486959)
 
 RELEASE 0.4.1

trunk/roundcubemail/program/include/rcube_imap.php

@@ -63 +63 @@
     private $search_sort_field = '';
     private $search_threads = false;
+    private $search_sorted = false;
     private $db_header_fields = array('idx', 'uid', 'subject', 'from', 'to', 'cc', 'date', 'size');
     private $options = array('auth_method' => 'check');
@@ -293 +294 @@
      * @param  string  Charset of search string
      * @param  string  Sorting field
-     */
-    function set_search_set($str=null, $msgs=null, $charset=null, $sort_field=null, $threads=false)
+     * @param  string  True if set is sorted (SORT was used for searching)
+     */
+    function set_search_set($str=null, $msgs=null, $charset=null, $sort_field=null, $threads=false, $sorted=false)
     {
         if (is_array($str) && $msgs == null)
@@ -308 +310 @@
         $this->search_sort_field = $sort_field;
         $this->search_threads    = $threads;
+        $this->search_sorted     = $sorted;
     }
 
@@ -322 +325 @@
                 $this->search_sort_field,
                 $this->search_threads,
+                $this->search_sorted,
             );
     }
@@ -616 +620 @@
         }
         // use SORT command
-        else if ($this->get_capability('SORT')) {
-            if ($msg_index = $this->conn->sort($mailbox, $this->sort_field, $this->skip_deleted ? 'UNDELETED' : '')) {
+        else if ($this->get_capability('SORT') &&
+            // Courier-IMAP provides SORT capability but allows to disable it by admin (#1486959)
+            ($msg_index = $this->conn->sort($mailbox, $this->sort_field, $this->skip_deleted ? 'UNDELETED' : '')) !== false
+        ) {
+            if (!empty($msg_index)) {
                 list($begin, $end) = $this->_get_message_range(count($msg_index), $page);
                 $max = max($msg_index);
@@ -859 +866 @@
 
         // sorted messages, so we can first slice array and then fetch only wanted headers
-        if ($this->get_capability('SORT')) { // SORT searching result
+        if ($this->search_sorted) { // SORT searching result
             // reset search set if sorting field has been changed
             if ($this->sort_field && $this->search_sort_field != $this->sort_field)
@@ -1169 +1176 @@
             }
             // sort with SORT command
-            else if ($this->get_capability('SORT')) {
+            else if ($this->search_sorted) {
                 if ($this->sort_field && $this->search_sort_field != $this->sort_field)
                     $this->search('', $this->search_string, $this->search_charset, $this->sort_field);
@@ -1225 +1232 @@
         }
         // fetch complete message index
-        else if ($this->get_capability('SORT')) {
-            $a_index = $this->conn->sort($mailbox,
-                $this->sort_field, $this->skip_deleted ? 'UNDELETED' : '');
-
-            if ($a_index !== false && $this->sort_order == 'DESC')
+        else if ($this->get_capability('SORT') &&
+            ($a_index = $this->conn->sort($mailbox,
+                $this->sort_field, $this->skip_deleted ? 'UNDELETED' : '')) !== false
+        ) {
+            if ($this->sort_order == 'DESC')
                 $a_index = array_reverse($a_index);
 
@@ -1440 +1447 @@
         $results = $this->_search_index($mailbox, $str, $charset, $sort_field);
 
-        $this->set_search_set($str, $results, $charset, $sort_field, (bool)$this->threading);
+        $this->set_search_set($str, $results, $charset, $sort_field, (bool)$this->threading,
+            $this->threading || $this->search_sorted ? true : false);
 
         return $results;
@@ -1477 +1485 @@
                 );
             }
-        }
-        else if ($sort_field && $this->get_capability('SORT')) {
+
+            return $a_messages;
+        }
+        
+        if ($sort_field && $this->get_capability('SORT')) {
             $charset = $charset ? $charset : $this->default_charset;
             $a_messages = $this->conn->sort($mailbox, $sort_field, $criteria, false, $charset);
@@ -1487 +1498 @@
                 $a_messages = $this->conn->sort($mailbox, $sort_field,
                     $this->convert_criteria($criteria, $charset), false, 'US-ASCII');
+
+            if ($a_messages !== false) {
+                $this->search_sorted = true;
+                return $a_messages;
+            }
+        }
+
+        if ($orig_criteria == 'ALL') {
+            $max = $this->_messagecount($mailbox);
+            $a_messages = $max ? range(1, $max) : array();
         }
         else {
-            if ($orig_criteria == 'ALL') {
-                $max = $this->_messagecount($mailbox);
-                $a_messages = $max ? range(1, $max) : array();
-            }
-            else {
+            $a_messages = $this->conn->search($mailbox,
+                ($charset ? "CHARSET $charset " : '') . $criteria);
+
+            // Error, try with US-ASCII (some servers may support only US-ASCII)
+            if ($a_messages === false && $charset && $charset != 'US-ASCII')
                 $a_messages = $this->conn->search($mailbox,
-                    ($charset ? "CHARSET $charset " : '') . $criteria);
-
-                // Error, try with US-ASCII (some servers may support only US-ASCII)
-                if ($a_messages === false && $charset && $charset != 'US-ASCII')
-                    $a_messages = $this->conn->search($mailbox,
-                        'CHARSET US-ASCII ' . $this->convert_criteria($criteria, $charset));
-
-                // I didn't found that SEARCH should return sorted IDs
-                    if (is_array($a_messages) && !$this->sort_field)
-                    sort($a_messages);
-            }
-        }
-
-        // update messagecount cache ?
-//      $a_mailbox_cache = get_cache('messagecount');
-//      $a_mailbox_cache[$mailbox][$criteria] = sizeof($a_messages);
-//      $this->update_cache('messagecount', $a_mailbox_cache);
+                    'CHARSET US-ASCII ' . $this->convert_criteria($criteria, $charset));
+
+            // I didn't found that SEARCH should return sorted IDs
+            if (is_array($a_messages) && !$this->sort_field)
+                sort($a_messages);
+        }
+
+        $this->search_sorted = false;
 
         return $a_messages;
@@ -1592 +1604 @@
         else { // ($sort_field == 'date' && $this->threading != 'REFS')
             // use SORT command
-            if ($this->get_capability('SORT')) {
-                $a_index = $this->conn->sort($mailbox, $this->sort_field,
-                        !empty($ids) ? $ids : ($this->skip_deleted ? 'UNDELETED' : ''));
-
+            if ($this->get_capability('SORT') && 
+                ($a_index = $this->conn->sort($mailbox, $this->sort_field,
+                        !empty($ids) ? $ids : ($this->skip_deleted ? 'UNDELETED' : ''))) !== false
+            ) {
                     // return unsorted tree if we've got no index data
                     if (!$a_index)
@@ -1673 +1685 @@
         if (!empty($this->search_string))
             $this->search_set = $this->search('', $this->search_string, $this->search_charset,
-                $this->search_sort_field, $this->search_threads);
+                $this->search_sort_field, $this->search_threads, $this->search_sorted);
 
         return $this->get_search_set();