Changeset 5316 in subversion

Timestamp:

Oct 6, 2011 11:49:33 AM (20 months ago)

Author:

alec

Message:

• Improve performance by storing sorted mailbox list in the cache

File:

• trunk/roundcubemail/program/include/rcube_imap.php (modified) (5 diffs)

trunk/roundcubemail/program/include/rcube_imap.php

@@ -2921 +2921 @@
      * @param   bool    $skip_sort Enable to return unsorted list (for better performance)
      *
-     * @return  array   List of mailboxes/folders
+     * @return  array   List of folders
      * @access  public
      */
     function list_mailboxes($root='', $name='*', $filter=null, $rights=null, $skip_sort=false)
     {
+        $cache_key = $root.':'.$name;
+        if (!empty($filter)) {
+            $cache_key .= ':'.(is_string($filter) ? $filter : serialize($filter));
+        }
+        $cache_key .= ':'.$rights;
+        $cache_key = 'mailboxes.'.md5($cache_key);
+
+        // get cached folder list
+        $a_mboxes = $this->get_cache($cache_key);
+        if (is_array($a_mboxes)) {
+            return $a_mboxes;
+        }
+
         $a_mboxes = $this->_list_mailboxes($root, $name, $filter, $rights);
+
+        if (!is_array($a_mboxes)) {
+            return array();
+        }
+
+        // filter folders list according to rights requirements
+        if ($rights && $this->get_capability('ACL')) {
+            $a_mboxes = $this->filter_rights($a_mboxes, $rights);
+        }
 
         // INBOX should always be available
@@ -2933 +2955 @@
         }
 
-        // sort mailboxes
-        if (!$skip_sort) {
+        // sort mailboxes (always sort for cache)
+        if (!$skip_sort || $this->cache) {
             $a_mboxes = $this->_sort_mailbox_list($a_mboxes);
         }
+
+        // write mailboxlist to cache
+        $this->update_cache($cache_key, $a_mboxes);
 
         return $a_mboxes;
@@ -2956 +2981 @@
     private function _list_mailboxes($root='', $name='*', $filter=null, $rights=null)
     {
-        $cache_key = $root.':'.$name;
-        if (!empty($filter)) {
-            $cache_key .= ':'.(is_string($filter) ? $filter : serialize($filter));
-        }
-        $cache_key .= ':'.$rights;
-
-        $cache_key = 'mailboxes.'.md5($cache_key);
-
-        // get cached folder list
-        $a_mboxes = $this->get_cache($cache_key);
-        if (is_array($a_mboxes)) {
-            return $a_mboxes;
-        }
-
         $a_defaults = $a_out = array();
 
@@ -2980 +2991 @@
         }
         else if (!$this->conn->connected()) {
-           return array();
+           return null;
         }
         else {
@@ -3027 +3038 @@
             $a_folders = array();
         }
-
-        // filter folders list according to rights requirements
-        if ($rights && $this->get_capability('ACL')) {
-            $a_folders = $this->filter_rights($a_folders, $rights);
-        }
-
-        // write mailboxlist to cache
-        $this->update_cache($cache_key, $a_folders);
 
         return $a_folders;