[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