CRM_Utils_SQL_* - Properly interpolate NULL values

Overview
--------

The `CRM_Utils_SQL_*` classes are used to generate SQL expressions with
properly escaped data. However, in many cases, the values of `NULL` are
processed in a way which clearly doesn't work, e.g.

Before
------

When passing in a parameter with a `NULL` value, the parameter *is not evaluated at all*.

```
>>> CRM_Utils_SQL_Select::from('civicrm_contact')->where('do_not_email IS @foo', ['foo' => NULL])->toSQL()
=> """
   SELECT *\n
   FROM civicrm_contact\n
   WHERE (do_not_email IS @foo)\n
   """
```

Specifically:

* `CRM_Utils_SQL_Select` - PHP NULL parameters are ignored (as if the value does not exist). No test coverage.
* `CRM_Utils_SQL_Delete` - PHP NULL parameters are ignored (as if the value does not exist). No test coverage.
* `CRM_Utils_SQL_Insert` - PHP NULL inputs are represented as SQL NULLs. No test coverage.

After
-----

When passing in a parameter with a PHP `NULL` value, the parameter is encoded as a SQL `NULL` value.

```
>>> CRM_Utils_SQL_Select::from('civicrm_contact')->where('do_not_email IS @foo', ['foo' => NULL])->toSQL()
=> """
   SELECT *\n
   FROM civicrm_contact\n
   WHERE (do_not_email IS NULL)\n
   """
```

Specifically:

* `CRM_Utils_SQL_Select` - PHP NULL inputs are represented as SQL NULLs. Has test coverage.
* `CRM_Utils_SQL_Delete` - PHP NULL inputs are represented as SQL NULLs. Has test coverage.
* `CRM_Utils_SQL_Insert` - PHP NULL inputs are represented as SQL NULLs. Has test coverage.

Comment
-------

I'm generally hesitant about changing behavior like this.  However, given
that the old behavior was clearly non-functional, I'm struggling to figure a
way where this would be problematic. Moreover, I'm a bit encouraged by the
fact that:

* There are unit-tests specifically for these classes,
* When grepping core for references to `CRM_Utils_SQL_ `, there are several
  matches involving APIv3 and ActionSchedules -- both of which have a long
  practice of unit-testing.
* When grepping my local ext's, there are several matches for things in
  `apiv4` and `civicase` - which also some test-coverage.

Merge pull request #14239 from colemanw/haveACrack

Remove broken delete from customGroupCreate

Remove broken delete from customGroupCreate

Merge pull request #14181 from eileenmcnaughton/group_search

dev/core#926 [ref] adds a test & does a preliminary extraction

Merge pull request #14195 from mlutfy/dev932

dev/core#932 Fix dedupe contacts flip selection

Merge pull request #14160 from demeritcowboy/fix-change-status-warning

dev/core#896 - fix notice warning on closing a case

Merge pull request #14206 from mattwire/paymentforms_refactor

REF Refactor to reduce duplication on payment forms

Merge pull request #14226 from eileenmcnaughton/master

Update testGetActivityAccessCiviCRMEnough test for clarity

Merge pull request #14238 from civicrm/5.14

5.14

Merge pull request #14118 from eileenmcnaughton/no_fin_item

Fix financial acl permissions to respect check_permissions

Merge pull request #14236 from seamuslee001/activity_test

Fix Failing Activity Test

Fix Failing Activity Test

Merge pull request #14223 from eileenmcnaughton/5.14

dev/core#942 fix failure to render names for some activities

dev/core#942 fix failure to render names for some activities

Overview
----------------------------------------
Set limit for activity_contact retrieval to 0, allowing to retrieve more than 25 activity contacts when rendering the first 25 activities on the activity contact tab

Before
----------------------------------------

After
----------------------------------------

Technical Details
----------------------------------------
This moves the logic for retrieving the target contacts back into the getActivities function. We are stil not wanting to bypass the ACLs so still using the
api but strictly limiting the number of contacts we retrieve (at the cost of extra queries, but cheap ones).

Some tests added on the Bulk Mail activity.

Comments
----------------------------------------

