Changeset 2084 in subversion

Timestamp:

Nov 23, 2008 8:24:49 AM (5 years ago)

Author:

thomasb

Message:

Also clean cache table when removing old messages from cache

Location:

trunk/roundcubemail/program/include

Files:

• main.inc (modified) (1 diff)
• session.inc (modified) (1 diff)

trunk/roundcubemail/program/include/main.inc

@@ -152 +152 @@
  * Remove all expired message cache records
  */
-function rcmail_message_cache_gc()
-  {
-  global $DB, $CONFIG;
-  
-  // no cache lifetime configured
-  if (empty($CONFIG['message_cache_lifetime']))
-    return;
+function rcmail_cache_gc()
+  {
+  $rcmail = rcmail::get_instance();
+  $db = $rcmail->get_dbh();
   
   // get target timestamp
-  $ts = get_offset_time($CONFIG['message_cache_lifetime'], -1);
-  
-  $DB->query("DELETE FROM ".get_table_name('messages')."
-             WHERE  created < ".$DB->fromunixtime($ts));
+  $ts = get_offset_time($rcmail->config->get('message_cache_lifetime', '30d'), -1);
+  
+  $db->query("DELETE FROM ".get_table_name('messages')."
+             WHERE  created < " . $db->fromunixtime($ts));
+
+  $db->query("DELETE FROM ".get_table_name('cache')."
+              WHERE  created < " . $db->fromunixtime($ts));
   }
 

trunk/roundcubemail/program/include/session.inc

@@ -126 +126 @@
   }
 
-  if ($rcmail->config->get('enable_caching'))
-    rcmail_message_cache_gc();
-
   // just delete all expired sessions
   $DB->query("DELETE FROM " . get_table_name('session') . "
-    WHERE changed < " . $now);
+    WHERE changed < " . $DB->fromunixtime(time() - $maxlifetime));
+
+  if ($rcmail->config->get('enable_caching'))
+    rcmail_cache_gc();
 
   rcmail_temp_gc();