fatal error when using pager

Merge pull request #16377 from ejegg/serviceFileFinder-5.22

loadServices: tighten up file match regex

Merge pull request #16384 from totten/5.22-nfc

(NFC) Civi, CRM, tests - Update for Drupal.Commenting.VariableComment.IncorrectVarType

[NFC] Fix var declarations in the test folder

Update Var delcartion to be standard in CRM folder

(NFC) Civi/API - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/CCase - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/Core - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/Payment - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/Test - Update for Drupal.Commenting.VariableComment.IncorrectVarType

loadServices: tighten up file match regex

Before: Civi code living under a path containing the characters php
(such as ~/src/php/civicrm) will crash trying to load services with
a message like the following:
   ReflectionException: Class Civi\Api4\Service\Spec\Provider\src
   does not exist in ReflectionClass->__construct()

After: Civi loads services normally no matter what the enclosing path

Merge pull request #16356 from totten/5.22-gencode-ts

(dev/core#1546) Fix translation of seed data

(NFC) LocalizedDataTest - Add tips on running

(dev/core#1546) Fix translation of seed data

Overview
--------

This fixes a regression in which seed data is not translated.

Before
------

The `civicrm_data.*.mysql` files are not translated.

After
-----

The `civicrm_data.*.mysql` files are translated.

There's a unit-test to check this.

Comments
--------

This regression stems from #15411, which aimed to allow extensions to define
custom variants of `ts()`.  The crux of the issue is "What happens if you
try to translate a string before the system is bootstrapped - before the
extension is loaded?  What's your fallback behavior?"

In #15411, it used a fallback behavior of "do no translation".  In theory,
you shouldn't really get into this scenario since UIs are pretty much always
generated post-boot.

However, it turns out that there is a situation where you have an un-booted
system and need to translate strings -- i.e. when generating the localized
`civicrm_data.*.mysql` data.  Hence the bug.

This patch preserves most of the changes from #15411, but it changes the
fallback behavior from "do no translation" to "use the built-in/default
translator".

Merge pull request #16313 from eileenmcnaughton/522

dev/core#1217 Fix for failure to respect event_id (or any?) criteria in smart groups

dev/core#1217 Fix for failure to respect event_id (or any?) criteria in smart groups

From what I can tell it's possible to create smart groups from Advanced search & other contact searchs & Membership Search & Event search.

Membership search actually just leverages advanced search but Event Search smart group has it's own code & reloads into Event search.

This has been flawed for months but the current iteration is the criteria are not respected. On digging I found the decision was made in
contact search to store the parsed QueryParams (in [field_name, 'IN', [1,2]] format not the formValues. I think this was probably a mistake.

However, by adjusting the event code to do the same & ensuring the defaults are set it starts working again

Merge pull request #16292 from eileenmcnaughton/522

dev/user-interface#13 fix regression where option to email is not longer displayed

dev/user-interface#13 fix regression where option to email is not longer displayed.

The api parameter here was removed, seemingly unintentionally. It is picked up on by the script when it checks for email so
needs re-instating

Merge pull request #16287 from MegaphoneJon/core-1524-rc

fix created_date/modified_date relative date filter searches

fix created_date/modified_date relative date filter searches

Merge pull request #16277 from seamuslee001/5.22

Add release notes for 5.21.1

Add release notes for 5.21.1

Merge pull request #16239 from MegaphoneJon/mailing-60-522

mailing#60: Fix regression where `is_bulkmail` flag isn't respected

mailing#60: Fix regression where multiple bulk recipient option isn't respected

test fixes

Merge pull request #16225 from demeritcowboy/contribution-view-missing-status

dev/core#1508 Regression/E_NOTICE - Missing table on contribution view

put back mistakenly removed variable

Merge pull request #16190 from alifrumin/rn5.22

5.22 Release Notes Annotating and Sorting

5.22 Release Notes Annotating and Sorting

Merge pull request #16187 from agh1/5.22.0-releasenotes

5.22.0 release notes - initial run

5.22.0 release notes: added boilerplate

5.22.0 release notes: raw from script

Set version to 5.22.beta1

Merge pull request #16182 from seamuslee001/dev_core_1144

dev/core#1144 (NFC) Handle fact that MySQL 8 always returns the colum…

dev/core#1144 (NFC) Handle fact that MySQL 8 always returns the column name as it is in the DB not as it is in the query

Merge pull request #16180 from eileenmcnaughton/act2

[REF] remove unused  from getACLRoles

Merge pull request #16181 from civicrm/5.21

5.21 to master

Merge pull request #16176 from eileenmcnaughton/mem_fix

dev/membership#21 fix regression on membership handling

[REF] remove unused  from getACLRoles

Following on from https://github.com/civicrm/civicrm-core/pull/16175 we now know that group_id is not
passed into this function as the only place that calls this no longer passes group id, eliminate.

Merge pull request #16167 from eileenmcnaughton/mem2

[NFC] Add test on buildMembershipTypeValues & cleanup class

Merge pull request #16175 from eileenmcnaughton/get_acls

[REF] Remove unused params from function signature for getACLs

Merge pull request #16179 from seamuslee001/dev_core_1143

dev/core#1143 dev/core#1480 Permit CiviCRM installation and running o…

New year (2020) test fixes. (Backport #16172)

We have 8 tests failing that seem to be due to 2020 starting. This fixes 2

Merge pull request #16178 from seamuslee001/regen_dao

[NFC] Re-generate DAOs

dev/core#1143 dev/core#1480 Permit CiviCRM installation and running on MySQL 8 by adding backticks around grouping

[NFC] Re-generate DAOs

dev/membership#21 fix regression on membership handling

This turns https://lab.civicrm.org/dev/membership/issues/21#note_29439 into a patch. As new code it seems to
have triggered an underlying issue. I also have https://github.com/civicrm/civicrm-core/pull/16139 open but
would be happy to see this hit the rc & the other have a full rc cycle.

The logic for this change makes sense as described & it is trivial. This has been a very hard bug
to reproduce so I'm unable to give full steps

Add test on buildMembershipTypeValues & cleanup class

I found the class wasn't running reliably in isolation so had to strip out some invalid things

[REF] Remove unused params from function signature for getACLs

It turns out this function is only called once & the second 2 acls are hard-coded so we don't need them

Merge pull request #16172 from eileenmcnaughton/happy_new_year

2020 test fixes.

Merge pull request #16100 from jitendrapurohit/dev-1473

dev/core#1473 - Missing address on /user when location type label dif…

Merge pull request #16162 from colemanw/contactFields

Add contact type to schema to allow GetFields filtering

2020 test fixes.

We have 8 tests failing that seem to be due to 2020 starting. This fixes 2

Merge pull request #16168 from colemanw/options

Pass api4 getFields values through to getOptions

Merge pull request #16171 from civicrm/5.21

Merge pull request #16170 from agh1/5.21.0-releasenotes-final

Merge pull request #16170 from agh1/5.21.0-releasenotes-final

5.21.0 releasenotes final

Merge pull request #16169 from agh1/5.21.0-releasenotes-final

Final 5.21.0 release notes

5.21.0 release notes: last batch of edits

5.21.0 release notes - updated big contributors list for 2019

5.21.0 release notes: added late changes

Pass api4 getFields values through to getOptions

Merge pull request #16165 from eileenmcnaughton/mod_member_api

[NFC] Extract duplicated code lines in test

[NFC] Extract duplicated code lines in test

Merge pull request #16164 from colemanw/controlField

Add controlField metadata for ChainSelect fields

Add controlField metadata for ChainSelect fields

Merge pull request #16163 from eileenmcnaughton/mod_member

[NFC] Clean up unit test

[NFC] Clean up unit test

Removes duplication (indluding one test entirely covered by other tests. Switches to using api create
and removes passing in of  as part of goal to eliminate this

Merge pull request #16149 from seamuslee001/dev_core_1489

dev/core#1489 do not reconcile managed entities whilst in upgrade mod…

Merge pull request #16117 from seamuslee001/dev_core_1424

dev/core#1424 Support MS Excel for Diacritic Characters when exporting a report to CSV

Add contact type to schema to allow GetFields filtering

Merge pull request #16154 from colemanw/getFields

Allow values to be passed to api4 GetFields

Merge pull request #16161 from civicrm/5.21

5.21

Merge pull request #16160 from seamuslee001/5.21

5.20.3 Release Notes

Update 5.20.3.md

Add release-notes/5.20.3.md

Merge pull request #16157 from seamuslee001/dev_drupal_99

dev/drupal#99 Fix E-notice on undefined property id of non object

Merge pull request #16156 from civicrm/5.21

5.21

Allow values to be passed to api4 GetFields

So far this only filters custom fields by contact type,
but values is now being passed to allow more filtering.

dev/drupal#99 Fix E-notice on undefined property id of non object

Merge pull request #16155 from seamuslee001/5.21

dev/core#183 Ensure that having a rule where each field is the same w…

dev/core#183 Ensure that having a rule where each field is the same weight with the threshold being the total sum of the weights does not cause a fatal error when searching for dupes

Merge pull request #16152 from colemanw/multi

Api4 - Fix auto serialize array input for CheckBox/MultiSelect fields

Merge pull request #15135 from mattwire/case_links_refactor_report

 Add case activity action links to activity report

Merge pull request #16079 from eileenmcnaughton/email

Remove unnecessary, and possibly incorrect query from email update

Merge pull request #16135 from eileenmcnaughton/pay_total

[REF] simplify recordAdjustedAmt function

Api4 - Fix auto serialize array input for CheckBox/MultiSelect fields

Merge pull request #16151 from eileenmcnaughton/seamus

[REF] extract member form batch update & add test

Merge pull request #16131 from eileenmcnaughton/acl

[Ref] improve readability of acl code

[REF] extract member form batch update & add test

Merge pull request #16112 from seamuslee001/remove_zend_validator

Remove unneeded zendframwork/zend-validator and zendframwork/zend-stdlib packages as not required by …

dev/core#1489 do not reconcile managed entities whilst in upgrade mode and add in a step at the end of the upgrade process to reconcile managed entiies and add in civicrm_permissions hook to the whitelist of upgrade friendly hooks

Merge pull request #16140 from eileenmcnaughton/part_name

dev/event#25 Fix regression on billing name overwrite

Merge pull request #16136 from eileenmcnaughton/member_test

[REF] remove CRM_Core_Error check

Merge pull request #16145 from eileenmcnaughton/index_api

Add api to check for missing indices

Merge pull request #16144 from eileenmcnaughton/acl_index

dev/core#1486 Add index to civicrm_acl_cache.modified_date

Merge pull request #16143 from eileenmcnaughton/deadlock_err2

Fix misleading error message on deadlock

Add api to check for missing indices

Add index to civicrm_acl.modified_date

Fix misleading error message on deadlock

We were seeing error messages like
API_Exception : financial_type_id is not valid : Donation
when a deadlock was hit. The error made not sense as 'Donation' DID exist and did not help us
to identify the issue as a deadlock. It's still helpful with a deadlock to know if
a real constraint violation is hit as per https://lab.civicrm.org/dev/core/issues/1481 sometimes
a disturbing amount of queries have been rolled back.

Let's fix to handle pseudoconstants here & give a useful message.
The previously converted params did not get to this point but we can re-do that work in this low
volume function rather than do a lot of re-thinking

Merge pull request #16142 from eileenmcnaughton/deadlock_err

[NFC] Declare exceptions, use single quotes where possible, use strict comparison

Merge pull request #16141 from eileenmcnaughton/fatal

dev/core#560 Remove instances of fatal errors