[wp-trac] [WordPress Trac] #19914: wp-app.php assumes elements exist

WordPress Trac wp-trac at lists.automattic.com
Sat Feb 4 20:30:43 UTC 2012


#19914: wp-app.php assumes elements exist
--------------------------+------------------------------
 Reporter:  rmccue        |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  AtomPub       |     Version:  2.3
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:8 kurtpayne]:
 > Switched these to `empty()` calls and reversed the order of the ternary
 to be in-line with the coding standards.

 BTW, there's an exception for `!empty()` in the coding standards (testing
 for false seems more readable in this case).

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


More information about the wp-trac mailing list