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.
Is there a way to prevent subscriptions paid by eCheck from going active until after the echeck clears?
Currently if someone pays for a subscription via echeck, LM_Paypal activates the subscription immediately instead of waiting for the echeck to clear..
Comment | File | Size | Author |
---|---|---|---|
#2 | lm_paypal_subscriptions_escalate_on_paid.patch | 2.55 KB | mrtoner |
Comments
Comment #1
mrtoner CreditAttribution: mrtoner commentedI noticed that when looking at the code last night. Although lm_paypal ignores pending payments, it accepts a subscription creation as reason to assign the role/group/node. I understand the reason for this: a free trial won't generate a payment for the trial period, so the assignation must happen during subscription creation.
To keep non-free subscription signups from going active when the payment is pending, it should be possible to test the IPN variable amount1 and only activate for a value of '0' (free trial). All other signups will only activate upon payment completion. I'll write a patch and upload it tomorrow.
Comment #2
mrtoner CreditAttribution: mrtoner commentedOkay, this patch changes the behavior of a subscription signup. A transaction of "subscr_signup" will only escalate a role/group if the signup has a free trial. For all other signups, the role/group escalation will occur when the payment is completed.
YMMV. Please test fully.
Comment #3
MrGeek CreditAttribution: MrGeek commentedThanks..
I'm willing to give it a run, but, can you point me in the direction of the easiest way to apply this patch ?
Comment #4
mrtoner CreditAttribution: mrtoner commentedSee http://drupal.org/patch/apply
Comment #5
MrGeek CreditAttribution: MrGeek commentedThank you MrToner..
Before i go ahead and apply the patch i want to verify this is for/will work with my v4.7 install ?
Comment #6
mrtoner CreditAttribution: mrtoner commentedI only tested this on a 5.x installation, but the version of lm_paypal that the patch applies to (HEAD) is suitable for 4.7 as well. There's nothing in there specific to either core version.
Comment #7
John Franklin CreditAttribution: John Franklin commentedComment #8
John Franklin CreditAttribution: John Franklin commentedThis patch no longer applies cleanly. A surprise to no one, I'm sure.