MDL-23948 we should make CFG global before assigning value to it, you could not inclu...
authorPetr Skoda <skodak@moodle.org>
Thu, 26 Aug 2010 15:31:18 +0000 (15:31 +0000)
committerPetr Skoda <skodak@moodle.org>
Thu, 26 Aug 2010 15:31:18 +0000 (15:31 +0000)
config-dist.php
lib/installlib.php
lib/setup.php

index 79b96f2..8a6e52e 100644 (file)
@@ -28,6 +28,7 @@
 //                                                                       //
 ///////////////////////////////////////////////////////////////////////////
 unset($CFG);  // Ignore this line
+global $CFG;  // This is necessary here for PHPUnit execution
 $CFG = new stdClass();
 
 //=========================================================================
index fb12c18..e7cacf9 100644 (file)
@@ -138,6 +138,7 @@ function install_generate_configphp($database, $cfg) {
     $configphp = '<?php  // Moodle configuration file' . PHP_EOL . PHP_EOL;
 
     $configphp .= 'unset($CFG);' . PHP_EOL;
+    $configphp .= 'global $CFG;' . PHP_EOL;
     $configphp .= '$CFG = new stdClass();' . PHP_EOL . PHP_EOL; // prevent PHP5 strict warnings
 
     $dbconfig = $database->export_dbconfig();
index 55e27c1..cc3f99c 100644 (file)
@@ -40,7 +40,7 @@
  * @global object $CFG
  * @name $CFG
  */
-global $CFG;
+global $CFG; // this should be done much earlier in config.php before creating new $CFG instance
 
 // We can detect real dirroot path reliably since PHP 4.0.2,
 // it can not be anything else, there is no point in having this in config.php