Fix BillingName to be assigned in email receipts IF available

[eileenmcnaughton]

Overview

Moves the assignment of billingName to message templates to a place used by all receipts rather than relying on form to assign

Before

Not assigned for IPN type receipts - even if stored

After

Assigned

Technical Details

Previously it was assigned at the form layer, now we assign it in the email sending routine, depending on
whether the contribution has address_id & there is a value assigned

Comments

@seamuslee001 this will beat one of the blockers to getting 'contributeMode' out of the receipts

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Code looks right to be and i'm confident this has plenty of unit tests so if Jenkins is ok with this i am too merge on pass

[eileenmcnaughton]

@seamuslee001 I think I'd be OK with pulling the other check that is failing out of the test - based on the current params it legit shouldn't have the string 'Credit Card Information' & I feel like we've addressed this one 'properly' so could take a shortcut on that one

[seamuslee001]

@eileenmcnaughton test fails relate

[eileenmcnaughton]

Nothing like a good enotice