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

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

(REF,NFC) TokenProcessor - Minor DX improvements

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

(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

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

Cleanup and throw exceptions in OpenCase form

Merge pull request #17182 from eileenmcnaughton/dep

Remove deprecated calls to optionValueQuery and no-longer-need param

Merge pull request #17177 from mattwire/showsendreceiptonrecurdetail

Show email receipt status on view recurring contribution

Remove deprecated calls to optionValueQuery and no-longer-need param

All calls to optionValueQuery that don't pass  have been deprecated for well over a year. We can remove them now,
and hence the handling of

dev/core#1724 Fix Changes to CiviContribute Component Settings not saved

It seems that, depending on the order in which settings are loaded, this is not always populated. Moving it here
addresses, although medium term we should address all references to it

Merge pull request #17127 from totten/master-dispatch-policy

dev/core#1460 - CiviEventDispatcher - Add policy options

Merge pull request #17129 from eileenmcnaughton/email2

Switch cc field on emailTrait to use entity reference

Display email receipt status on recur details

Convert view recurring contribution to use EntityPageTrait

Move batch-form support code back to the form

I've dug around onn this and there is no reason for this code to be in the BAO - it is only
relevant to support this form & should live there. Longer term the form should call
api v4....

Further removal of long-deprecated skipCleanMoney

Per https://github.com/civicrm/civicrm-core/commit/243320eda23195985815ceb65ec7ee9079ae3014
NOT setting this true (ie having the function clean up the money has been giving deprecation notices
for a year, it can go now

Merge pull request #17174 from pradpnayak/ruleTitle

show title on status message instead machine name

Merge pull request #16882 from wmortada/wp#46

dev/wp#46 - Remove styling that attempts to make inputs match select2

Merge pull request #17071 from eileenmcnaughton/payment

Refactor api3 Payment.Get API to support options + most fields in civicrm_financial_trxn

show title on status message instead machine name

Refactor api3 Payment.Get API to support options + most fields in civicrm_financial_trxn

Unit test for PR#16603

Alternative for fix ups on Payment.get