[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