From 7f10bb0ffb5d1691f44826a5084dc8369ae2c471 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Tue, 23 Nov 2010 13:37:34 -0700
Subject: [PATCH] Allow theme cache lifetime to be configured

---
 framework/Core/lib/Horde/Core/Factory/ThemesCache.php |  4 ++--
 horde/config/conf.xml                                 | 19 +++++++++++--------
 2 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/framework/Core/lib/Horde/Core/Factory/ThemesCache.php b/framework/Core/lib/Horde/Core/Factory/ThemesCache.php
index e90f86e51..d6bbab465 100644
--- a/framework/Core/lib/Horde/Core/Factory/ThemesCache.php
+++ b/framework/Core/lib/Horde/Core/Factory/ThemesCache.php
@@ -72,7 +72,7 @@ class Horde_Core_Factory_ThemesCache
                 $instance = new Horde_Themes_Cache($app, $theme);
             } else {
                 try {
-                    $instance = @unserialize($cache->get($sig, 86400));
+                    $instance = @unserialize($cache->get($sig, $GLOBALS['conf']['cachethemesparams']['lifetime']));
                 } catch (Exception $e) {
                     $instance = null;
                 }
@@ -102,7 +102,7 @@ class Horde_Core_Factory_ThemesCache
 
         foreach ($this->_instances as $key => $val) {
             if ($val->changed) {
-                $cache->set($key, serialize($val), 86400);
+                $cache->set($key, serialize($val), $GLOBALS['conf']['cachethemesparams']['lifetime']);
             }
         }
     }
diff --git a/horde/config/conf.xml b/horde/config/conf.xml
index ade0b177f..e2de2fdff 100644
--- a/horde/config/conf.xml
+++ b/horde/config/conf.xml
@@ -1084,7 +1084,7 @@
        directory on the web frontend.
        </configdescription>
       </case>
-      <case name="horde_cache" desc="Horde_Cache">
+      <case name="horde_cache" desc="Horde Cache">
        <configdescription>
         &lt;strong&gt;Requires a configured Horde caching
         backend.&lt;/strong&gt;
@@ -1126,7 +1126,7 @@
        directory on the web frontend.
        </configdescription>
       </case>
-      <case name="horde_cache" desc="Horde_Cache">
+      <case name="horde_cache" desc="Horde Cache">
        <configdescription>
         &lt;strong&gt;Requires a configured Horde caching
         backend.&lt;/strong&gt;
@@ -1166,12 +1166,15 @@
    is HIGHLY RECOMMENDED. &lt;strong&gt;Requires a configured Horde caching
    backend.&lt;/strong&gt;
   </configdescription>
-  <configenum name="cachethemes" desc="Cache themes?">false
-   <values>
-    <value desc="No">false</value>
-    <value desc="Yes">true</value>
-   </values>
-  </configenum>
+  <configswitch name="cachethemes" desc="Cache themes?">false
+   <case name="false" desc="No" />
+   <case name="true" desc="Yes">
+    <configsection name="cachethemesparams">
+     <configinteger name="lifetime" desc="Lifetime of the cached files, in
+     seconds.">604800</configinteger>
+    </configsection>
+   </case>
+  </configswitch>
  </configtab>
 
  <configtab name="lock" desc="Lock System">
-- 
2.11.0