| Project: | Ubercart Ajax Attribute Calculations |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | AntoineSolutions |
| Status: | closed (fixed) |
Issue Summary
A client of mine requested a number of changes to uc_aac, including updating the SKU and weight fields and including support for other modules (such as uc_price_per_role) when recalculating the attribute prices. I have produced a heavily patched version of uc_aac which simplifies the existing server-side code and also removes the need for dynamic JavaScript, replacing it with a uc_aac.js file which does the client-side work.
The updated module is attached. This code is for 5.x but it should be possible to port it forward to 6.x. It has been tested on two of my development sites and is also in testing at present on one live site. Can you take a look at my changes and see if they may be suitable for opening a 2.x branch of development? I believe my changes should also help to close #470002: Let other modules interact with UC_AAC and #402632: uc_acc not updating attribute options.
| Attachment | Size |
|---|---|
| uc_aac-5.x.tar_.gz | 15.78 KB |
Comments
#1
Longwave, I really like the looks of this. I took basically the same approach as you for solving #470002: Let other modules interact with UC_AAC in a recent re-write/update of the module in preparation of either a 2 branch or an entirely different module, but had not done the js cleanup that you did here. I don't think I'll be starting a new branch for D5, but this definitely looks like a good path to take for a D6 branch and how I'd like to see things working moving forward. I will probably play around with this a bit, move it to D6, and then create a branch. Thanks for posting your code up here.
#2
cYu,
I have ported the code provided by Longwave to Drupal 6.x and Ubercart 2.x. Many people need this code for their Drupal 6 Ubercart 2 sites, so please help us get this into a uc_acc 6.x-2.x build right away. I would be more than willing to come on as a co-maintainer if you need help maintaining this module.
#3
AntoineSolutions: I've granted CVS access, go ahead and create the branch to house this code. Thanks for the work.
#4
cYu,
Thanks for CVS access. I have committed the code and a new 6.x-2.x branch has been created. I am marking this issue fixed. Any new issues can be created against the new branch.
*edit: A new 6.x-2.x-dev snapshot will be available for download within 24 hours.
Cheers,
Antoine
#5
Automatically closed -- issue fixed for 2 weeks with no activity.
#6
I would like to know if there's the possibility of adding a "Please Select..." default option on my select boxes. I'm afraid most avid buyers will just click add to cart without choosing the proper options.
I'm thinking this could be done by simply concatenating that option to the output of select boxes and linking it to an alert js function. Do you see any problems that could arise from doing this?
Or maybe there's someone who has a better idea... or maybe it's already implemented and I couldn't find it. Either way, I would really appreciate some input.
#7