Update Mailing.php is_deceased queries

[galgeek]

Include contacts where is_deceased is set to null or False (not True) in bulk e-mail lists

[civicrm-builder]

Can one of the admins verify this patch?

[bhugh]

Two observations, just based on experimentation:
#1. The change to is_deceased <> 1 doesn't seem to work for the case where is_deceased is set to NULL (at least, not on my DB installation).

Is seems that you need (is_deceased = 0 OR is_deceased IS NULL).

#2. There are several other things that malfunction based on this same problem, scattered around CiviCRM. I spotted a similar issue in Labels, but more relevant to this problem is that pulling tokens in will fail if the is_deceased is set to NULL. That is because utils/Token.php has the same is_deceased=0 idiom around line 1198:

https://github.com/civicrm/civicrm-core/blob/master/CRM/Utils/Token.php

I'm wondering if a better solution might not be to somehow check periodically and set all is_deceased to 0 if they are NULL.

[civicrm-builder]

Can one of the admins verify this patch?

[totten]

jenkins, ok to test

[davecivicrm]

@bhugh Are you aware of recent additional PR which set defaults for these columns to 0 and updated all NULL values to 0 : https://github.com/civicrm/civicrm-core/pull/7386/files