Regression fix on Paypal Std with memberships not being finalised cor…

…rectly. Per civicrm#15053 (comment)
eileenmcnaughton · Oct 18, 2019 · 107f03e · 107f03e
1 parent a1b3cb0
commit 107f03e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/CRM/Core/Payment/PayPalIPN.php b/CRM/Core/Payment/PayPalIPN.php
@@ -336,7 +336,8 @@ public function main() {
 
     Civi::log()->debug('PayPalIPN: Received (ContactID: ' . $ids['contact'] . '; trxn_id: ' . $input['trxn_id'] . ').');
 
-    if ($this->retrieve('membershipID', 'Integer', FALSE)) {
+    // Debugging related to possible missing membership linkage
+    if ($contributionRecurID && $this->retrieve('membershipID', 'Integer', FALSE)) {
       $templateContribution = CRM_Contribute_BAO_ContributionRecur::getTemplateContribution($contributionRecurID);
       $membershipPayment = civicrm_api3('MembershipPayment', 'get', [
         'contribution_id' => $templateContribution['id'],