release-notes 5.26: annotating release notes

Merge pull request #17438 from eileenmcnaughton/cust

dev/core#1747 don't drop temp tables on class destruct.

Merge pull request #17457 from agh1/5.26.0-release-notes-initial

5.26.0 release notes initial run

5.26.0 release notes: added boilerplate

5.26.0 release notes: raw from script

dev/core#1747 don't drop temp tables on class destruct.

Recent code clean up means the search class is now dropped more quickly - meaning that dropping
these tables in the destructors is too aggressive. Since they are memory tables & temp tables they get
cleaned up anyway I think. We should move this search to an extension that ships with core, not necessarily
enabled on initial install but that is up for discussion

Merge pull request #17418 from eileenmcnaughton/trash

dev/core#1784 fix regression on restore contacts button not working.

dev/core#1784 fix regression on restore contacts button not working.

This should fix https://lab.civicrm.org/dev/core/-/issues/1784

I think the message from deleting multiple contacts is a bit wonky, but no worse

Merge pull request #17292 from eileenmcnaughton/prop

Fixes a regression when cancelling a recurring with no processor_id

Permit empty processorID

On deploying this code I'm seeing InvalidArgumentException: 'processorID field has max length of 255'

Merge pull request #17352 from christianwach/lab-wp-49-civi5-26

Remove "page=CiviCRM" query string from WordPress front-end (5.26)

Merge pull request #17365 from eileenmcnaughton/pdf

Revert "[REF] Code readability changes on activity tokens."

Revert "[REF] Code readability changes on activity tokens."

Per https://docs.civicrm.org/dev/en/latest/framework/token/#token-processor
The contactId is actually documented. While I've been removing those in various places in
favour of contactID or contact_id it seems Tim has been putting them in so
I think it's safer to revert this change at this stage

This reverts commit 2ebb452b239c28e7ed2c50da69d324a8d40a7e88.

Remove "page=CiviCRM" query string from WordPress front-end

Merge pull request #17334 from seamuslee001/fix_d8_install

[REF] Fix install on Drupal 8 using new setup code

[REF] Fix install on Drupal 8 using new setup code

Merge pull request #17261 from seamuslee001/dev_core_1681

dev/core#1681 Add in deprecation notice for Systems using MySQL versions before 5.7 and require 5.5 for install

Set minimum install MySQL version to be 5.5 and a recommended minimum of 5.7

Fix to use min install version

Add in deprecation notice for MySQL/MariaDB versions below 5.7

Merge pull request #17265 from MegaphoneJon/mime-error-rc

[regression] Fix error on no MIME type

fix regression on new MIME type library

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

SyntaxConformanceTest::testSqlOperators - Fix failure on MySQL 8

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.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

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