Posted by rbayliss on February 3, 2011 at 9:33pm
6 followers
| Project: | Ubercart Userpoints |
| Version: | 6.x-2.0-beta1 |
| Component: | Product |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
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.
| Attachment | Size |
|---|---|
| multiple_tid_per_product.patch | 1.26 KB |
Comments
#1
I do not consider this a bug. It was not designed for that purpose at the time. Changing to feature request.
#2
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).
#3
subscribing
#4
subscribe...
#5
Works fine for me. Would be good to include it in dev. Thx.
#6
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.
#7
This patch do not takes into account the product quantity during userpoints save process!
Cheers
#8
Moved the issue to "Product" component