diff --git a/CRM/Core/Payment/StripeIPN.php b/CRM/Core/Payment/StripeIPN.php index 62140d43385c52f47c641ee99315013ffd3a2b89..f1706188a6d47ae919e4ebe9992efcb49ab65669 100644 --- a/CRM/Core/Payment/StripeIPN.php +++ b/CRM/Core/Payment/StripeIPN.php @@ -214,7 +214,6 @@ class CRM_Core_Payment_StripeIPN { } break; - case 'invoice.paid': case 'invoice.payment_succeeded': // Successful recurring payment. Either we are completing an existing contribution or it's the next one in a subscription if (!$this->setInfo()) { diff --git a/CRM/Stripe/Webhook.php b/CRM/Stripe/Webhook.php index 2b48b5b1fa4847909d67f00904d9879881a69d91..f0c9cdbf638ed4904d8184e20a106c77731861f8 100644 --- a/CRM/Stripe/Webhook.php +++ b/CRM/Stripe/Webhook.php @@ -228,7 +228,8 @@ class CRM_Stripe_Webhook { public static function getDefaultEnabledEvents() { return [ 'invoice.finalized', - 'invoice.paid', + //'invoice.paid' Ignore this event because it sometimes causes duplicates (it's sent at almost the same time as invoice.payment_succeeded + // and if they are both processed at the same time the check to see if the payment already exists is missed and it gets created twice. 'invoice.payment_succeeded', 'invoice.payment_failed', 'charge.failed', diff --git a/info.xml b/info.xml index b113dff7ffa536e36792c0dcea9f3d7b8d898723..7a8c87874ebecf82209f955317c1bba4f36cea00 100644 --- a/info.xml +++ b/info.xml @@ -15,7 +15,7 @@ <author>Matthew Wire (MJW Consulting)</author> <email>mjw@mjwconsult.co.uk</email> </maintainer> - <releaseDate>2020-08-25</releaseDate> + <releaseDate>2020-08-26</releaseDate> <version>6.5-dev</version> <develStage>alpha</develStage> <compatibility>