[REF] Simplify code calculating the number of membership terms
authoreileen <emcnaughton@wikimedia.org>
Sun, 14 Mar 2021 03:29:50 +0000 (16:29 +1300)
committereileen <emcnaughton@wikimedia.org>
Mon, 15 Mar 2021 19:55:17 +0000 (08:55 +1300)
commite84c5dc202a37cf0d99af7e02ba472fa2da5388a
tree5d2605ec049c2a11c94e1d61e96a80bee70d56c4
parent835c9b62e2ccd39e81ea12e80509745a52eb2ac9
[REF] Simplify code calculating the number of membership terms

We have 2 possibilities
1) there is a price set in use - the number of line items comes from the calculated line items
2) default priceset in use - it is a submitted value

The option to submit is only present in 2 so we know that if it is
submitted it should take precedence, otherwise it comes from the line item

This simplifies the code to do that clearly
CRM/Financial/BAO/Order.php
CRM/Member/Form/Membership.php
tests/phpunit/CRM/Member/Form/MembershipTest.php