ID: 13551 Updated by: [EMAIL PROTECTED] Reported By: [EMAIL PROTECTED] -Status: Assigned +Status: Closed Bug Type: BC math related Operating System: Linux 2.2.18 PHP Version: 4.0.6 -Assigned To: andi +Assigned To: pollita New Comment:
This bug has been fixed in CVS. In case this was a PHP problem, 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/. In case this was a documentation problem, the fix will show up soon at http://www.php.net/manual/. In case this was a PHP.net website problem, the change will show up on the PHP.net site and on the mirror sites in short time. Thank you for the report, and for helping us make PHP better. bcmath functions now apply scale only to result. Previous Comments: ------------------------------------------------------------------------ [2003-01-02 18:28:39] [EMAIL PROTECTED] Andi can you take a look? Or just close it if it's bogus. ------------------------------------------------------------------------ [2001-10-05 09:02:55] [EMAIL PROTECTED] That is incorrect. BC is for arbitrary percision arithmetic. Arguments should be strings not integers. Just change the precision argument in the example from 0 to 2 and you will see that it will consider the decimal places of the multiplication arguments. The problem is that the way PHP BC module is working is that it applies the decimal places to arguments when it should only apply to the result. bcmul('8.78','100',2) should be '878.00' and bcmul('8.78','100',0) should be '878' and not '800', otherwise BC PHP module is useless because you can't perform a calculation and control the decimal places of the result without affecting the operands. It should work like sprintf("%.2f",$operand_1*$operand_2) where 2 is the number of decimal places of the output and and not of the operand values because these may have much more decimal places than the result. ------------------------------------------------------------------------ [2001-10-05 02:21:10] [EMAIL PROTECTED] It's not a bug, but a feature (tm). BC only works with integer values, and all arguments must be integer values. However, the documentation is not quite clear here. Derick ------------------------------------------------------------------------ [2001-10-04 19:49:04] [EMAIL PROTECTED] Confirmed with latest CVS. Either the documentation is wrong or the implementation. Both won't need rocket sience to fix ... Btw, this seems to be true for all bc*() functions. Probably the documentation isn't clear enough about the current behaviour. Andi? - Markus ------------------------------------------------------------------------ [2001-10-04 19:18:17] [EMAIL PROTECTED] BC functions apply decimal places also on arguments but it should only apply them on the result. For example bcmul('8.78','100',0) should return 878 and not 800 as it does now. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=13551&edit=1
