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 (at) horde (dot) org
Requester	adrieder (at) sbox (dot) tugraz (dot) at
Created	06/21/2007 (6566 days ago)
Due
Updated	06/21/2007 (6566 days ago)
Assigned
Resolved	06/21/2007 (6566 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

06/21/2007 02:02:24 AM	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/21/2007 12:25:20 AM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #1 State ⇒ Unconfirmed Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ wrong parameter count for number_format Queue ⇒ IMP	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?