Switch to calling Payment.create api when processing a refund from AdditionalPayment...
authoreileen <emcnaughton@wikimedia.org>
Fri, 24 May 2019 01:46:52 +0000 (13:46 +1200)
committereileen <emcnaughton@wikimedia.org>
Fri, 24 May 2019 03:52:22 +0000 (15:52 +1200)
commit957655fe51f2944c6062434a9c0ec8b325a735f8
tree0c9cede9a7785b988320cc8dc687152e0be0399c
parenta0741dc66a56012dca28350f8c1ddaad5ec03918
Switch to calling Payment.create api when processing a refund from AdditionalPayment form

This is in conjunction with a series of changes consolidating & adding testing for creating payments with the goal
being that the logic is all in the Payment.create api/BAO & none of the logic remains in the CRM_Contribute_Form_AdditionalPayment form

This change completes switching the refund action over. There is logic in there to update the Participant status to
Registered when a refund is processed - seemingly regardless of what it's original status is or the appropriate status
afterwards.

I tested through the UI and if you create a participant record & then change the line items, thus reducing the cost
and changing the contribution status to 'Pending Refund' the participant status remains 'Completed' so I think
this was not working / not required already & I simply dropped it
CRM/Contribute/BAO/Contribution.php
CRM/Contribute/Form/AdditionalPayment.php
CRM/Financial/BAO/Payment.php