[wp-trac] Re: [WordPress Trac] #8481: Improved version of checked() in template.php

WordPress Trac wp-trac at lists.automattic.com
Fri May 1 22:37:33 GMT 2009


#8481: Improved version of checked() in template.php
-------------------------+--------------------------------------------------
 Reporter:  chmac        |       Owner:  anonymous
     Type:  enhancement  |      Status:  new      
 Priority:  normal       |   Milestone:  2.8      
Component:  General      |     Version:           
 Severity:  normal       |    Keywords:  has-patch
-------------------------+--------------------------------------------------

Comment(by Denis-de-Bernardy):

 I considered submitting an updated patch along those lines before bumping
 to Future.

 Possible issues I foresaw:

  - Should $current and $test be documented as accepting anything?
  - If so, and if the is_array() check is not first, the $current == $test
 may end up being erroneously true with the new usage. Probably irrelevant,
 but it did have me thinking twice.

 Just FYI...

 +1 to the patch with updated phpdoc (i.e. accepts numeric/bool/string,
 different behavior with array).

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


More information about the wp-trac mailing list