MDL-41997 cache: dummy store now uses static acceleration for session + request
authorSam Hemelryk <sam@moodle.com>
Thu, 26 Sep 2013 04:24:50 +0000 (16:24 +1200)
committerSam Hemelryk <sam@moodle.com>
Sun, 29 Sep 2013 20:41:03 +0000 (09:41 +1300)
cache/classes/definition.php
cache/classes/dummystore.php

index 302bc17..281fa30 100644 (file)
@@ -824,7 +824,7 @@ class cache_definition {
             // Request caches should never use static acceleration - it just doesn't make sense.
             return false;
         }
-        return $this->staticacceleration || $this->mode === cache_store::MODE_SESSION;
+        return $this->staticacceleration;
     }
 
     /**
index 13c3017..55ee3ad 100644 (file)
@@ -113,7 +113,12 @@ class cachestore_dummy extends cache_store {
         //     store things in its static array.
         //   - If the definition is not using static acceleration then the cache loader won't try to store anything
         //     and we will need to store it here in order to make sure it is accessible.
-        $this->persist = !$definition->use_static_acceleration();
+        if ($definition->get_mode() !== self::MODE_APPLICATION) {
+            // Neither the request cache nor the session cache provide static acceleration.
+            $this->persist = true;
+        } else {
+            $this->persist = !$definition->use_static_acceleration();
+        }
     }
 
     /**