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)

Comments

amack’s picture

Assigned: Unassigned » amack
amack’s picture

Ignore that comment about the product renewal page being broken

gordon’s picture

This patch is broken too.

amack’s picture

StatusFileSize
new666 bytes

Sorry about the broken patches Gordon. This one should be fine

mo6’s picture

Version: 5.x-3.0 » 5.x-3.4
Status: Needs review » Needs work

The patch is broken, but the issue still applies. Please submit a new CVS patch against 5.x-3.4. Thanks.

davea’s picture

StatusFileSize
new671 bytes

This 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

davea’s picture

Status: Needs work » Fixed

Fixed in the latest dev version.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.