ID: 28228
Updated by: [EMAIL PROTECTED]
Reported By: bichinhoverde at spwinternet dot com dot br
Status: Assigned
Bug Type: Math related
Operating System: *
PHP Version: 4CVS, 5CVS (2004-12-12)
Assigned To: iliaa
New Comment:
This is not fixed correctly. See also bug #29538
Previous Comments:
------------------------------------------------------------------------
[2004-12-12 23:46:00] [EMAIL PROTECTED]
This is not fixed correctly. See also bug 29538
------------------------------------------------------------------------
[2004-09-23 19:43:49] bichinhoverde at spwinternet dot com dot br
Well, according to the documentation, dec_point and thousands_sep
should be both string. I wouldn't know what to expect when I put a
boolean value in ther.
But, regarding:
print number_format(1234.56, 2, '', ',');
it really outputs 123,456 when it should output 1,23456. I tested it on
4.3.8 on windows. I thinks it is another "feature" ;)
------------------------------------------------------------------------
[2004-09-23 17:08:45] mark at quarella dot co dot uk
To be a bit clearer than my previous comment:
number_format(1234.56, 2, '', ',')
should result in
1,23456
not
123,456
as it does now.
Ideally
number_format(1234.56, 2, FALSE, ',')
would give
1,234.56
but
1,23456
would be better than the current behaviour.
------------------------------------------------------------------------
[2004-09-23 17:04:49] mark at quarella dot co dot uk
I have just discovered that:
number_format(11.99, 2, FALSE, ',')
which used to display
11.99
now displays
1,199
I'm not sure whether FALSE ought to be supported as a "use the default"
rather than equivalent to the empty string, but surely at worst this
should give:
1199
My preference, for what it's worth, is that FALSE should take the
default ('.'), but '' should work better than it does?
[I only have 4.3.8 to test against but no mention of this is made in
the 4.3.9 release notes]
------------------------------------------------------------------------
[2004-04-30 15:26:29] [EMAIL PROTECTED]
This bug has been fixed in CVS.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
Thank you for the report, and for helping us make PHP better.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/28228
--
Edit this bug report at http://bugs.php.net/?id=28228&edit=1
