Enforce E_ALL? [#963182]

Drupal 7.0 beta2:
includes/bootstrap.inc file
drupal_environment_initialize() function
Line 509-510:

  // Enforce E_ALL, but allow users to set levels not part of E_ALL.
  error_reporting(E_ALL | error_reporting());

It doesn't seems logical for me. The E_ALL bits have "1" already. A bit OR doesn't matter. Then the error_reporting is always E_ALL.

Comments

Comment #1

foripepe commented 15 November 2010 at 10:14

Status:

Active

» Needs review

The latest Drupal 7.0 beta3 has this bug too.
I suggest a patch, like this:

--- a/includes/bootstrap.inc	Mon Nov 08 12:34:41 2010 +0100
+++ b/includes/bootstrap.inc	Mon Nov 15 11:09:25 2010 +0100
@@ -496,9 +496,6 @@ function drupal_environment_initialize()
   // e.g. "&" or "%" that have special meanings in URLs and must be encoded.
   $_GET['q'] = request_path();
 
-  // Enforce E_ALL, but allow users to set levels not part of E_ALL.
-  error_reporting(E_ALL | error_reporting());
-
   // Override PHP settings required for Drupal to work properly.
   // sites/default/default.settings.php contains more runtime settings.
   // The .htaccess file contains settings that cannot be changed at runtime.