Changeset 5d66a4b in github

Timestamp:

Apr 19, 2012 3:42:19 AM (13 months ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo

Children:

6c95b3e4

Parents:

6a8b4c2

Message:

• Improved ttl values handling

Location:

program/include

Files:

• rcube.php (modified) (1 diff)
• rcube_cache.php (modified) (2 diffs)
• rcube_imap.php (modified) (1 diff)
• rcube_shared.inc (modified) (2 diffs)

program/include/rcube.php

@@ -237 +237 @@
    * @param string $name   Cache identifier
    * @param string $type   Cache type ('db', 'apc' or 'memcache')
-   * @param int    $ttl    Expiration time for cache items in seconds
+   * @param string $ttl    Expiration time for cache items
    * @param bool   $packed Enables/disables data serialization
    *

program/include/rcube_cache.php

@@ -60 +60 @@
      * @param int    $userid User identifier
      * @param string $prefix Key name prefix
-     * @param int    $ttl    Expiration time of memcache/apc items in seconds (max.2592000)
+     * @param string $ttl    Expiration time of memcache/apc items
      * @param bool   $packed Enables/disabled data serialization.
      *                       It's possible to disable data serialization if you're sure
@@ -83 +83 @@
         }
 
+        // convert ttl string to seconds
+        $ttl = get_offset_sec($ttl);
+        if ($ttl > 2592000) $ttl = 2592000;
+
         $this->userid    = (int) $userid;
-        $this->ttl       = (int) $ttl;
+        $this->ttl       = $ttl;
         $this->packed    = $packed;
         $this->prefix    = $prefix;

program/include/rcube_imap.php

@@ -3533 +3533 @@
             $rcube = rcube::get_instance();
             $ttl = $rcube->config->get('message_cache_lifetime', '10d');
-            $ttl = get_offset_time($ttl) - time();
-
             $this->cache = $rcube->get_cache('IMAP', $this->caching, $ttl);
         }

program/include/rcube_shared.inc

@@ -147 +147 @@
 
 /**
- * Create a unix timestamp with a specified offset from now.
- *
- * @param string $offset_str  String representation of the offset (e.g. 20min, 5h, 2days)
- * @param int    $factor      Factor to multiply with the offset
- *
- * @return int Unix timestamp
- */
-function get_offset_time($offset_str, $factor=1)
-{
-    if (preg_match('/^([0-9]+)\s*([smhdw])/i', $offset_str, $regs)) {
-        $amount = (int)$regs[1];
+ * Returns number of seconds for a specified offset string.
+ *
+ * @param string $str  String representation of the offset (e.g. 20min, 5h, 2days, 1week)
+ *
+ * @return int Number of seconds
+ */
+function get_offset_sec($str)
+{
+    if (preg_match('/^([0-9]+)\s*([smhdw])/i', $str, $regs)) {
+        $amount = (int) $regs[1];
         $unit   = strtolower($regs[2]);
     }
     else {
-        $amount = (int)$offset_str;
+        $amount = (int) $str;
         $unit   = 's';
     }
 
-    $ts = time();
     switch ($unit) {
     case 'w':
@@ -175 +173 @@
     case 'm':
         $amount *= 60;
-    case 's':
-        $ts += $amount * $factor;
-    }
-
-    return $ts;
+    }
+
+    return $amount;
+}
+
+
+/**
+ * Create a unix timestamp with a specified offset from now.
+ *
+ * @param string $offset_str  String representation of the offset (e.g. 20min, 5h, 2days)
+ * @param int    $factor      Factor to multiply with the offset
+ *
+ * @return int Unix timestamp
+ */
+function get_offset_time($offset_str, $factor=1)
+{
+    return time() + get_offset_sec($offset_str) * $factor;
 }