When you have multiple point TID's features for a single product, going back to edit one will always show you the last point feature you've added for that product. This is because the database record is loaded by nid rather than pfid. See the attached patch for a proposed fix.

Also, it seems like both the upid and nid columns aren't needed in the uc_userpoints_products table.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bmagistro’s picture

Category: bug » feature

I do not consider this a bug. It was not designed for that purpose at the time. Changing to feature request.

rbayliss’s picture

Fair enough. What feels buggy about it is that you can add multiple points features to a product, but then they don't work. Regardless, it's a few lines of code away from having this feature. This last patch also changes the point decrementing on order complete to allow for multiple TID's. Please note that I also changed a couple of strings so they made more sense (to me).

YK85’s picture

subscribing

that0n3guy’s picture

subscribe...

eme’s picture

Works fine for me. Would be good to include it in dev. Thx.

bmagistro’s picture

I am looking at moving the current beta to stable and then integrating this. Can you guys confirm that it has been "stable" and some of the major bugs (such as the double discounting) are not exhibited anymore? Once that moves there I plan to start addressing these feature requests.

thePanz’s picture

This patch do not takes into account the product quantity during userpoints save process!
Cheers

thePanz’s picture

Component: Code » Product

Moved the issue to "Product" component