Follow up cleanup on multiple participant form edit
authorEileen McNaughton <emcnaughton@wikimedia.org>
Fri, 3 Nov 2023 01:42:33 +0000 (14:42 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Fri, 3 Nov 2023 01:42:33 +0000 (14:42 +1300)
commitcf131c8685497eb8c99985b56d569f67d618c53d
tree5f69eb22ab63433da2709437cff5f40b1aeb56fb
parentff573b52f51813777c3a3a16b9fea0a3facda851
Follow up cleanup on multiple participant form edit

In testing on the recent regression some code turned out to
be unreachable.

In the scenario were we have a pending pay later registration for multipl
participants we find that the field priceSetId is empty when submitted
and the field total_amount must be submitted, if it is submitted.

Therefore we find that a) any value added to total_amount in the
previously-shared preparePaidEventProcessing is overwritten
shortly after by the submitted value
and hence should not be set in this flow in preparePaidEventProcessing.

Also the code that sets additionalDetails is only hit when priceSetID is
not empty but it never is per above. So that code is setting an unused variable
CRM/Event/Form/Participant.php