From b1df0eb2cec4425c8ea8c4b6ca3c9a657cdcee63 Mon Sep 17 00:00:00 2001 From: martinlanghoff Date: Sun, 6 Jan 2008 22:56:07 +0000 Subject: [PATCH] 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. --- lib/setup.php | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) 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; -- 2.39.5