CRM_Core_ClassLoader - Fix loading with alternate packages path

CRM_Core_IDS - Fix loading with alternate packages path

CRM_Core_Smarty - Fix loading with alternate packages path

CRM_Core_Resources - Fix loading with alternate packages path

Api4 - add more debug output

CRM_Core_Menu - Fix warnings during local test run (dev/core#742)

Before
------

In certain local OSX+D8 test environments, when running the full suite, I got a large number of warnings like this:

```
Warning: simplexml_load_file(): I/O warning : failed to load external entity "/Users/totten/bknix/build/drupal8-clean/web/vendor/civicrm/civicrm-core//CRM/Campaign/xml/Menu/Campaign.xml" in /Users/totten/bknix/build/drupal8-clean/web/vendor/civicrm/civicrm-core/CRM/Core/Menu.php on line 99
    ```

After
-----

Those warnings seem to have gone away.

The menu still seems to populate correctly.

Comments
--------

This smells a lot like dev/core#742 -- e.g.  it's the same warning and
(after adjusting for changes in line-numbering over the months) same line of
code.

E2E_Core_LocalizedDataTest - Adaptation for composer/D8/git-style deployments

Civi\Test - Allow `headless()->apply()` (etc) to execute without setup.sh

Before
------

If you have a copy of the `git` codebase and wish to run any of the headless
test suites, you *must* run `setup.sh` aka `GenCode` beforehand.

This is because certain files - `sql/civicrm.mysql` and
`sql/civicrm_data.mysql` - are used in the headless suite.  As large
auto-generated files, they are not provided in git.

After
-----

The files `sql/civicrm.mysql` and `sql/civicrm_data.mysql` are no longer
required by the test suite -- the headless tests will directly use the
`GenCode` classes to produce the needful.

Comments
--------

The general thrust of this commit is to find spots which read an SQL file, e.g.

```php
\Civi\Test::execute(file_get_contents("foobar.mysql"));
```

and instead call the generator for that file, e.g.:

```php
$schema = new \CRM_Core_CodeGen_Schema(\Civi\Test::codeGen());
$result = $schema->generateFooBar();
\Civi\Test::execute($result['foobar.mysql']);
});
```

In doing so, we incorporate a couple tricks:

* The SQL content is cached for the duration of the test-run (`Civi\Test::$statics`).
  Generating the SQL is not super expensive... but `Civi\Test::headless()->...apply()`
  may be called thousands of times, so it could add-up. Just cache it.
* Most of the `generateFooBar()` functions depend in some fashion on `ts()`. This
  commit depends on a preceding commit to make `ts()` more amenable to execution
  during early stages of the test.
    * Related discussion: https://chat.civicrm.org/civicrm/pl/ehohytqkf7bd5prg9w75dq4qqw

(REF) Civi\Test - Use de-minimis ts() while runnning installation

When running early-stage installation helpers like
`Civi\Test::headless()->...->apply()` or `Civi\Test:data()->...->populate()`
it can be hard to call `CRM_CodeGen_*` functionality - because that relies
on `ts()` which relies on other system services.

This change basically suspends/swaps some of the default `ts()` behavior
while the test system performs installation.

The change is in two sub-parts:

* Implement `asPreInstall()` (Files: `Civi/Test.php` and `CRM/Core/I18n.php`)
    * Add the helper `\Civi\Test::asPreInstall()`
    * Skip the use of "word replacements" during the pre-install phase
* Use `asPreInstall()` (Files: `Civi/Test/Data.php` and `Civi/Test/CiviEnvBuilder.php`)
    * Wrap existing code with the helper `\Civi\Test::asPreInstall(). These changes look big, but that's
      just whitespace -- inserting `asPreInstall()` changes the indentation.

This lays groundwork for another commit which will replace some references to `sql/civicrm*.mysql`
with references to `CRM_CodeGen_*`.

(REF) CRM_Core_CodeGen_Schema - Allow generating SQL without making files

Overview
--------

This is a refactoring to allow consumers of `CRM_Core_CodeGen_Schema` to
request the SQL without creating files.

Before
------

Calls to `CRM_Core_CodeGen_Schema::generateDropSql` (ad nauseum) create SQL files.

After
-----

Calls to `CRM_Core_CodeGen_Schema::generateDropSql` (ad nauseum) return SQL strings.

Comments
--------

To ensure that files are still generated consistently, you can use this procedure:

```
git checkout MY_BASE_BRANCH
./bin/setup.sh -g
cp -r sql sql.orig

git checkout MY_MODIFIED_BRANCH

rm sql/civicrm.mysql sql/civicrm_acl*mysql sql/civicrm_data*mysql sql/civicrm_drop.mysql sql/civicrm_navigation.mysql -f
./bin/setup.sh -g

colordiff -ru sql.orig sql
```

Make $civicrm_paths less sensitive to trailing slashes. Add tests.

Overview
--------

Recall that the default value of `imageUploadURL` is `[civicrm.files]/persist/contribute/`. Now, suppose you have one of these two configurations in `civicrm.settings.php`:

```php
// (1) Trailing slash configuration
$civicrm_paths['civicrm.files']['url'] = 'http://tmpd8-clean.bknix:8001/sites/default/files/civicrm/';
$civicrm_paths['civicrm.files']['path'] = '/Users/totten/bknix/build/tmpd8-clean/web/sites/default/files/civicrm/';

// (2) Trimmed slash configuration
$civicrm_paths['civicrm.files']['url'] = 'http://tmpd8-clean.bknix:8001/sites/default/files/civicrm';
$civicrm_paths['civicrm.files']['path'] = '/Users/totten/bknix/build/tmpd8-clean/web/sites/default/files/civicrm';
```

You could inspect to see if the URL is generated correctly by running these commands:

```
$ cv api setting.get return=imageUploadURL
$ cv url -c imageUploadURL
$ cv url -d "[civicrm.files]/persist/contribute/"
$ cv path -c imageUploadDir
$ cv path -d "[civicrm.files]/persist/contribute/"
```

Before
------

Under either configuration (trailing-slash or trimmed-slash), you'll find that paths are generated properly (`cv path ...` or `Civi::paths()->getPath(...)`).

For generating URLs (`cv url ...` or `Civi::paths()->getUrl(...)`), only the trailing-slash cfg works.  The trimmed-slash cfg leads to a bad URL with a missing slash.

```
[bknix-max:~/bknix/build/tmpd8prj-clean/web] cv api setting.get return=imageUploadURL
{
    "is_error": 0,
    "version": 3,
    "count": 1,
    "id": 1,
    "values": {
        "1": {
            "imageUploadURL": "[civicrm.files]/persist/contribute/"
        }
    }
}

[bknix-max:~/bknix/build/tmpd8prj-clean/web] cv url -c imageUploadURL
"http://tmpd8prj-clean.bknix:8001/sites/default/files/civicrmpersist/contribute"

[bknix-max:~/bknix/build/tmpd8prj-clean/web] cv url -d "[civicrm.files]/persist/contribute/"
"http://tmpd8prj-clean.bknix:8001/sites/default/files/civicrmpersist/contribute/"
```

This is surprising because the path expression (`[civicrm.files]/persist/contribute`) includes a slash...  but it disappears in the final computation.

After
-----

Both configurations work, for paths and for URLs.

Add E2E_Core_PathUrlTest to ensure that various path/URL lookups work

This will allow E2E matrix to send a signal about whether the default
configurations produce valid URLs.

ThemesTest - Use more realistic expected-values

1. The expected value of `$civicrmBaseUrl` should not be blank. The old value was a fiction of
   the unit-test environment. Be more realistic.
2. The expected values for extension-generated URLs should abide the same `$civicrmBaseUrl` as others.

Allow most values of $civicrm_paths['XXX']['url'] to be relative

Overview
--------

The `$civicrm_paths` variable allows a sysadmin to override various path and
URL computations.

```php
$civicrm_paths['civicrm.packages']['url'] = 'https://example.com/libraries/civicrm/packages';
```

The variable was originally tested with absolute URLs, and the subsequent
examples/docs use absolute URLs (https://docs.civicrm.org/dev/en/latest/framework/filesystem/).

These values are used to generate addresses, as in:

```php
$abs = Civi::paths()->getUrl('[civicrm.packages]/foo.js', 'absolute');
$rel = Civi::paths()->getUrl('[civicrm.packages]/foo.js', 'relative');
```

The patch allows more values in `$civicrm_paths` while ensuring that
`getUrl()` works as expected.

Before
------

The `getUrl()` requests only behave correctly if the override is an absolute URL - not if it's relative.

After
-----

The `getUrl()` requests behave correctly if the override is either an absolute URL or a relative URL.

```php
$civicrm_paths['civicrm.packages']['url'] = 'https://example.com/libraries/civicrm/packages';
$civicrm_paths['civicrm.packages']['url'] = '/libraries/civicrm/packages';
```

Comments
--------

* `toAbsoluteUrl()` needs a base to prepend. I initially used `HTTP_HOST`
  but switched to `cms.root`, but correctly inferring scheme and host and port
  and httpd prefixes would be more complex - esp for background/CLI jobs.
  Using `cms.root` as the base is simpler.
* It's tempting to allow recursive variables. But it's not actually needed for
  my purposes, and it would add complexity/maintenance. If it's really needed,
  one could update `toAbsoluteUrl()` to quickly check for variables
  (`$url[0] === '['`) and then evaluate them. But for now... I think the
  simpler format is fine.

Merge pull request #16388 from eileenmcnaughton/event_ref

[REF] Extract & share code to determine revenue recognition date.

fatal error when using pager

Api4 - Improve docblocks and their display in the Explorer

Improve code docs for Api4 Entity entity

APIv4 - Accept multiple fieldNames in Get helper fn

Merge pull request #16390 from artfulrobot/tame-propbag-deprecation-logs

Prevent PropertBag from being so noisy about deprecation warnings

[REF] Extract & share code to determine revenue recognition date.

This also starts the process of consolidating building the 'contribution params' array

Merge pull request #16395 from eileenmcnaughton/cont_setting

Finish removal of deferred_revenue_enabled from non-standard setting

Merge pull request #16271 from mattwire/pcp_extractsupportertext

REF Extract code to build pcp_supporter_text and enable translation

Merge pull request #16396 from eileenmcnaughton/cont_setting2

[REF] Remove unused variables

Merge pull request #16394 from demeritcowboy/extdir-links

dev/core#1550 Invalid links to extension directory in popup help text

[REF] Remove deprecated code from CRM_Admin_Form_Preferences

This form was cleaned up back in 2018 but all the stuff to handle ->_varNames was deprecated rather than
removed in case extensions were leveraging it in some way. I'm removing it now although I commented rather
than removed a few lines that I suspect can also go

[REF] Remove unused variables

Finish removal of deferred_revenue_enabled from non-standard setting

Merge pull request #16372 from colemanw/api4Magic

Api4 - Use explicit adder functions rather than magicMethod

invalid links

dev/core#1319 UI - question mark on the same line as the checkbox

dev/core#1319 obvious mistakes after testing

dev/core#1319 generalize the creation of relationship when creating an shared address to make it work with household

issue 1522

Prevent PropertBag from being so noisy about deprecation warnings for #16390

Merge pull request #16389 from eileenmcnaughton/ev_static

Fix tests to use buildFeeFn

Merge pull request #15408 from ejegg/l10n

dev/translation#30 Move l10n resource dir under private files

(dev/translation#30) Undo new MagicMerge::$l10nDir property

The previous commit added a new property `l10nDir` to `MagicMerge`.

The property doesn't seem to be needed, so omit it.

Merge pull request #16352 from mlutfy/translation34

dev/translation#34 Fix Contact Reference option for Postal Code

Merge pull request #16367 from MegaphoneJon/event-32

event#32: Fix bad variable in message template

dev/translation#30 Move l10n resource dir under private files

Looks under [civicrm.private]/l10n if that exists, falling back to
[civicrm.root]/l10n.

Since I18n::getResourceDirectory needs to be called from GenCode,
where we don't have the base directory defined, we make the l10n
path factory function consult new #define CIVICRM_L10N_BASEDIR first.

In GenCode itself, keep using the old location for now, but make
it possible to change the location with an env var as soon as we
update buildkit.

Assert on fail

Fix tests to use buildFeeFn

Merge pull request #16348 from eileenmcnaughton/ev_online

Remove irrelevant code.

Merge pull request #16375 from seamuslee001/dev_core_1538

Update Export unit test to pass on MySQL 8

Merge pull request #16387 from demeritcowboy/blank-google-recaptcha

dev/translation#33 Fix incorrect quoting in a link and an incorrect link

Merge pull request #15912 from mydropwizard/d8-language-empty-prefix

dev/drupal#85 Drupal8: Fix bug with empty language prefix mangling https:// to http:/

fix invalid quoting and incorrect link

Merge pull request #16386 from civicrm/5.22

5.22

Merge pull request #16377 from ejegg/serviceFileFinder-5.22

loadServices: tighten up file match regex

Merge pull request #16385 from civicrm/5.22

5.22

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

Merge pull request #16382 from seamuslee001/crm_var_update

(NFC) Update Var declaration to be standard in CRM folder

Update Var delcartion to be standard in CRM folder

Merge pull request #16355 from eileenmcnaughton/ev_part

Add helper for getting participantValues

Merge pull request #16379 from totten/master-coder-ivt

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

(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

Merge pull request #16373 from colemanw/api4Stuff

Api4 - improve docblocks & display links in explorer

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

Update Export unit test to pass on MySQL 8

Merge remote-tracking branch 'origin/5.22' into 5.22-master-2020-01-24-13-18-00

Merge pull request #16370 from mattwire/membershipstatuscalc_params

NFC Add parameter definition to membershipstatus.calc

Merge pull request #16335 from colemanw/exportui

Move exportui extension into core

Merge pull request #16371 from yashodha/typo

Remove unused code

dev/translation#34 Fix Contact Reference option for Postal Code

Add @see annoations to api4 docblocks and show as links in api explorer

event#32: Fix bad URL in message template

Api4 - Use explicit adder functions rather than magicMethod

Api Actions had been relying on a magic method to provide getter/setter functions for params.
That works fine for simple get/set operations but it was also attempting to provide adder functions.
Those are more nuanced and the magic method didn't do a good job of understanding whether the param
was an indexed or unindexed array, and didn't do strict type checking.
Making our own adder functions gives better documentation, stricter checking of inputs and also
the convenience of variadic functions for adding several values at once.

Better looking checkboxes in api explorer

Merge pull request #16363 from eileenmcnaughton/deextract

Duplicate code (for real)

remove unused code

Add parameter definition to membershipstatus.calc

Merge pull request #16369 from eileenmcnaughton/export_house

dev/core#1364 Merge all addresses on export should INCLUDE merging households

Merge pull request #16129 from mattwire/gettotalamount_0

Make getTotalPayments return 0 instead of NULL

Merge pull request #16364 from eileenmcnaughton/export

[REF] Extract function to export header row

Merge pull request #16368 from yashodha/typo

fix more typo's in the text

dev/core#1364 Merge all addresses on export should INCLUDE merging households

This is an alternate to https://github.com/civicrm/civicrm-core/pull/15725 that still achieves the goal in
https://github.com/civicrm/civicrm-core/pull/15725#issuecomment-572408173 of making is so that the
mergeSharedAddress option mergesHouseholds first & then merges sharedAddresses.

In #15725 @monishdeb was fixing the code that differentiated between how master_id was treated
versus otherwise merged addresses. However that code 'broke' a long time ago and the arguably
broken behaviour is locked in by unit tests. Fixing it has not been requested - in this
issue or any other in the past year or 2 when it has been broken so I opted to remove the broken
code entirely & to open up the next query such that it does not exclude household addresses.
This didn't break any tests (woot) and also allowed me to enable the test rows @monishdeb wrote
to test this issue

fix more typo's in the text

Merge pull request #16358 from MegaphoneJon/event-30

event#30 - don't allow multiple waitlist registrations

event#30 - don't allow multiple waitlist registrations

Extract function to export header row

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

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

Duplicate code (for real)

This is going to seem unintuitive but I'm sure the path to cleaning up the participant form starts with
increasing code duplication.

Currently the code is sharing 2 functions with the front end form. The functions don't do a lot of actual processing
but they do do a lot of preparing parameters. The calling functions do a lot of pre-work to call them - so
we are not getting much benefit from the sharing. On the other hand having this code tied up with another form makes
it hard for us to do the 'right' sharing - ie. currently the code to create a contribution paid by credit card & participant record is totally different to another payment method & we should work to share these bits of functionality

Return 0 if total amount is 0 instead of NULL.

Merge pull request #16353 from pradpnayak/functionFix

Wrong parameter passed to executeQuery function

Merge pull request #16311 from MegaphoneJon/event-template-reminders

event#28: Don't send scheduled reminders for event templates

Don't send scheduled reminders for event templates

Merge pull request #16361 from yashodha/typo

fix typo's in the text

Merge pull request #16189 from eileenmcnaughton/acl

Update BAO_Acl internal functions to protected & unused to deprecated