[wp-trac] [WordPress Trac] #11090: Standardize checking for WP_DEBUG

WordPress Trac wp-trac at lists.automattic.com
Fri Nov 6 22:28:41 UTC 2009


#11090: Standardize checking for WP_DEBUG
------------------------------+---------------------------------------------
 Reporter:  nacin             |       Owner:  westi
     Type:  defect (bug)      |      Status:  new  
 Priority:  normal            |   Milestone:  2.9  
Component:  Warnings/Notices  |     Version:  2.9  
 Severity:  normal            |    Keywords:       
------------------------------+---------------------------------------------

Comment(by dd32):

 Is there any good reason for code like this:
 {{{
  if ( ! defined('WP_DEBUG') || ( defined('WP_DEBUG') && !WP_DEBUG ) ) {
 }}}

 Which is exactly the same as:
 {{{
  if ( ! defined('WP_DEBUG') || !WP_DEBUG ) {
 }}}

 I agree that adding == and === into it complicates things.

 We could even add to wp-settings:
 {{{
 if ( ! defined('WP_DEBUG') ) define('WP_DEBUG', false);
 }}}

 if that makes it easier?

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


More information about the wp-trac mailing list