[Ref] Remove copy and paste overkill
authoreileen <emcnaughton@wikimedia.org>
Thu, 11 Mar 2021 21:39:59 +0000 (10:39 +1300)
committereileen <emcnaughton@wikimedia.org>
Fri, 12 Mar 2021 09:22:13 +0000 (22:22 +1300)
commit5072c8946282fb50193e5e10aecabbf597645328
tree3d40307991bd79ce2c8ae0bb1fa5f4b35a05310b
parent7ec291020a29c6487fc439b1fcdffcccd8cb213e
[Ref] Remove copy and paste overkill

This is a previously shared function that was copied and pasted back. However in the
context of this form they don't make sense because

1) It doesn't make sense to try to maintain the integrity (via the transaction)
of the contribution & recurring contribution over & above the other
items on the form - without the memberships they don't actually have integrity
and this might have made a little more sense front end in terms
of ensuring they were committed if a payment is made but
we can expect a back office user to observe an error

2) setting the contribution id on the form here doesn't make sense as
this line is only hit for credit-card recurring contributions and
either we are doing it elsewhere anyway or we should do it nowhere - there
is no logic for 'only do this in this one obscure flow' that makes
sense back office. Front office this was likely added for an otherwise missed
scenario
CRM/Member/Form/Membership.php