MDL-20695 can not use trigger_error() for informing users about probl…

…ems in config.php, no need to log these
nilo72 · Oct 31, 2009 · ccda6d6 · ccda6d6
1 parent 2e9b772
commit ccda6d6
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/setup.php b/lib/setup.php
@@ -168,11 +168,11 @@
     define('NO_DEBUG_DISPLAY', false);
 }
 
-/// wwwroot is mandatory
-    if (!isset($CFG->wwwroot)) {
-        trigger_error('Fatal: $CFG->wwwroot is not configured! Exiting.');
-        die;
-    }
+// wwwroot is mandatory
+if (!isset($CFG->wwwroot)) {
+    // trigger_error() is not correct here, no need to log this
+    die('Fatal: $CFG->wwwroot is not configured! Exiting.');
+}
 
 /// Detect CLI scripts - CLI scripts are executed from command line, do not have session and we do not want HTML in output
     if (!defined('CLI_SCRIPT')) { // CLI_SCRIPT might be defined in 'fake' CLI scripts like admin/cron.php