[wp-trac] [WordPress Trac] #23089: Rewrite a confusing condition in default-widgets.php

WordPress Trac noreply at wordpress.org
Tue Jan 22 19:35:50 UTC 2013


#23089: Rewrite a confusing condition in default-widgets.php
-------------------------+-----------------------
 Reporter:  alexvorn2    |       Owner:
     Type:  enhancement  |      Status:  reopened
 Priority:  lowest       |   Milestone:  3.6
Component:  Widgets      |     Version:
 Severity:  minor        |  Resolution:
 Keywords:  has-patch    |
-------------------------+-----------------------

Comment (by SergeyBiryukov):

 Replying to [comment:8 azaozz]:
 > Perhaps the `isset()` has to be `!empty()` so it doesn't allow `$number`
 to be 0 (zero).

 Good point, [attachment:23089.2.patch] uses `!empty()`.

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


More information about the wp-trac mailing list