[wp-trac] [WordPress Trac] #11987: Incorrect handling of WP_DEBUG_DISPLAY (was: Use ini_set('display_errors', 0) instead of ini_set('display_errors', 1); in wp-settings.php)

WordPress Trac wp-trac at lists.automattic.com
Mon Jan 25 22:06:25 UTC 2010


#11987: Incorrect handling of WP_DEBUG_DISPLAY
--------------------------+-------------------------------------------------
 Reporter:  ocean90       |        Owner:          
     Type:  defect (bug)  |       Status:  reopened
 Priority:  normal        |    Milestone:          
Component:  General       |      Version:  3.0     
 Severity:  normal        |   Resolution:          
 Keywords:  WP_DEBUG      |  
--------------------------+-------------------------------------------------
Changes (by ocean90):

  * status:  closed => reopened
  * version:  2.9.1 => 3.0
  * resolution:  invalid =>


Comment:

 Sry, but please check it again.

 in wp-load.php
 {{{
 if ( WP_DEBUG_DISPLAY )
    ini_set( 'display_errors', 0 );
 }}}
 And now I should use ''define('WP_DEBUG_DISPLAY', false);'' to hide the
 errors. But it didn't make sense, because in this context
 if(WP_DEBUG_DISPLAY) must be true to set display_errors 0.

 In my opinion their is an ! missing:

 {{{
 if ( !WP_DEBUG_DISPLAY )
    ini_set( 'display_errors', 0 );
 }}}

 PS: in wp-load.php line 221 is an e too much (e'''e'''rrors to debug.log)

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11987#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list