Description: ------------ When the decimal separator is set to comma (,), what occurrs when locale is set to pt_BR (I didn't test with other locales), the parser outputs a broken JSON string. Test script: --------------- <?php setlocale(LC_ALL, 'pt_BR.UTF8'); // must have this locale installed to work $a = array( 'percent': 0.18975332068311 ); $s = new Services_JSON(); echo $s->encodeUnsafe(), "\n"; ?> Expected result: ---------------- {"percent":0.18975332068311} Actual result: -------------- {"percent":0,18975332068311} // note the "," instead of "."

Added patch fix-float-behavior.

Thank you for taking the time to report a problem with the package. This problem may have been already fixed by a previous change that is in the CVS of the package. Please log into CVS with: cvs -d :pserver:cvsread@cvs.php.net:/repository login and check out the CVS repository of this package and upgrade cvs -d :pserver:cvsread@cvs.php.net:/repository co pear/Services_JSON pear upgrade pear/Services_JSON/package2.xml or pear upgrade pear/Services_JSON/package.xml If you are able to reproduce the bug with the latest CVS, please change the status back to "Open". Again, thank you for your continued support of PEAR. Can you try SVN - I've used number format there. http://svn.php.net/repository/pear/packages/Services_JSON/trunk/JSON.php

Well those instructions are pretty useless ;) - try here http://svn.php.net/repository/pear/packages/Services_JSON/trunk/JSON.php

The number_format function doesn't resolve the problem. 1. Cast to float is the problem here. When you do this, the locale rules apply and the number stays with the comma instead of dot. 2. When you use number_format you have a (possible) loss of precision because of decimals parameter.

The previous submmited patch works on trunk.

try now, I think this is the correct solution printf("%F", $value) - non-locale aware printing of floats.. Regards Alan

It works, but you have a loss of precision, even if you don't specify the precision argument. I used the following script to test: <?php $s = new Services_JSON(); $list = array( 'percent' => 0.166554584 ); echo $s->encodeUnsafe($vetor), "\n"; ?> The result was: {"percent":0.166555} Off-topic: this is weird: if I change the sprintf statement to: sprintf("%.50F", $var); The result was: {"percent":0.1665545840000000055169948609545826911926} Regards, Rodrigo Machado

On previous script the following line was missing (after <?php): setlocale(LC_ALL, 'pt_BR.UTF8');

See also: Bug #16931

Having looked through the PHP source I don't think there is any clean way to output the float without locale formating or loosing precision. This leaves a few options: a) use localeconv() and issue a warning or error if the decimal_seperator is not a '.' b) use setlocale() and force it to a certain local before output, and restore after. I'm tempted to do b).. Any thoughts. Regards Alan

Try this and see how it works. http://svn.php.net/repository/pear/packages/Services_JSON/trunk/JSON.php Regards Alan

It works! Thanks! One more suggestion: instead of LC_ALL use LC_NUMERIC, just in case if the user has modified only the LC_NUMERIC part of locale. Thanks again! Regards, Rodrigo Machado

Thank you for your bug report. This issue has been fixed in the latest released version of the package, which you can download at http://pear.php.net/get/ Thanks - changed it to LC_NUMERIC - hopefully that did not break anything ;)