Enforce E_ALL?

Posted by foripepe on November 5, 2010 at 11:52am

4 followers

Issue Summary

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.

Posted by foripepe on November 15, 2010 at 10:14am

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.