[wp-trac] [WordPress Trac] #14184: HTTP API is not allowing '0' for headers and cookies.
WordPress Trac
wp-trac at lists.automattic.com
Wed Jul 7 12:09:19 UTC 2010
#14184: HTTP API is not allowing '0' for headers and cookies.
--------------------------+-------------------------------------------------
Reporter: mailnew2ster | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: HTTP | Version:
Severity: normal | Keywords: has-patch needs-testing
--------------------------+-------------------------------------------------
Comment(by dd32):
thanks for the extra details mailnew2ster I'll go through both the
original and updated patches and test them all out, Thanks for this and
the other patches on the other tickets.
jacob: While empty() is the general coding standard used by WordPress,
{{{isset() / '' ===}}} is also used where appropriate, Doing a string
comparison for every case is going to just end up as ugly code, but there
are times that its appropriate, There are also times where empty() is
appropriate due to the expected contents, and expected input (For example,
having a scheme of 0 is unlikely, and completely unexpected in terms of
HTTP access, therefor, empty() would be appropriate)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/14184#comment:11>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list