Fix fatal error regression in master

Fatal from this line due to pass by reference

https://github.com/civicrm/civicrm-core/pull/16514/files#diff-a8611c9a226a40b585001082b38467b0R207

Merge pull request #17565 from eileenmcnaughton/processor

dev/financial#131 Fix Elavon processor to throw exceptions

Merge pull request #17531 from colemanw/customIcon

Add icons to multi-valued custom groups and display on contact tab

Merge pull request #17576 from colemanw/emptyLeaf

APIv4 - Skip empty leaves in WHERE clause

Merge pull request #17578 from yashodha/dev_1787

(dev/core#1787) Regression : File fields export results in DB error

(dev/core#1787) Regression : File fields export results in DB error

Merge pull request #17573 from seamuslee001/incorrect_use_array_key_exists

[REF] Replace incorrect usages of array_key_exists when the variable …

[REF] Replace incorrect usages of array_key_exists when the variable is not an array with property_exists and also fix the parameter order for an implode statement

Merge pull request #15796 from seamuslee001/dev_core_183_requirements

dev/core#183 Use CRM_Utils_SQL_TempTable for generating temp table na…

APIv4 - Skip empty leaves in WHERE clause

Fixes an annoying problem with search builder
where the api gives a sql error while building your where clause.
The solution is to skip empty leaves, treating them as WIP.

Merge pull request #17574 from seamuslee001/string_offsets_curly

[REF] Using {} to access string or array offsets has been deprecated …

Merge pull request #17440 from eileenmcnaughton/log_summ

dev/core#1749 Fix missing parameter in link to logging detail

Merge pull request #15422 from artfulrobot/queue-parallel

Include a new SqlParallel queue type that enables multiple queue runners to process in parallel

Merge pull request #17575 from seamuslee001/php74_deprecations

[REF] Using ternary operation without nesting is deprecated and also …

Merge pull request #17570 from seamuslee001/conact_type_populated

[REF] Add in upgrade step to populate missing contact_type.name field

[REF] Using ternary operation without nesting is deprecated and also using methods and properly instanciate the CiviCase Info object in test

[REF] Using {} to access string or array offsets has been deprecated in PHP7.4

[REF] Add in upgrade step to populate missing contact_type.name field

Make changes to the other columns

Merge pull request #17571 from jitendrapurohit/core-1809

dev/core#1809 - Add start/end adjustment columns to membership status…

dev/core#1809 - Add start/end adjustment columns to membership status page

Merge pull request #17548 from eileenmcnaughton/contacttype

Set ContactType.name as required in the schema.

Merge pull request #17566 from eileenmcnaughton/dupe1

[NFC] Minor code cleanup

Set ContactType.name as required in the schema.

I came across this weird WHERE clause & I'm pretty sure it's compensating for a mistake in the schema

I also set what I think are sensible defaults on is_reserved & is_active

dev/financial#131 Fix Elavon processor to throw exceptions

In line with our goal of modelling 'good behaviour' in core processors

[NFC] Minor code cleanup

Merge pull request #17525 from mattwire/baseipnremovebillingid

Remove deprecated billingID from baseIPN

Merge pull request #17556 from eileenmcnaughton/renewline

[Ref] Remove never reachable lines.

Remove deprecated billingID from baseIPN

Merge pull request #17559 from eileenmcnaughton/realex

dev/financial#131 Fix Realex processor to correctly throw an exception

Merge pull request #17557 from ngo360/master

dev/core#1801 Fix hardcoded activity priority by label

dev/financial#131 Fix Realex processor to correctly throw an exception

Merge pull request #17515 from seamuslee001/replace_legacy_rebuild_module_data

[REF] Replace the deprecated system_rebuild_module_data function with…

Merge pull request #17555 from seamuslee001/dev_core_560_deprecated_function_warning

dev/core#560 Add in deprecated function warning now that fatal is dep…

dev/core#1801 Fix hardcoded activity priority

Merge pull request #17450 from mattwire/activitydefaults

dev/core#1782 Set activity_date_time and created_date to default to current_timestamp

[Ref] Remove never reachable lines.

There is an early return on  so updateStatusId can never be true

Merge pull request #17501 from eileenmcnaughton/guzzle3

[Ref] Convert Authorize.net doDirectPayment to use guzzle & add test

Merge pull request #17554 from alifrumin/entryFoundFlag

Fix "Undefined variable: entryFound" notice on Bookkeeping report

Merge pull request #17506 from mattwire/propertybagremovewarning

Payment Propertybag remove warning

dev/core#560 Add in deprecated function warning now that fatal is deprecated

[REF] Replace the deprecated system_rebuild_module_data function with equivilant extension.list.module service function calls to be Drupal 9 compatiable

Merge pull request #17552 from mattwire/deprecatedfunctionwarning

Support passing old method name into deprecatedFunctionWarning

set flag

Merge pull request #17553 from semseysandor/semseysandor-contributor

Update contributor-key.yml

Update contributor-key.yml

Merge pull request #17550 from eileenmcnaughton/request

Remove last instance of CRM_Core_Error::fatal....

Merge pull request #16610 from mattwire/settingstrait_datepicker

Allow adding datepicker to metadata based settings pages

Support passing old method name into deprecatedFunctionWarning

Remove last instance of CRM_Core_Error::fatal....

Merge pull request #16514 from mattwire/phpnotice_ccparams

Fix PHP notice when we don't have credit card params

Set activity_date_time and created_date to default to current_timestamp

Merge pull request #17374 from totten/master-setlocale

CRM_Core_I18n::setLocale() - Fix bug with repeated usage

Merge pull request #17179 from mattwire/addpaymentprocessortorecurlist

Add payment processor to details on list of recurring contributions

Merge pull request #17512 from semseysandor/relative-date-filter

dev/core#1798 - "End of yesterday" Relative Date Filter

Merge pull request #17523 from mattwire/customsearchzipcode

Zip code range search only supports numeric zip codes

Fix PHP notice when we don't have credit card params

Merge pull request #17549 from eileenmcnaughton/friend

[Ref] Remove another instance of fatal()

Merge pull request #17543 from eileenmcnaughton/dummy

[Ref] Readability extraction in Dummy class

Merge pull request #17546 from eileenmcnaughton/validate2

Remove isThrowException from CRM_Utils_Type::validate() signature

[Ref] Remove another instance of fatal()

[Ref] Readability extraction in Dummy class

Merge pull request #17545 from seamuslee001/dev_core_560_fin

dev/core#560 Convert final CRM_Core_Error::fatal to statusBounces or …

Remove isThrowException from CRM_Utils_Type::validate() signature

There are no instances in core where FALSE is passed

Merge pull request #17544 from eileenmcnaughton/validate

Fix validateAll to no longer support unused abort param

dev/core#560 Convert final CRM_Core_Error::fatal to statusBounces or Exceptions

Merge pull request #17542 from eileenmcnaughton/pay_error

dev/financial#131 Fix Eway processor to throw exceptions rather than return errors

Merge pull request #17533 from eileenmcnaughton/fatal

[REF] Replace some instances of fatal with thrown exceptions.

Merge pull request #17540 from colemanw/getActionsPerm

APIv4 - filter getActions results based on user permissions

Merge pull request #17491 from eileenmcnaughton/toemail

[Ref] Simplify the replacing of the email with the email from the url

Merge pull request #17500 from eileenmcnaughton/renew_can

[Ref] Throw exceptions from Authorize.net rather than return errors

Merge pull request #17499 from eileenmcnaughton/unhandled

[Ref] Remove calls to, and deprecate, unhandled function

Merge pull request #17504 from eileenmcnaughton/renewd

[REF] Remove handling for 2 fields not on the form

Fix validateAll to no longer support unused abort param

Default of TRUE is good here

Merge pull request #17539 from eileenmcnaughton/fatal2

Replace calls to fatal with statusBounces

dev/financial#131 Fix Eway processor to throw exceptions rather than return errors

This is part of a push to model the correct behaviour in processors we can alter. This is safe as CRM_Core_Payment
already converts these

[REF] Replace some instances of fatal with thrown exceptions.

This is ongoing cleanup to consisently throw execptions. The only thing notable is in the OG class where
it turned out the exception was thrown if a never-passed-in-parameter was passed in, so I removed the
construct

Merge pull request #17541 from seamuslee001/dev_core_560_third_to_last

dev/core#560 Replace deprecated CRM_Core_Erorr::fatal with Exceptions…

dev/core#560 Replace deprecated CRM_Core_Erorr::fatal with Exceptions or status bounces as appropriate

APIv4 - filter getActions results based on user permissions

Replace calls to fatal with statusBounces

Merge pull request #17511 from mattwire/paymentprocessoraddfrontendfunctions

Add isBackOffice, isPayLater, getPaymentMode helpers to frontendpaymentformtrait

Merge pull request #17534 from eileenmcnaughton/fatal2

Remove instances of fatal from upgrade folder

Earlier.day test

Merge pull request #17535 from eileenmcnaughton/gone

Remove unused deprecated class

Merge pull request #17528 from seamuslee001/nfv_apiv3_mem_type_apiv4_test

[NFC] Update existing APIv3 MembershipType Tests to also test against…

Merge pull request #17536 from eileenmcnaughton/fatal3

[Ref] Remove some instances of fatal in BAO classes

Merge pull request #17538 from civicrm/5.27

5.27

Merge pull request #17502 from eileenmcnaughton/first

Fix probably-unused core payment processors to throw extensions

Merge pull request #17532 from totten/5.27-queue-test

QueueTest - Be more forgiving about slow execution

Merge pull request #17510 from mattwire/paymentprocessorinstrumentnotrequired

Payment instrument ID is not required at processorform level

[Ref] Remove some instances of fatal in BAO classes

Remove unused deprecated class

Remove instances of fatal from upgrade folder

QueueTest - Be more forgiving about slow execution

The `CRM_Queue_QueueTest` is producing failures with the
[TIME_FUNC](https://github.com/civicrm/civicrm-core/pull/17414) option of
`linear:500` (the value currently used in PR test runs) -- which is to say,
it fails when the system runs very slowly (with some 500ms stalling during
execution).

This gets it working under some very slow scenarios:

```
for TIME_FUNC in natural frozen linear:500 linear:1250 prng:500 prng:666 prng:1000 prng:1500 ; do
  export TIME_FUNC;
  echo;
  echo "TIME_FUNC=$TIME_FUNC" ;
  env CIVICRM_UF=UnitTests phpunit6 tests/phpunit/CRM/Queue/QueueTest.php ;
done
```

Merge pull request #17530 from civicrm/5.27

5.27

Add icons to multi-valued custom groups and display on contact tab

Merge pull request #17527 from colemanw/nameAndIcon

Add titles and icons to entities

Merge pull request #17360 from totten/master-wp-var

WP - Change definitions of `cms.root`, `civicrm.root`

Merge pull request #17529 from totten/5.27-comm-msg

CommunityMessages - Define more consistent behavior in the face of slow execution

CommunityMessages - Define more consistent behavior in the face of slow execution

This passes under some brutally slow scenarios:

```
for TIME_FUNC in natural frozen linear:500 linear:1250 prng:500 prng:666 prng:1000 prng:1500 ; do
  export TIME_FUNC;
  echo;
  echo "TIME_FUNC=$TIME_FUNC" ;
  env CIVICRM_UF=UnitTests phpunit6 tests/phpunit/CRM/Core/CommunityMessagesTest.php ;
done
```