Rounding issue with store credit
-
I’m having a small issue using the store credit feature. Clients are able to check out normally if they have enough credit to cover the entire order, but if they have less store credit than the total at checkout and attempt to enter the remaining amount of credit they have, sometimes they will get an alert that they are entering an invalid amount.
The strange thing is if they decrease the amount of credit they want to use by $0.01 it applies properly. This looks like some kind of issue with how the system is rounding or not rounding store credit values at checkout.
-
Hello,
To replicate this issue is not that easy.
You must have added and removed store credits to an user to finally get this issue.
As a matter of fact, the following sequence did the trick for me:
From a brand new client, with no store credit: +5.01, +5, -10.01, +4.99, +0.02, +0.03, +5.03, -0.6 .
This way, user gets 9.47 store credits.In this case, the user can’t redeem 9.47 credits. But 9.46 is ok, so is 9.469…
That is because of float limitations (read this).
I did some debug and here is the results I got: screenshot of notification.
You can see there that$amount - $balanceequals MINUS ZERO dot zero (-0.000000), and not just ZERO (0.000000). This demonstrates that$amountis considered greater than$balance… On the contrary,$balance - $amountequals 0.000000 !Knowing that, the best I could achieve seems to be changing this line into :
if ($amount < 0 || round($amount, 2) > round($balance, 2) ) {as exposed in https://www.php.net/manual/en/language.types.float.php .Hope it helps.
The topic ‘Rounding issue with store credit’ is closed to new replies.