Merge pull request #14233 from seamuslee001/participant_count_search_fix

dev/core#956 Fix hard breakage in find participants form when you do a filter on o…

Merge pull request #14220 from pradpnayak/core/issues/923

core/issues/923, Fixed notice error when creating/editing profile

dev/core#956 Fix hard breakage in find participants form when you do a filter on one event and n participant statues

Merge pull request #14216 from alifrumin/dontmunge

When creating relationship types don't munge names

Merge pull request #14227 from civicrm/5.14

5.14 to master

Merge pull request #14214 from jitendrapurohit/activity-input

Add select2 to input field of datepicker

Merge pull request #14221 from seamuslee001/activty_wrapper_date_picker_template

Add in Wrapper template around DatePickerRange template to have bette…

Update testGetActivityAccessCiviCRMEnough test for clarity

Add in Wrapper template around DatePickerRange template to have better layout of Activity Search Form

Allow for colspan and class to be set when adding in wrapper

Update the wrapper to be only one td and not all and fix activity template appropriately

Merge pull request #14219 from colemanw/hover

dev/core#950 Remove deprecated :hover jQuery selector

Remove deprecated :hover jQuery selector

core/issues/923, Fixed notice error when creating/editing profile

Merge pull request #14217 from prondubuisi/this-in-static-method

dev/core#945 - fix  use of $this error in static method

Merge pull request #14004 from mfb/set-utf8

Remove CIVICRM_TEMP_FORCE_UTF8; deprecate TempTable::setUtf8()

Merge pull request #14185 from mfb/watchdog

Pass the RFC 3164 severity level thru to drupal watchdog.

dev/core#945 - fix  use of $this error in static method

When creating relationship types thru the api Don't munge names

Add select2 to input field of datepicker

Merge pull request #14203 from yashodha/dev_705

