Tickets :: [#5485] wrong parameter count for number

Summary	wrong parameter count for number_format
Queue	IMP
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	1. Low
Owners	Jan Schneider <jan (at) horde (dot) org>
Requester	adrieder (at) sbox (dot) tugraz (dot) at
Created	06/20/2007 (483 days ago)
Due
Updated	06/20/2007 (483 days ago)
Assigned
Resolved	06/20/2007 (483 days ago)
Attachments
Milestone
Patch	No

06/20/2007	Chuck Hagenbuch	Comment #2 Assigned to Jan Schneider State ⇒ Resolved	Reply to this comment
I believe it should actually be $localeinfo['thousands_sep'] like the other number_format calls there are. Fixed in CVS, thanks!

06/20/2007	adrieder (at) sbox (dot) tugraz (dot) at	Comment #1 State ⇒ Unconfirmed Queue ⇒ IMP Summary ⇒ wrong parameter count for number_format Priority ⇒ 1. Low Type ⇒ Bug	Reply to this comment
wrong parameter count for number_format in imp/compose.php on line 1120 number_format() returns a formatted version of number. This function accepts either one, two or four parameters (not three). this patch removed an empty parameter: http://cvs.horde.org/diff.php/imp/compose.php?r1=2.1047&r2=2.1048&ty=u shouldn't it be re added?