[mms] Don't store preference UI elements in the session cache storage.

horde · Mar 25, 2014 · 2edbd14 · 2edbd14
1 parent 2fbcce2
commit 2edbd14
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/framework/Core/lib/Horde/Core/Prefs/Storage/Configuration.php b/framework/Core/lib/Horde/Core/Prefs/Storage/Configuration.php
@@ -36,9 +36,12 @@ public function get($scope_ob)
             return $scope_ob;
         }
 
+        /* List of UI-only config options. */
+        $ui = array('link', 'prefslink', 'rawhtml', 'container', 'special');
+
         if (isset($pconf->config['_prefs'])) {
             foreach ($pconf->config['_prefs'] as $name => $pref) {
-                if (!isset($pref['value'])) {
+                if (!isset($pref['value']) || in_array($pref['type'], $ui)) {
                     continue;
                 }
 

diff --git a/framework/Core/package.xml b/framework/Core/package.xml
@@ -39,6 +39,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
  <notes>
+* [mms] Don&apos;t store preference UI elements in the session cache storage.
 * [mjr] Send the PRIMARYSMTPADDRESS to EAS clients, if appropriate (Request #13062).
 * [jan] Fix loading of configuration screen with basedir restrictions or libxml entity loading disabled (Bug #13025).
 * [mms] The location of dynamically served static files is now configurable (Request #13070).
@@ -3359,6 +3360,7 @@
    <date>2014-02-13</date>
    <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
    <notes>
+* [mms] Don&apos;t store preference UI elements in the session cache storage.
 * [mjr] Send the PRIMARYSMTPADDRESS to EAS clients, if appropriate (Request #13062).
 * [jan] Fix loading of configuration screen with basedir restrictions or libxml entity loading disabled (Bug #13025).
 * [mms] The location of dynamically served static files is now configurable (Request #13070).