Changeset 5898 in subversion

Timestamp:

Feb 21, 2012 4:43:39 PM (16 months ago)

Author:

thomasb

Message:

Refactored IMAP cache expunge: delegate to storage object; don't rely on deprecated 'enable_caching' config option

Location:

trunk/roundcubemail/program/include

Files:

• main.inc (modified) (1 diff)
• rcmail.php (modified) (4 diffs)
• rcube_imap.php (modified) (2 diffs)
• rcube_imap_cache.php (modified) (1 diff)
• rcube_session.php (modified) (1 diff)
• rcube_storage.php (modified) (1 diff)

trunk/roundcubemail/program/include/main.inc

@@ -156 +156 @@
     closedir($dir);
   }
-}
-
-
-/**
- * Garbage collector for cache entries.
- * Remove all expired message cache records
- * @return void
- */
-function rcmail_cache_gc()
-{
-  $rcmail = rcmail::get_instance();
-  $db = $rcmail->get_dbh();
-
-  // get target timestamp
-  $ts = get_offset_time($rcmail->config->get('message_cache_lifetime', '30d'), -1);
-
-  $db->query("DELETE FROM ".get_table_name('cache_messages')
-        ." WHERE changed < " . $db->fromunixtime($ts));
-
-  $db->query("DELETE FROM ".get_table_name('cache_index')
-        ." WHERE changed < " . $db->fromunixtime($ts));
-
-  $db->query("DELETE FROM ".get_table_name('cache_thread')
-        ." WHERE changed < " . $db->fromunixtime($ts));
-
-  $db->query("DELETE FROM ".get_table_name('cache')
-        ." WHERE created < " . $db->fromunixtime($ts));
 }
 

trunk/roundcubemail/program/include/rcmail.php

@@ -130 +130 @@
   private $action_map = array();
   private $shutdown_functions = array();
+  private $expunge_cache = false;
 
 
@@ -768 +769 @@
 
     $this->session->register_gc_handler('rcmail_temp_gc');
-    if ($this->config->get('enable_caching'))
-      $this->session->register_gc_handler('rcmail_cache_gc');
+    $this->session->register_gc_handler(array($this, 'cache_gc'));
 
     // start PHP session (if not in CLI mode)
@@ -1279 +1279 @@
     }
 
-    if (is_object($this->storage))
+    if (is_object($this->storage)) {
+        if ($this->expunge_cache)
+            $this->storage->expunge_cache();
       $this->storage->close();
+  }
 
     // before closing the database connection, write session data
@@ -1313 +1316 @@
   {
     $this->shutdown_functions[] = $function;
+  }
+
+
+  /**
+   * Garbage collector for cache entries.
+   * Set flag to expunge caches on shutdown
+   */
+  function cache_gc()
+  {
+    // because this gc function is called before storage is initialized,
+    // we just set a flag to expunge storage cache on shutdown.
+    $this->expunge_cache = true;
   }
 

trunk/roundcubemail/program/include/rcube_imap.php

@@ -3528 +3528 @@
         if ($this->caching && !$this->cache) {
             $rcmail = rcmail::get_instance();
-            $this->cache = $rcmail->get_cache('IMAP', $this->caching);
+            $ttl = $rcmail->config->get('message_cache_lifetime', '10d') - mktime();
+            $this->cache = $rcmail->get_cache('IMAP', $this->caching, $ttl);
         }
 
@@ -3573 +3574 @@
             $cache->remove($key, $prefix_mode);
         }
+    }
+
+    /**
+     * Delete outdated cache entries
+     */
+    public function expunge_cache()
+    {
+        if ($this->mcache) {
+            $ttl = rcmail::get_instance()->config->get('message_cache_lifetime', '10d');
+            $this->mcache->expunge($ttl);
+        }
+
+        if ($this->cache)
+            $this->cache->expunge();
     }
 

trunk/roundcubemail/program/include/rcube_imap_cache.php

@@ -600 +600 @@
 
     /**
+     * Delete cache entries older than TTL
+     *
+     * @param string $ttl  Lifetime of message cache entries
+     */
+    function expunge($ttl)
+    {
+        // get expiration timestamp
+        $ts = get_offset_time($ttl, -1);
+
+        $this->db->query("DELETE FROM ".get_table_name('cache_messages')
+              ." WHERE changed < " . $this->db->fromunixtime($ts));
+
+        $this->db->query("DELETE FROM ".get_table_name('cache_index')
+              ." WHERE changed < " . $this->db->fromunixtime($ts));
+
+        $this->db->query("DELETE FROM ".get_table_name('cache_thread')
+              ." WHERE changed < " . $this->db->fromunixtime($ts));
+    }
+
+
+    /**
      * Fetches index data from database
      */

trunk/roundcubemail/program/include/rcube_session.php

@@ -333 +333 @@
    * @param mixed Callback function
    */
-  public function register_gc_handler($func_name)
-  {
-    if ($func_name && !in_array($func_name, $this->gc_handlers))
-      $this->gc_handlers[] = $func_name;
+  public function register_gc_handler($func)
+  {
+    foreach ($this->gc_handlers as $handler) {
+      if ($handler == $func) {
+        return;
+      }
+    }
+
+    $this->gc_handlers[] = $func;
   }
 

trunk/roundcubemail/program/include/rcube_storage.php

@@ -972 +972 @@
     abstract function get_cache($key);
 
+    /**
+     * Delete outdated cache entries
+     */
+    abstract function expunge_cache();
+
 }  // end class rcube_storage