From: martinlanghoff <martinlanghoff>
Date: Sun, 6 Jan 2008 22:56:07 +0000 (+0000)
Subject: lib/setup: restore sanity to rcache handling
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=b1df0eb2cec4425c8ea8c4b6ca3c9a657cdcee63;p=moodle.git

lib/setup: restore sanity to rcache handling

$CFG->rcache is safe to keep, even as we are reading the config table.

This is because whenever we update the config table, all the rcache
plugins either mark the cache as dirty or they update the cache.

The reason we want to be able to preserve this is that rcache settings
in CFG should usually be set in config.php rather than in the config
table.

With this fix, we support both control points.
---

diff --git a/lib/setup.php b/lib/setup.php
index 65f20e36d0..98c5909b0c 100644
--- a/lib/setup.php
+++ b/lib/setup.php
@@ -211,7 +211,6 @@ global $HTTPSPAGEREQUIRED;
     configure_dbconnection();
 
 /// Load up any configuration from the config table
-    unset($CFG->rcache);
     $CFG = get_config();
 
 /// Turn on SQL logging if required
@@ -286,9 +285,7 @@ global $HTTPSPAGEREQUIRED;
 /// Ensure we define rcache - so we can later check for it
 /// with a really fast and unambiguous $CFG->rcache === false
     if (!empty($CFG->cachetype)) {
-        if (array_key_exists('rcache', $CFG->config_php_settings)) {
-            $CFG->rcache = (bool)$CFG->config_php_settings['rcache']; // always use config.php setting if present
-        } else if (empty($CFG->rcache)) {
+        if (empty($CFG->rcache)) {
             $CFG->rcache = false;
         } else {
             $CFG->rcache = true;