[wp-trac] [WordPress Trac] #7653: Count in wp_sprintf_l() is used, but not defined.

WordPress Trac wp-trac at lists.automattic.com
Sat Aug 30 21:07:41 GMT 2008


#7653: Count in wp_sprintf_l() is used, but not defined.
-------------------------+--------------------------------------------------
 Reporter:  jacobsantos  |       Owner:  anonymous              
     Type:  defect       |      Status:  new                    
 Priority:  low          |   Milestone:  2.7                    
Component:  General      |     Version:  2.7                    
 Severity:  minor        |    Keywords:  has-patch needs-testing
-------------------------+--------------------------------------------------
 The function checks for the {{{$i}}} variable, which I assume is supposed
 to be from the count, but is never defined. The condition that uses that
 variable will always be false all cases. Patch defines the variable, which
 is hopefully what the developer had in mind.

 The note about being greater than 2 items is not checked for and is
 another bug, I suspect. I suppose the intention is to have the remaining
 item appended after and break out of the loop, however the only check is a
 boolean, which is not suffice of a check to ensure this.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/7653>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list