Fix bootstrap chicken and egg condition #50

catalyst · Jun 12, 2024 · 7a857e4 · 7a857e4
1 parent 7f7e90b
commit 7a857e4
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 12 deletions.
diff --git a/classes/cache_config.php b/classes/cache_config.php
@@ -112,23 +112,22 @@ private function generate_config_array(): array {
         // Generate locks.
         $locks = $this->generate_locks();
 
-        // Get the siteidentifier. Copies pattern from cache_config.
-        // Uses 'forcedcache' if not known.
-        if (!empty($CFG->siteidentifier)) {
-            $siteidentifier = md5((string) $CFG->siteidentifier);
-        } else {
-            $siteidentifier = 'forcedcache';
-        }
-
         // Throw it all into an array and return.
-        return array(
-            'siteidentifier' => $siteidentifier,
+        $config = [
             'stores' => $stores,
             'modemappings' => $modemappings,
             'definitions' => $definitions,
             'definitionmappings' => $definitionmappings,
             'locks' => $locks
-        );
+        ];
+
+        // Get the siteidentifier. Copies pattern from cache_config.
+        if (!empty($CFG->siteidentifier)) {
+            $config['siteidentifier'] = md5((string) $CFG->siteidentifier);
+        }
+
+        return $config;
+
     }
 
     /**

diff --git a/classes/cache_factory.php b/classes/cache_factory.php
@@ -61,7 +61,11 @@ public function create_config_instance($writer = false) {
             $this->configs[$class]->load();
         }
 
-        $this->set_state(self::STATE_READY);
+        if (empty($CFG->siteidentifier)) {
+            $this->set_state(self::STATE_STORES_DISABLED);
+        } else {
+            $this->set_state(self::STATE_READY);
+        }
 
         // Return the instance.
         return $this->configs[$class];