Changeset 710e274 in github

Timestamp:

Oct 20, 2010 9:33:27 AM (3 years ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

659cf14c

Parents:

8794f16

Message:

• Improve performance of unseen messages counting, use STATUS instead of SELECT+SEARCH (#1487058)

Files:

• CHANGELOG (modified) (1 diff)
• program/include/rcube_imap_generic.php (modified) (3 diffs)

CHANGELOG

@@ -38 +38 @@
 - Fix decoding of e-mail address strings in message headers (#1487068)
 - Fix handling of attachments when Content-Disposition is not inline nor attachment (#1487051)
+- Improve performance of unseen messages counting (#1487058)
 
 RELEASE 0.4.2

program/include/rcube_imap_generic.php

@@ -734 +734 @@
     }
 
+    /**
+     * Executes STATUS comand
+     *
+     * @param string $mailbox Mailbox name
+     * @param array  $items   Requested item names
+     *
+     * @return array Status item-value hash
+     * @access public
+     * @since 0.5-beta
+     */
+    function status($mailbox, $items)
+    {
+            if (empty($mailbox) || empty($items)) {
+                    return false;
+            }
+
+        list($code, $response) = $this->execute('STATUS', array($this->escape($mailbox),
+            '(' . implode(' ', (array) $items) . ')'));
+
+        if ($code == self::ERROR_OK && preg_match('/\* STATUS /i', $response)) {
+            $result   = array();
+            $response = substr($response, 9); // remove prefix "* STATUS "
+
+            list($mbox, $items) = $this->tokenizeResponse($response, 2);
+
+            for ($i=0, $len=count($items); $i<$len; $i += 2) {
+                $result[$items[$i]] = (int) $items[$i+1];
+            }
+
+                        return $result;
+                }
+
+        return false;
+    }
+
     function checkForRecent($mailbox)
     {
@@ -758 +793 @@
                     return $this->data['EXISTS'];
             }
+
+        return false;
+    }
+
+    /**
+     * Returns count of messages without \Seen flag in a specified folder
+     *
+     * @param string $mailbox Mailbox name
+     *
+     * @return int Number of messages, False on error
+     * @access public
+     */
+    function countUnseen($mailbox)
+    {
+        // Try STATUS, should be faster
+        $counts = $this->status($mailbox, array('UNSEEN'));
+        if (is_array($counts)) {
+            return (int) $counts['UNSEEN'];
+        }
+
+        // Invoke SEARCH as a fallback
+        // @TODO: ESEARCH support
+        $index = $this->search($mailbox, 'ALL UNSEEN');
+        if (is_array($index)) {
+            return count($index);
+        }
 
         return false;
@@ -1414 +1475 @@
     }
 
-    function countUnseen($folder)
-    {
-        $index = $this->search($folder, 'ALL UNSEEN');
-        if (is_array($index))
-            return count($index);
-        return false;
-    }
-
     // Don't be tempted to change $str to pass by reference to speed this up - it will slow it down by about
     // 7 times instead :-) See comments on http://uk2.php.net/references and this article: