Merge pull request #17251 from totten/master-hooktest

E2E_Core_HookTest - Fix test failure due to leak

E2E_Core_HookTest - Fix test failure due to leak

Before
------

The `HookTest` class fails when running on WordPress.

The event-listener from `testSymfonyListener_names` leaks and remains active during `testSymfonyListener_int`.

After
-----

The `HookTest` class passes when running on WordPress.

The event-listeners from `testSymfonyListener_names` and `testSymfonyListener_int` are cleaned up.

Merge pull request #17248 from seamuslee001/master

5.25

Merge in 5.25

Merge pull request #17247 from totten/5.25-finalize

5.25.0 - Small finalizations

Set version to 5.25.0

release-notes/5.25.0.md - Small copy edits

Merge pull request #17243 from civicrm/5.25

5.25

Merge pull request #17238 from seamuslee001/dev_core_1739

dev/core#1739 Enable Longitude on Address editing by default (option 12)

Merge pull request #17242 from agh1/5.25.0-releasenotes-final

5.25.0 release notes final edits

5.25.0 release notes: final edits

Merge pull request #17241 from agh1/apiv4-access-civicrm-or-ajax-api

Api4 AJAX endpoint: allow either access CiviCRM or access AJAX API

Api4 AJAX endpoint: allow either access CiviCRM or access AJAX API

5.25.0 release notes: halfway through final edits

Merge pull request #17239 from colemanw/fixSchemaMap

APIv4 - Fix schema map to use canonical field names

5.25.0 release notes: added late changes

Merge pull request #17240 from totten/master-event-names

(REF) dev/core#1744 - Cleanup event naming

dev/core#1744 - Civi/API - Simplify event naming

dev/core#1744 - SystemInstallEvent - Simplify event naming

dev/core#1744 - ActionSchedule - Simplify event naming

dev/core#1744 - Civi/Token - Simplify event naming

Merge pull request #17236 from civicrm/5.25

5.25

dev/core#1563 - Fix schema map to use canonical field names

This fixes a bug in SchemaMapBuilder where it was accidentally using "unique names"
which do not map to the real names of db columns.

dev/core#1739 Enable Longitude on Address editing by default (option 12)

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