dev/core#1630 Partial fix on renew membership missing
authoreileen <emcnaughton@wikimedia.org>
Fri, 13 Mar 2020 02:03:08 +0000 (15:03 +1300)
committereileen <emcnaughton@wikimedia.org>
Fri, 13 Mar 2020 02:03:08 +0000 (15:03 +1300)
commit52b04786d3d0bdd9e17ccf198c3d28b96b51cd8c
tree012c9394a5010cb6c8643eb3d1c70ebdd88822b5
parent62cc144db1889f7976a1da6819cc963c486890d2
dev/core#1630 Partial fix on renew membership missing

I'm pretty comfortable that this regressed at least  a year ago - possibly much longer - so it's not
a recent regression. However, the issue is that when a price set is configured it is still 'listening to'
the  presence or otherwise of auto_renew in civicrm_membership_block.membership_types. This makes it
ignore that if not present (which is the case when creating a new price set from scratch).
When converting a price set the membership_types field is still populated and still kicks in
- I think we should really empty that out when a block is converted but I've left it out of scope as this
seems like an edge case bug & it was only the possibity it was a regression that caused me to dig into it
CRM/Contribute/Form/ContributionBase.php