(dev/core#705) Disabling Alphabetical Pager is not respected for events

Merge pull request #14183 from eileenmcnaughton/copy_dao

Fix CRM-21832 - Recurring activities don't carry over custom datas & add test provided by Agileware

Merge pull request #14205 from civicrm/5.14

5.14

Merge pull request #14210 from seamuslee001/5_13_2_release_notes

Add 5.13.2 release notes

Update 5.13.2.md

Update Release notes for dev/core#940

Include Dave D for review purposes

Add 5.13.2 release notes

Merge pull request #14208 from seamuslee001/dev_core_940_5_14

(dev/core#940) Fix regression involving Email Processor filing and ".unknown" attachments

Fix Regression in Email Processor filing all emails as .unknown attachments

Merge pull request #14198 from eileenmcnaughton/pay_activity

[REF+ test] start process of cleaning up payment activity handling

Merge pull request #14197 from eileenmcnaughton/cont_cancel

dev/core#927 [ref] Extract contribution.cancel function

Refactor to reduce duplication on payment forms

Merge pull request #14204 from totten/5.14-act-sort

(dev/core#934; followup) Fix escaping on new query code

Fix failure to copy custom fields & add test provided by Agileware

Per https://github.com/civicrm/civicrm-core/pull/13470 custom fields are
inconsistently copied where copying entities. This makes the code from
BAO_Event called from the genericCopy function.

I did a bit of an audit and the places where this is currently called from don't appear
to call the copyGeneric function with the 'custom' param that would have activated the old
code. I also consistently removed the & when it was being called so I could take it
out of the signature.

The original PR handled tags as well, but not in a generic way. I've left that out of scope
but the test is present, commented out, so it would be easy enough to revist

(dev/core#934; followup) Fix escaping on new query code

This updates a line which was added in the past day (#14194) to ensure that
the data is escaped.

(dev/core#705) Disabling Alphabetical Pager is not respected for events

Merge pull request #14199 from colemanw/relActive

Stop overriding is_active default when saving new relationship

Stop overriding is_active default when saving new relationship

For some obscure reason the relationsip BAO was forcing the is_active field
to default to 0 even though the schema sets a more sensible default of 1.

Merge pull request #14202 from civicrm/5.14

5.14

Merge pull request #14200 from demeritcowboy/to-be-or-not-to-be

contributor-keys file - jira/lab.c.o username fix

Merge pull request #14194 from eileenmcnaughton/activity_sort

dev/core#934 Fix regression on sorting activity tab by 'Added by'

fix change case status warning

Merge pull request #14177 from eileenmcnaughton/free

IDE re-format of tools directory

[REF] start process of cleaning up payment activity handling

At the moment activities are created when using the additional payment form but not the UI.
This needs cleaning up & this takes the first step by adding testing & slightly
simplifying where decisions are made about variables (assign title directly rather than component
just to assign title deeper down

Merge pull request #14180 from eileenmcnaughton/recur_search

Enotice fix on creating a smart group

dev/core#934 Fix regression on sorting activity tab by 'Added by'

jira

dev/core#927 Extract contribution.cancel function

dev/core#932 Fix dedupe contacts flip selection

Merge pull request #14193 from seamuslee001/5.14

On PHP 7.2 we cannot start session until it has been configured by Dr…

On PHP 7.2 we cannot start session until it has been configured by Drupal.

Merge pull request #14192 from civicrm/5.14

5.14

Merge pull request #14191 from seamuslee001/5_13_release_notes

Add in release notes for 5.13.1 drop

Merge pull request #14074 from mfb/session-start

Don't call session_start() before CMS bootstrap (PHP 7.2 compat)

Merge pull request #14135 from colemanw/menubarVars

Hook to alter menubar css variables & fix breakpoint in WP

Update 5.13.1.md

Merge pull request #14189 from MegaphoneJon/wordpress-26-5.14

wordpress#26 - fix REST

Add in release notes for 5.13.1 drop

wordpress#26 - fix REST

Merge pull request #14188 from seamuslee001/dev_core_931_5_14

Resolve dev/core#931 by not doing translation on the query if field e…

Resolve dev/core#931 by not doing translation on the query if field exists during the upgrade process

Pass the RFC 3164 severity level thru to drupal watchdog.

Merge pull request #14171 from eileenmcnaughton/copy_dao

[ref] Move copyCustomFields function from Event to Core_DAO for re-usablibilty

Merge pull request #14152 from seamuslee001/dev_translation_26

dev/translation#26 Fix Notice errors caused by l10n rewriting of quer…

IDE re-format of tools directory

We didn't fix array syntax on this one yet

[REF] premlinary tidy up of code as tested - ready for actual fix

Add unit test for group_contact_status in search

This currently works for non-smart groups so this will pass.

Enotice fix

Dates back to https://github.com/civicrm/civicrm-core/commit/eda34f9b15c494715e8fc54f09b5ea4308d16b17#diff-8041069091a94b730ff8ad60bdd24042R70
(Mar 2018).

To recreate do a first name search & create a smart group from it

Merge pull request #14178 from eileenmcnaughton/config

[cleanup] Remove deprecated pattern &CRM_Core_Config::singleton()

Merge pull request #14179 from eileenmcnaughton/free_dao

dev/core#562 [cleanup] remove some more instances of ->free()

Set version to 5.15.alpha1

Set version to 5.14.beta1

dev/core#562 remove some more instances of ->free()

[cleanup] Remove deprecated pattern &CRM_Core_Config::singleton()

Merge pull request #14176 from civicrm/5.13

5.13

Merge pull request #14056 from agh1/5.13.0-releasenotes

5.13.0 release notes

Update 5.13.0.md

Merge pull request #14173 from mlutfy/cart-payment-cid

Event Cart: pass the contactID to fix payment on Stripe

Merge pull request #14170 from mlutfy/cart-emails

Event Cart: fix sending of email receipts

Merge pull request #14175 from mlutfy/cart-cc-icons

Event Cart: add support for the Credit Card type icons

Merge pull request #14174 from mlutfy/cart-allow-same-participant

Event Cart: honor the allow_same_participant_emails setting

Event Cart: pass the contactID to fix payment on Stripe

5.13.0 release notes: late changes

uppercase if work is continued or begun

moving gitlab issues to the end

5.13.0 release notes: bulk of edits

5.13.0 release notes: added boilerplate

5.13.0 release notes: raw from script