dev/core#861 Ensure that when processing mailings that no emails are …

[seamuslee001]

…sent to deceased contacts or those with emails now on hold

Overview

This resurects #14020 from Yash by adding a test and also expanding the checks so that for SMSes we also exclude deceased contacts

Before

Newly deceased contacts may get mailed if mailing scheudled before contact is marked as deceased

After

No deceased contacts emailed or smsed

ping @eileenmcnaughton

[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).

[eileenmcnaughton]

Merge on pass as this is a reviewer's cut

[eileenmcnaughton]

@yashodha I've given this MOP as a reviewer's cut - perhaps you could double check it though.

[yashodha]

@seamuslee001 thanks!

[MegaphoneJon]

@yashodha @seamuslee001 This PR seems to have caused a regression where "Enable multiple bulk email" setting is no longer respected: https://lab.civicrm.org/dev/mail/issues/60