Closed (fixed)
Project:
Ubercart Userpoints
Version:
6.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
7 Jul 2009 at 07:06 UTC
Updated:
18 Jun 2015 at 03:32 UTC
Jump to comment: Most recent
Comments
Comment #1
dobeerman commentedissue is solved?
Comment #2
Anonymous (not verified) commentedno, problem still exists, site dev is on hold.
Comment #3
zeezhao commentedSounds like fix posted here, or a related fix?
http://www.ubercart.org/forum/support/4558/balance_not_checked_order#com...
Edit:
I tested the latest dev version and it appears to work i.e. user points get subtracted from balance.
However, the scenario that did not work is when the "Points Discounts" is used on an order to reduce final amount to pay by other means. The points did not get subtracted from user's balance, even though invoice says it got deducted. So a bug here but not sure if also tied into uc_discounts module. Also, it happens when points you try to deduct is more than available balance.
Comment #4
savmac commentedI encountered this problem as well. Not only was it not deducting from the user's points, it actually doubled the "Approved Points Balance" and the "Net Points Balance" on the /myuserpoints page.
I remedied this by removing all of the taxonomy categories for the points so that they all are now "uncategorized". Now everything works just fine except there is now a rounding error. On a product costing $12.50, the payment module requires 1300 points to pay for it and then deducts 1300 points after payment instead of 1250 (100 points for every dollar in settings).
Any ideas on how to fix this?
Comment #5
savmac commentedI was able to fix the rounding error by removing the rounding in the payment module. It seems to have done the job although I hope I didn't break anything :) I could not think of an instance where you would want to round the product/order total pricing before applying the points, am I missing something? It must have been written that way for a reason right?
Thanks!
Comment #6
longwaveShould be fixed in latest -dev now #698884: Userpoints Payment improvements has been committed.