MDL-39474 fix incorrect bitwise ops when setting $CFG->debugdeveloper
authorPetr Škoda <commits@skodak.org>
Wed, 21 Aug 2013 11:38:33 +0000 (13:38 +0200)
committerPetr Škoda <commits@skodak.org>
Wed, 21 Aug 2013 11:38:33 +0000 (13:38 +0200)
lib/setup.php
lib/weblib.php

index f0ba4a5..ecf4917 100644 (file)
@@ -339,7 +339,7 @@ if (isset($CFG->debug)) {
 } else {
     $CFG->debug = 0;
 }
-$CFG->debugdeveloper = ($CFG->debug & E_ALL and $CFG->debug & E_STRICT); // DEBUG_DEVELOPER is not available yet.
+$CFG->debugdeveloper = (($CFG->debug & (E_ALL | E_STRICT)) === (E_ALL | E_STRICT)); // DEBUG_DEVELOPER is not available yet.
 
 if (!defined('MOODLE_INTERNAL')) { // Necessary because cli installer has to define it earlier.
     /** Used by library scripts to check they are being called by Moodle. */
@@ -599,7 +599,7 @@ if (isset($CFG->debug)) {
 }  else {
     $CFG->debug = 0;
 }
-$CFG->debugdeveloper = ($CFG->debug & DEBUG_DEVELOPER);
+$CFG->debugdeveloper = (($CFG->debug & DEBUG_DEVELOPER) === DEBUG_DEVELOPER);
 
 // Find out if PHP configured to display warnings,
 // this is a security problem because some moodle scripts may
index 2b6c7d7..fdde141 100644 (file)
@@ -2802,7 +2802,7 @@ function set_debugging($level, $debugdisplay = null) {
     global $CFG;
 
     $CFG->debug = (int)$level;
-    $CFG->debugdeveloper = ($CFG->debug & DEBUG_DEVELOPER);
+    $CFG->debugdeveloper = (($CFG->debug & DEBUG_DEVELOPER) === DEBUG_DEVELOPER);
 
     if ($debugdisplay !== null) {
         $CFG->debugdisplay = (bool)$debugdisplay;