Update ContributionCancelActions to also handle 'failed'
authoreileen <emcnaughton@wikimedia.org>
Sun, 22 Nov 2020 21:05:50 +0000 (10:05 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 23 Nov 2020 20:26:33 +0000 (09:26 +1300)
commit61470a1a09b2122b99243c042ede134bf5c9e2de
treeb1d9e9f77b6ccd20a84e9ac24c791fee739d74f6
parent9c25ba085848043e887960fbeb49e35e5c1e05e2
Update ContributionCancelActions to also handle 'failed'

The failed action has no discernable difference from the no-longer-used cancel action. The cancel action had some minor refactoring
done to work through the conclusion that the call to addRecurLineItems was never reached in a meaningful way
but I think we can skip all that & see that fail is not functionally different to cancel & just deprecate the function.

At this point we are very close to simply removing the last core handling for failed & cance & making the extension visible.

The only thing I can see that still needs checking in the 2 functions still to be removed is the handling of
is_override & override_date is tested - which I will do as a follow up
CRM/Core/Payment/BaseIPN.php
CRM/Core/Payment/PayPalIPN.php
CRM/Core/Payment/PayPalProIPN.php
ext/contributioncancelactions/contributioncancelactions.php
tests/phpunit/CRM/Core/Payment/BaseIPNTest.php