dev/membership#13 Add handling for missing MembershipPayment record.
authoreileen <emcnaughton@wikimedia.org>
Fri, 16 Aug 2019 23:00:32 +0000 (11:00 +1200)
committereileen <emcnaughton@wikimedia.org>
Wed, 4 Sep 2019 23:14:57 +0000 (11:14 +1200)
commitfefee63643af32bc28fdc43196e886ede6194006
treeedf2bdced7b80128ef1f53a4be15fd430dd5f784
parent843756645cbf84e8e251697f4c89c6cecd752103
dev/membership#13 Add handling for missing MembershipPayment record.

We have an intermittent regression on extending memberships off paypal payments. This adds debug messages
and handling / tests for the scenario I suspect may be in play.

The 'decision' the code makes about how to create recurring transactions is all dependent on the 'template transaction'
- if that is not correct the follow on payments will not be.

This PR allows the membershipID input from paypal to be passed through and used.  This will override any
intended changes - but that just restores previous behaviour and we do not have good handling for these

https://lab.civicrm.org/dev/membership/issues/13
CRM/Contribute/BAO/Contribution.php
CRM/Core/Payment/PayPalIPN.php
api/v3/Contribution.php
tests/phpunit/CRM/Core/Payment/PayPalIPNTest.php