So the readme for the plugin says that the workflow options may not be working. This is true, however I have added the fix in the module and tested. I do not know how to create the diff files though and I added a "Seats" option into the table.
I have uploaded my .module file. I would appreciate if people would test the fix. Basically they weren't checking for the workflow variable at all - only if the node type had signup_ecommerce enabled. So on both order completion and payment completion it was trying to add the signup.
Please let me know if you have questions or concerns about the code. Also, if you would like to clean the code up be my guest.
The only changes were in the following two functions:
function signup_ecommerce_ec_transactionapi and function signup_ecommerce_ecommerceapi
I added if statements checking for the correct workflow option
In signup_ecommerce_ecommerceapi it should add the signup on payment completion so I added:
if (variable_get('signup_ecommerce_workflow_' . $node->type) == 2) {
In signup_ecommerce__ec_transactionapi it should add the signup on ordercompletion so I added:
if (variable_get('signup_ecommerce_workflow_' . $node->type) == 1) {
Anyway - check out the file. Sorry had to make it a zip to upload ... need to figure out this .diff and .patch stuff.
Thanks,
~ Tom
| Comment | File | Size | Author |
|---|---|---|---|
| signup_ecommerce.zip | 2.14 KB | tcocca |
Comments
Comment #1
tizzo commentedGreat! Thanks for uploading a fix! I'll take a closer look at this as soon as I can. Sorry for the delay.
Comment #2
socialnicheguru commentedI will try these fixes out.
did you ever get them committed?