ID:               28228
 Updated by:       [EMAIL PROTECTED]
 Reported By:      bichinhoverde at spwinternet dot com dot br
-Status:           Assigned
+Status:           Closed
 Bug Type:         Math related
 Operating System: *
 PHP Version:      4CVS, 5CVS (2004-12-12)
 Assigned To:      iliaa
 New Comment:

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.




Previous Comments:
------------------------------------------------------------------------

[2004-12-12 23:46:14] [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

Reply via email to