| Issue 169: | patch to truncate total cost | |
| 1 person starred this issue and may be notified of changes. | Back to list |
What steps will reproduce the problem? 1. set tax to calculate a float, like 6.55 2. get an order that doesn't round evenly 3. check the output of order#total What is the expected output? What do you see instead? the total should be truncated to two decimals, in case [ex:] a promotion gives a percentage off or what not. authorize.net SIM requires 2 decimals, so this should help that to be the case always. What version of the product are you using? On what operating system? trunk.
Feb 21, 2009
#1
rogerpack2005
Feb 21, 2009
Just to note, if you pass a tax or shipping cost to PayPal that has more than 2 decimal places, PayPal will completely ignore it. It is definitely worth truncating, because I had a couple of orders that were paid for but the shipping cost never got transferred.
Feb 21, 2009
authorize.net SIM rejects the transaction if a value has more than two decimals. Perhaps we should add truncation in several locations [?] -=r
May 4, 2010
Roger, any chance of getting you to apply / test this patch on trunk?
Owner:
rogerdpack
Labels: -Priority-Medium Priority-High
May 4, 2010
sure
May 4, 2010
committed r225
May 4, 2010
(No comment was entered for this change.)
Status:
Fixed
|