[REF] Move not-actually shared-code out of shared code function
authoreileen <emcnaughton@wikimedia.org>
Mon, 21 Sep 2020 00:16:48 +0000 (12:16 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 21 Sep 2020 00:16:48 +0000 (12:16 +1200)
commita83a1f47f25254f79d6028d5648dae64a40a5706
treec56b73316f9501add95f28e7356ccf02668282e9
parente49c90d1d45811c480d04d2ac5bee3c6e5b86ff0
[REF] Move not-actually shared-code out of shared code function

On looking at the recur function it turns out it never calls  without a
paymentStatus of 'Completed' - so the lines handling other payment statuses are not actually shared
between the 2 code paths but specifically belong to the 'main' (non-recur) code path.

Moving it out of this function we start to see there is very little shared code & probably no case for
the function to exist when we get down to it
CRM/Core/Payment/PayPalIPN.php