[REF] Move not-actually shared-code out of shared code function

[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