Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I am working on writing a custom conditional action, and was looking at the different triggers. The one I want to use, 'uc_recurring_renewal_failed' is fired in uc_recurring_renew(). In the case of of Paypal WPS, the IPN page callback only calls uc_recurring_renew() for txn_type = 'subscr_payment' or txn_type = 'recurring_payment'.
I think that adding these two lines from uc_recurring_renew() to the subscr_failed and recurring_payment_failed conditions would fix this.
module_invoke_all('recurring_renewal_failed', $order, $fee);
ca_pull_trigger('uc_recurring_renewal_failed', $order, $fee);
Does that sound right, or am I missing something?
Comment | File | Size | Author |
---|---|---|---|
#1 | uc_recurring-paypal_wps_renewal_failed_trigger-1414830-1.patch | 1.14 KB | jojonaloha |
Comments
Comment #1
jojonaloha CreditAttribution: jojonaloha at Metal Toad commentedHere is a patch for the proposed change.