Merge pull request #17257 from colemanw/tagParam

Tag create: respect created_date and created_id params

Merge pull request #17268 from colemanw/getBAOClassName

Add getBAOClassName utiltiy function

Merge pull request #17269 from greenpeace-cee/simplify-length-test

CRM/Logging - Remove obsolete cache static clear in test

Merge pull request #17267 from eileenmcnaughton/bag

Add propertyBag handling for getEmail when incoming uses email-5

Merge pull request #17270 from totten/master-regtest

(NFC) RegionTest - Small cleanups

(NFC) RegionTest - Small cleanups

CRM/Logging - Remove obsolete cache static clear in test

This simplifies CRM_Logging_SchemaTest::testLengthChange() and
CRM_Logging_SchemaTest::testEnumChange() by removing a manual reset on
a cache static and a second call to
CRM_Logging_Schema::fixSchemaDifferences(). This was made obsolete by
a40a5a4875bf5caa03eaa8732c958f5c714c7b61, which has added a cache flush
to fixSchemaDifferences() via fixSchemaDifferencesForAll().

Add getBAOClassName utiltiy function

Add propertyBag handling for getEmail when incoming uses email-5

Calling code is often setting email-5. The property bag should iron out this inconsistency, and getEmail (and similar)
should handle the appended billing id. We can't do this in the static property map as it is a calculated variable

Merge pull request #17263 from civicrm/5.26

5.26

Merge pull request #17264 from colemanw/grantCleanup

[REF] Grant BAO cleanup

Merge pull request #17255 from agh1/sr-only

[NFC] Add sr-only class for content only for screen readers

Grant BAO cleanup

Tag create cleanup: respect created_date and created_id params

Merge pull request #17262 from totten/5.26-test-sql-op

SyntaxConformanceTest::testSqlOperators - Fix failure on MySQL 8

Merge pull request #17183 from eileenmcnaughton/tax_t

Remove incorrect retrievals of tax-term setting

SyntaxConformanceTest::testSqlOperators - Fix failure on MySQL 8

Before
------

`SyntaxConformanceTest::testSqlOperators` frequently fails on the `Dedupe` entity when running on MySQL 8 (`bknix-edge`).

After
------

`SyntaxConformanceTest::testSqlOperators` repeatedly passes for all entities on my copy of MySQL 8 (`bknix-edge`).

Technical Details
-----------------

The `testSqlOperators` creates a small pool of example records
(`$entities`), then it slices/dices that pool with a few SQL operators and
asserts the number of matches.

In particular:

```php
$this->callAPISuccessGetCount($entityName, ['id' => ['>' => $entities[0]]], $totalEntities - 1);
```

The problem is that `$entities[0]` does not necessarily have the lowest ID
-- because `$entities` was not fetched in any particular order.  The default
ordering is *often* the same as the `id`, but not always.

I suspect that this problem manifests most frequently with the `Dedupe`
entity because the underlying table (`civicrm_prevnext_cache`) is highly
volatile (many writes+deletes) -- thus it's more likely to reuse old storage
slots for new rows.

Set version to 5.27.alpha1

Set version to 5.26.beta1

Merge pull request #17227 from eileenmcnaughton/workflow_name

Add workflow_name column to civicrm_msg_template, deprecate workflow_id

Merge pull request #17258 from demeritcowboy/fix-getfromto-workaround

[NFC/Test] - Fix typo in `if` statement that seems intended as a workaround

fix typo in workaround

MessageTemplate API - Keep the workflow_id and workflow_name sync'd

[NFC] Add sr-only class for content only for screen readers

Merge pull request #17252 from civicrm/5.25

5.25

sql/civicrm_generated.mysql - Update with current message template schema

Add workflow_name column to civicrm_msg_template, deprecate workflow_id

Merge pull request #17251 from totten/master-hooktest

E2E_Core_HookTest - Fix test failure due to leak

Merge pull request #17249 from totten/5.25-backdrop-install

install/index.php - Fix leak which breaks compatibility with current Backdrop

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.

Update release-notes/5.25.0.md

install/index.php - Fix leak which breaks compatibility with current Backdrop

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.