dev/financial#162 Simplify decision as to whether to use a pdf on membership emails
authoreileen <emcnaughton@wikimedia.org>
Thu, 10 Dec 2020 21:08:55 +0000 (10:08 +1300)
committereileen <emcnaughton@wikimedia.org>
Thu, 10 Dec 2020 21:08:55 +0000 (10:08 +1300)
commit4f4d52d5007dea3552ef29535427192a0eac35cf
tree778bb221e51a9d06ef0005d7b5295679fde5809e
parentef84a7da1da67fa6b68f3dd51af5a6d8c77a06d4
dev/financial#162 Simplify decision as to whether to use a pdf on membership emails

Currently the code that determines whether to attach a pdf for membership emails only attaches it as a pdf if the tax amount is non-zero
This seems both confusing and pointless  - the proposal here is to send as a pdf as long as

is_email_pdf is true and
invoicing is enabled

I feel like 1 and 2 could be separated but they are currently linked so I would suggest any proposals to change that are a follow up
CRM/Member/Form/Membership.php