From: skodak <skodak>
Date: Thu, 1 Feb 2007 21:11:50 +0000 (+0000)
Subject: MDL-8384 debugdisplay does not respect original value of display_errors
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=6349a3ba48046abb089df2a66462ed242aca6a54;p=moodle.git

MDL-8384 debugdisplay does not respect original value of display_errors
---

diff --git a/lib/adminlib.php b/lib/adminlib.php
index cc62831788..4f94c005a3 100644
--- a/lib/adminlib.php
+++ b/lib/adminlib.php
@@ -2579,7 +2579,8 @@ class admin_setting_special_debugdisplay extends admin_setting_configcheckbox {
         $name = 'debugdisplay';
         $visiblename = get_string('debugdisplay', 'admin');
         $description = get_string('configdebugdisplay', 'admin');
-        parent::admin_setting_configcheckbox($name, $visiblename, $description, '');
+        $default = ini_get('display_errors');
+        parent::admin_setting_configcheckbox($name, $visiblename, $description, $default);
     }
 
     function write_setting($data) {
@@ -2594,7 +2595,7 @@ class admin_setting_special_debugdisplay extends admin_setting_configcheckbox {
     function output_html() {
 
         if ($this->get_setting() === NULL) {
-            $currentsetting = ini_get('display_error');
+            $currentsetting = $this->defaultsetting;
         } else {
             $currentsetting = $this->get_setting();
         }
diff --git a/lib/setup.php b/lib/setup.php
index 28b0020ad5..e4e36dcd47 100644
--- a/lib/setup.php
+++ b/lib/setup.php
@@ -258,7 +258,9 @@ global $HTTPSPAGEREQUIRED;
 
 
 /// If we want to display Moodle errors, then try and set PHP errors to match
-    if (empty($CFG->debugdisplay)) {
+    if (!isset($CFG->debugdisplay)) {
+        //keep it as is during installation
+    } else if (empty($CFG->debugdisplay)) {
         @ini_set('display_errors', '0');
         @ini_set('log_errors', '1');
     } else {