dev/translation#39 - Fix `core:install --lang`. Realign boot code.

Overview
--------

Suppose you run a command like:

```
cv core:install --lang=fr_FR --cms-base-url=http://example.com/
```

This fixes that use-case by realigning some of the bootstrap steps under `setup/plugins/installDatabase/*`.

Before
------

* The above `core:install` command crashes because the `paths` service is unavailable.

* The `civi.setup.installDatabase` begins with these two steps:

    1. `InstallSchema.civi-setup.php` manipulates some global options (to make certain services work),
       and it also installs the schema, and it also reboots the `Container`.
    2. `BootstrapCivi.civi-setup.php` tries to boot again - e.g. it loads `civicrm.settings.php` and
       calls `CRM_Core_Config::singleton($loadFromDB=TRUE)`. But this is suspect because it
       neglects to set `$force=TRUE`.

After
-----

* The above `core:install` command runs.

* The `civi.setup.installDatabase` begins with these *three* steps:

    1. `Preboot.civi-setup.php` starts the system in a *partially booted* mode (ie with `$loadFromDB==FALSE`).
    2. `InstallSchema.civi-setup.php` just installs the schema.
    3. `BootstrapCivi.civi-setup.php` switches the system to a fully booted mode (ie with `$loadFromDB==TRUE`).

Technical details
-----------------

I tested this on a D7 build. My dev/test loop was basically these steps:

```
cv core:uninstall -f
cv core:install --lang fr_FR --cms-base-url='http://dmaster.bknix:8001' -vvv
echo 'select name, label, value from civicrm_option_value where name like "Meet%"' | cv sql
```

If the install succeeded, then the SQL command would display localized
labels for "Meeting" ("Reunion").

Merge pull request #17236 from civicrm/5.25

5.25

Merge pull request #17232 from seamuslee001/dev_core_1742

dev/core#1742 Expose CiviCRM Database details for views configuration…

Merge pull request #17197 from eileenmcnaughton/act_r_test

[REF] [Test] IDE cleanup on ReportTemplateTest

Merge pull request #17234 from MegaphoneJon/reporting-20-alt

Reporting #20 - correct stats when force=1 on Contribution Detail Report

Merge pull request #16584 from eileenmcnaughton/role

[REF] Simplify non-creditcard participant.create

Merge pull request #17237 from eileenmcnaughton/525

