[REF] Simplify getContributionStatuses
authorEileen McNaughton <emcnaughton@wikimedia.org>
Mon, 20 Dec 2021 01:15:14 +0000 (14:15 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Wed, 29 Dec 2021 22:02:05 +0000 (11:02 +1300)
commitf2f50411bb9ab9720bfad71997f3fb62e44804c3
tree441f58657c6e6eb9d3a227f0949a7fa064cae280
parentddee4195e948cc3cb0a924a036a10087d43dae43
[REF] Simplify getContributionStatuses

The effect of the altered calls to getContributionStatus is to get a small set of options returned.

In the cases of participant & membership forms this is pending or completed. For
the ufGroup option it seems cancelled & failed are also data entry options (this method
should still cope if they don't exist).

The existing function is really confusing - but all that confusion is designed for one
form - back office contribution form - and augmented by overloading by other functions.

This splits it out.

Could sites be using 'weird and wonderful other statuses'? Our system really is set up
around supporting specific statuses and doing financial entries for those so I think maybe
we are right to not try to pander to any that could exist
CRM/Contribute/BAO/Contribution/Utils.php
CRM/Core/BAO/UFGroup.php
CRM/Event/Form/Participant.php
CRM/Member/Form/Membership.php
CRM/Member/Form/MembershipRenewal.php