Remove all calls to BAO_Contribution::completeOrder other than from Payment.create
The goal is that we should either
- call Payment.create to add a payment - which would itself call completeOrder
- OR call repeatTransaction via the repeattransaction api to create a pending order - which we would then update to cancelled or failed if need be or - if completed call payment.create
Note that currently Payment.create calls Contribution.completetransaction - the goal is to reverse this
I'm currently working on removing calls to completeOrder other than from the 2 api calls - at which point we can change the api calls to no longer call completeOrder directly