(dev/core#1740) Cumulative contributions not being calculated with the correct criteria

(dev/core#1740) Cumulative contributions not being calculated with the correct criteria

Merge pull request #17026 from jitendrapurohit/dev-57

dev/core#57 On Behalf Of fails to populate in Email Receipt

Merge pull request #17235 from colemanw/apiLog

APIv4 - Deprecate option_value joins and display notices in Explorer

Merge pull request #17233 from alifrumin/rn5.25

[NFC] First Pass at 5.25 release notes

APIv4 - Deprecate option_value joins in favor of pseudoconstant suffix

API - Display all logged warnings, errors, etc. in debug output and APIv4 explorer.

Reporting #20 - correct stats when force=1 on Contribution Detail Report

Merge pull request #17231 from totten/master-token-tweak

(REF,NFC) TokenProcessor - Minor DX improvements

First Pass at 5.25 releasenotes

Merge pull request #17230 from seamuslee001/domain_token_thank_you_letter

[REF] Permit domain tokens being used within Thank You letters

[REF] Permit domain tokens being used within Thank You letters

Re-arrange code as per Matt's comment

dev/core#1742 Expose CiviCRM Database details for views configuration in Drupal 8 as well as in Drupal 7 and Backdrop

(NFC) TokenProcessor - Improve docblocks

(REF) TokenProcessor - Allowing adding rows by array

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

When composing a message, you need to add a row and then use the fluent
`TokenRow` stub, eg:

```php
$p->addRow()->context(['contact_id' => 123]);
$p->addRow()->context(['contact_id' => 456]);
```

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

When composing a message, you don't have to use the fluent `TokenRow` stub.
Just give an array:

```php
$p->addRow(['contact_id' => 123]);
$p->addRow(['contact_id' => 456]);
```

Or:

```php
$p->addRows([
  ['contact_id' => 123],
  ['contact_id' => 456],
]);
```

You still have the option of using the fluent interface.

(NFC) TokenRow - Improve docblocks

Merge pull request #17212 from eileenmcnaughton/fatal

Convert fatals to statusBounces in case forms

Merge pull request #17124 from totten/master-hook-cleanup

(REF) CRM_Utils_Hook - Remove deprecated formulations of `invoke(int,…)`

Merge pull request #17126 from totten/master-upgr-dispatch

dev/core#1460, dev/core#1713 - Categorical fix for upgrade<=>hook issues

Merge pull request #17006 from mattwire/l10nallowoverrideinherit

Allow override of inherited CMS language when in CiviCRM

Merge pull request #17130 from jitendrapurohit/mail-62

dev/mail#62 - Mailing Error when civicrm_mailing_group has duplicate …

Merge pull request #17091 from colemanw/noArrayValue

[REF] Switch CRM_Utils_Array::value to empty in conditionals

Merge pull request #17175 from eileenmcnaughton/batch

Further removal of long-deprecated skipCleanMoney

Merge pull request #17225 from eileenmcnaughton/cont

[REF] Minor code cleanup

Merge pull request #17219 from colemanw/api4deprecateOptions

APIv4 - Deprecate old way of retrieving activityType/optionValue ids

[REF] Minor code cleanup

Do not pass params as reference as unchanged

Merge pull request #17222 from eileenmcnaughton/obh

[REF] Minor code cleanup.

APIv4 - Deprecate old way of retrieving activityType/optionValue ids

Merge pull request #17224 from eileenmcnaughton/testc

[NFC] Test cleanup

Merge pull request #17223 from demeritcowboy/conference-slots

Fix admin page url for "conference slots" option values

[NFC] Test cleanup

fix admin page url

REF - Switch CRM_Utils_Array::value to empty in conditionals

Merge pull request #17221 from civicrm/5.25

5.25

[REF] Minor code cleanup.

Once we compress the 2 ifs we find the value in the 'else' is never used

Merge pull request #17218 from seamuslee001/5.25

[REF] Fix retrieving invoice related settings when doing complete tra…

Merge pull request #17216 from totten/master-disp-cleanup

dev/core#1460 - Small cleanups in CiviEventDispatcher{,Test}

Merge pull request #17217 from civicrm/5.25

5.25

[REF] Fix retreiving invoice related settings when doing complete transaction

Merge pull request #17215 from eileenmcnaughton/act

dev/core#1714 Fix mis-filtering on activity type

Merge pull request #17172 from colemanw/writeRecord

[REF] Reduce boilerplate code in BAO add/create functions

Merge pull request #17151 from eileenmcnaughton/actsched

[REF] Minor cleanup around action schedule code.

Merge pull request #17213 from eileenmcnaughton/test

[NFC][Test] Preliminary cleanup

(NFC) CiviEventDispatcher - Improve docblocks

Convert fatals to statusBounces in case forms

[REF] [Test] IDE cleanup on ReportTemplateTest

CiviEventDispatcherTest - Fix warning on newer versions of PHP

Merge pull request #17184 from mattwire/opencasenofatal

Cleanup and throw exceptions in OpenCase form

dev/core#1714 Fix mis-filtering on activity type

This fixes a bug where the activity_type_id was being compared against the label of the option value

Merge pull request #17211 from eileenmcnaughton/id

(NFC) Remove $Id$ artifacts from old SCM

Merge pull request #17122 from colemanw/buildOptions

Cleanup core pseudoconstant buildOptions

REF - Reduce boilerplate code in BAO add/create functions

The new CRM_Core_DAO::writeRecord function can be used instead of boilerplate

[NFC][Test] Preliminary cleanup

Margin cleanup - declare exceptions, use right quotes, remove unused params

Merge pull request #17167 from colemanw/moreOptions

APIv4 - Allow field options to be returned in multiple formats

Merge pull request #17196 from eileenmcnaughton/session

[REF] Cleanup a few instances where we use old call to get loggedInUserID

A bit less svn

APIv4 - Improve pseudoconstants in HAVING and Explorer

APIv4 - Allow field options to be returned in multiple formats

Merge pull request #17209 from civicrm/5.25

5.25

Merge pull request #17208 from seamuslee001/5.25

Update 5.24.6.md

Update 5.24.6.md

Merge pull request #17207 from civicrm/5.25

5.25

Merge pull request #17205 from colemanw/noMultiJoins

APIv4 - Don't advertise implicit multi-joins in Explorer

Merge pull request #17206 from seamuslee001/5.25

5.24.6 Release Notes

Update release-notes/5.24.6

Add release-notes/5.24.6.md

Merge pull request #17201 from civicrm/5.25

5.25

Merge pull request #17176 from eileenmcnaughton/batch1

Move batch-form support code back to the form

APIv4 - Don't advertise implicit multi-joins in Explorer

These joins are problematic and may be removed. The Explorer should not display them.

Merge pull request #17195 from eileenmcnaughton/act_report2

[REF] Simplify determination of enabled components

Merge pull request #17200 from eileenmcnaughton/rep

dev/core#1729 Fix filters skipping on activity report

dev/core#1729 Fix filters skipping onn activity report

Merge pull request #17190 from mattwire/contributionpageselectedchildfromactions

dev/core#1738 Fix opening specific contribution page tab from Manage Contribution Pages

Merge pull request #17199 from seamuslee001/new_master

5.25

merge in 5.25

Merge pull request #17188 from totten/5.25-contrib-virt-itemcanon

 Fix loading, saving, and upgrade of CiviContribute settings

Merge pull request #17194 from eileenmcnaughton/act_report

[REF] stop overriding postProcess function in activity report detail

[REF] Cleanup a few instances where we use old call to get loggedInUserID

[REF] stop overriding postProcess function in activity report detail

The function is the same as master except one extra thing that can be done in the function intended for
report-specific preProcess overrides

[REF] Simplify determination of enabled components

testHandlingOfContributionInvoiceSetting() - More realistic data+assertion

In practice, for the old setting `contribution_invoice_settings`, the
`invoicing` property could take one of two values:

1. If enabled, the value is an array with trueish subkey: `['invoicing'=>['invoicing'=>1]]`
2. If disabled, the value is a numeric `0`: `['invoicing'=>0]`

Following this means better compatibility.

The format is counter-intuitive, and the unit-test sets the value to enabled
using a more intuitive format (`['invoicing'=>1]`).  It's currently valid to
`set()` this way, but `get()`ting the value produces the
more-compatible/historical format.

dev/core#1724 - Backward-compatible handling of 'contribution_invoice_settings.invoicing'

In the old setting `contribution_invoice_settings`, the `invoicing`
property (aka "CiviContribute Component Settings" => "Enable Tax and
Invoicing") could take one of two values:

1. If enabled, it's an array with trueish subkey: `['invoicing'=>1]`
2. If disabled, it's a numeric zero.

This encoding is weird, and `contribution_invoice_settings` is generally deprecated
and has been replaced by a virtual rendition which needs to be consistent with the
old representation -- so that a couple-dozen things continue working.

For testing, it helped me to try these commands at different points:

```
cv ev 'return CRM_Invoicing_Utils::isInvoicingEnabled();'
cv api setting.get return=invoicing
cv api setting.get return=contribution_invoice_settings

select * from civicrm_setting where name like 'contrib%' or name like 'invoic%';
```

FiveTwentyFive - Convert settings during upgrade

Consider this use-case:

* Install 5.22
* Configure `CiviContribute Component Settings`
* Upgrade to 5.25.beta

The settings are lost during the upgrade.

Set version to 5.25.beta3

Merge pull request #17068 from colemanw/currentPath

[REF] Reconcile CRM_Utils_System::getUrlPath and CRM_Utils_System::currentPath

Fix opening specific contribution page tab from Manage Contribution Pages

Merge pull request #17125 from eileenmcnaughton/dupe1

Fix Dedupe entity_tag mangling bug

Merge pull request #17187 from alexymik/recur_contribution_source

Add contribution source to recurring contribution report

Merge pull request #17186 from demeritcowboy/gitlab-issue-template

Gitlab issue template - Ask for link to stackexchange or chat conversations

dev/core#1724 - Fix defaults for invoice-related settings

The default values were previously stored in 'contribution_invoice_settings', but
the updates are stored in individual settings, which makes it difficult to reconcile
the default values and explicit values.

With this revision, 'contribution_invoice_settings' is strictly virtual, and
all the other settings (like 'invoice_due_date') are canonical.

(REF) dev/core#1724 - SettingsBag - Convert to computeVirtual/updateVirtual

The `SettingsBag` uses the `$combined` property to (locally) cache a full
view of the active settings (based on combining different layers of data --
default-values, explicit-values, and mandatory-values). This design is
good for read-mostly data.

This patch changes the `__ContributionSettings` to be another layer in the
construction of `$combined` -- the virtual-values layer.  It fixes issues
wherein:

1. The virtual value is recomputed during every call to `get($key)`.

2. The virtual value is only based on explicit-values -- it disregards
   default-values and mandatory-values.

dev/core#1724 - SettingsBagTest - Add coverage for contribution_invoice_settings

Add contribution source to recurring contribution report

ask for link to external conversations