Closed (fixed)
Project:
e-Commerce
Version:
5.x-3.4
Component:
ec_recurring
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
9 Apr 2007 at 06:57 UTC
Updated:
23 Jul 2008 at 10:56 UTC
Jump to comment: Most recent file
ec_recurring_token_values() returns an incorrect value for the product renewal URL as it assumes the existence of an variable '$product', which, as far as I can see, is not defined anywhere. The attached patch uses the defined variable $entry to get the node id, producing a correct URL (although the renewal page itself is still broken for me, but that's probably a job for a different bug)
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | ec_recurring-080522.patch | 671 bytes | davea |
| #4 | ec_recurring.3_0.patch | 666 bytes | amack |
| ec_recurring.3.patch | 493 bytes | amack |
Comments
Comment #1
amack commentedComment #2
amack commentedIgnore that comment about the product renewal page being broken
Comment #3
gordon commentedThis patch is broken too.
Comment #4
amack commentedSorry about the broken patches Gordon. This one should be fine
Comment #5
mo6The patch is broken, but the issue still applies. Please submit a new CVS patch against 5.x-3.4. Thanks.
Comment #6
davea commentedThis patch applies correctly to ec_recurring.module that comes with the 2007-Oct-17 tarball of ec-5.x-3.4 .
PLEASE TEST!!!!
Thanks!
DaveA
Comment #7
davea commentedFixed in the latest dev version.
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.