Merge pull request #25601 from eileenmcnaughton/import_constants

Remove unused constants left over from import cleanup

Merge pull request #25589 from lemniscus/api4-explorer-cv-syntax-short-boolean

api4 explorer: make boolean params work in cv (short syntax)

Merge pull request #25552 from totten/master-abbr-php

Disambiguate `Address.state_province_id:abbr` (PHP asort)

Remove unused constants left over from import cleanup

Merge pull request #25568 from eileenmcnaughton/search_display_expires

Filter expired searches from search kit results

Merge pull request #25569 from masetto/master

Fix empty values of Note on Import contribution

Merge pull request #25594 from eileenmcnaughton/dep

Deprecate unused `CRM_Core_BAO_UFField::copy()` function

Merge pull request #25572 from herbdool/core-4127-2

dev/core#4127 move ipAddress to userSystem

Deprecate unused CRM_Core_BAO_UFField::copy() function

Merge pull request #25593 from eileenmcnaughton/dao

Fix more schema checks to use mysql DATABASE() function, deprecate php function

dev/core#4127 move ipAddress to userSystem

Merge pull request #25592 from eileenmcnaughton/import_empty

Remove empty function

Merge pull request #25302 from eileenmcnaughton/smarty-notice

Fix Contact import e-notice on preview screen

Fix more schema checks to use mysql DATABASE() function, deprecate php function

Merge pull request #25571 from herbdool/core-4127-1

dev/core#4127 use userSystem method to check for Views

Remove empty function

Merge pull request #25591 from seamuslee001/master

[REF] Fix undefined variable notice in WordPress tests

[REF] Fix undefined variable notice in WordPress tests

Remove reference to  as not need

Merge pull request #25590 from civicrm/5.59

5.59

Merge branch '5.59' of github.com:civicrm/civicrm-core into 5.59

api4 explorer: make boolean params work in cv (short syntax)

Merge pull request #25584 from civicrm/5.59

5.59

Merge pull request #25583 from totten/5.59-tz-severity

Status Check - Tweak severity of timezone warning

Status Check - Tweak severity of timezone warning

Merge pull request #25582 from civicrm/5.59

5.59

Merge pull request #25578 from seamuslee001/dev_core_4119

dev/core#4119 Only generate error on blank confirm_title if confirmat…

Merge pull request #25581 from seamuslee001/master

5.59

Merge branch '5.59'

security/core#121 Add in hard coded list of file extensions that should be prohibited form being treated as safe no matter what

Expand the list of extensions as per Tim and allow for users to define their own list via a constant

dev/core#4119 Only generate error on blank confirm_title if confirmation page is enabled or the event is monetary

Merge pull request #25576 from totten/master-suppressed

CiviContribute - Fix warning about 'suppressedEmails' when generating PDF

Merge pull request #25547 from eileenmcnaughton/559

dev/core#4080 Fix wrong pdf format selection for invoice, when configured

CiviContribute - Fix warning about 'suppressedEmails' when generating PDF

Use-case:

* Open and execute "Find Contributions"
* Next to a contribution, choose "more > Send receipt"
* Make a PDF
* Open another tab and view any other page.
* There is warning:
    Notice: Undefined index: suppressedEmails in CRM_Contribute_Form_Task_PDF->postProcess()
    (line 139 of ..../web/sites/all/modules/civicrm/CRM/Contribute/Form/Task/PDF.php)."

Before:

* `$pdfElements['suppressedEmails']` is not always returned
*  $pdfElements['excludeContactIds'] is alsways returned

After:

* Both are always returned, even if empty

Merge pull request #25548 from eileenmcnaughton/559_token

(dev/core#4109) Fix tokens like `{contact.email_primary.email}`

dev/core#4127 use userSystem method to check for Views

Merge pull request #25383 from totten/master-ts-smarty

Smarty {ts} - For extensions, use fallback similar to E::ts() and JS ts()

fix empty values of Note on Import contribution

Merge pull request #25565 from eileenmcnaughton/act_links

Make `activity_type_id` available to `links` hook, test

Merge pull request #25517 from eileenmcnaughton/gettree

Make unshared `getTree` function private, remove never passed params

Filter expired searches from search kit results

Merge pull request #25567 from civicrm/5.59

5.59 to master

Make activity_type_id available to links, test

Add test to is_current on SavedSearch, fix

Merge pull request #25564 from eileenmcnaughton/activity_links

Preliminary cleanup, comments, type hints

Merge pull request #25555 from seamuslee001/dev_core_4116

dev/core#4116 Add in rebuilding log table schema when modifying schema

Preliminary cleanup, comments, type hints

Merge pull request #25329 from herbdool/core-4076

dev/core#4076 split up the theme method

Merge pull request #25554 from demeritcowboy/upgrade-crash

Fix fatal error on upgrade success screen

dev/core#4076 split up the theme method

Merge pull request #25560 from mlutfy/mailingSummaryPseudo

Mailing Summary Report: support pseudofields

Merge pull request #25562 from braders/properties-emailtest

[REF][PHP8.2] Remove use of dynamic properties in CRM_Contact_Form_Task_EmailTest

[REF][PHP8.2] Remove use of dynamic properties in CRM_Contact_Form_Task_EmailTest

Merge pull request #25561 from braders/dynamic-properties-authorize-net-test

[REF][PHP8.2] Get rid of dynamic properties in CRM_Core_Payment_AuthorizeNetTest

[REF][PHP8.2] Get rid of dynamic properties in CRM_Core_Payment_AuthorizeNetTest

Mailing Summary Report: support pseudofields

Merge pull request #25559 from braders/dynamic-property-additionalpaymenttest

[REF][PHP8.2] Stop use of dynamic property in AdditionalPaymentTest

Merge pull request #25558 from braders/dynmaic-property-ajaxtest

[REF][PHP8.2] Use variable instead of dynmaic property (CRM_Group_Page_AjaxTest)

Merge pull request #25557 from braders/pledge-block-test-property

[REF][PHP8.2] Declare property in CRM_Pledge_BAO_PledgeBlockTest

Merge pull request #25556 from braders/fake-object-php8.2

[REF][PHP8.2] Declare property in CRM_Utils_FakeObject

[REF][PHP8.2] Stop use of dynamic property in AdditionalPaymentTest

[REF][PHP8.2] Use variable instead of dynmaic property (CRM_Group_Page_AjaxTest)

[REF][PHP8.2] Declare property in CRM_Pledge_BAO_PledgeBlockTest

Merge pull request #25514 from eileenmcnaughton/smarty-search

A couple of minor smarty notice fixes

[REF][PHP8.2] Declare property in CRM_Utils_FakeObject

dev/core#4116 Add in rebuilding log table schema when modifying schema

Only rebuild if logging is enabled

Merge pull request #25551 from eileenmcnaughton/tokens

Upgrade test to use more recent methodology for testing forms

fix fatal error

Merge pull request #25544 from eileenmcnaughton/smarty_tpl

Smarty notice fix

(NFC) ContactJoinTest - Ensure that ambiguous state is matched to country

Before: There is one test using 'AK' -- in the US.

After: There are two tests using 'AK -- in the US and in Nigeria.

Disambiguate `Address.state_province_id:abbr` (PHP asort)

Consider `ContactJoinTest::testCreateWithPrimaryAndBilling` which writes the value:

  'address_billing.state_province_id:abbr' => 'AK',

The symbol 'AK' can map to three places: Akwa Ibom (Nigeria), Atakora
(Benin), and Alaska (USA).  This is an ambiguous choice.  It should be
resolved in a consistent way.

One flavor of ambiguity comes from PHP.  After fetching from MySQL, there is
a secondary sorting (via `CRM_Utils_Array::asort()` => `asort()`).  Per
https://www.php.net/asort, the outcome is quirky:

> If two members compare as equal, they retain their original order.  Prior
> to PHP 8.0.0, their relative order in the sorted array was undefined.

On PHP 7, you cannot resolve this with any of the standard sort methods
(`asort`, `usort`, `ksort`, etc). They all have the same issue.

However...  the docs for `\Collator::asort()` indicate no such issue.
Switching to `\Collator::asort()` causes the tests to pass on PHP 7.3.  And
we already use `\Collator::asort()`.

Upgrade test to use more recent methodology for testing forms

The tested class had been hacked to accept parameters to postProcess,
but we can now avoid that & it will likely be rejected in a later php
version / muddy the waters

dev/core#4109 Fix advertised tokens to work

A couple of minor smarty notice fixes

dev/core#4080 Fix wrong pdf format selection for invoice, when configured

To replicate this I

1) created a new pdf format at civicrm/admin/pdfFormats?reset=1 with a top margin
of about 6 metres
2) edited the invoice MessageTemplate to use this pdf format
3) enabled invoicing
4) generated a pdf from the user dashboard
5) lots of white space.....
6) reset the invoice MessageTemplate not to have a selected pdf
7) it looked ... normal

Merge pull request #25519 from eileenmcnaughton/gettree2

Remove always-NULL `$singleRecord` variable

Merge pull request #25546 from eileenmcnaughton/smarty_pledge

Enotice fix, pledge block on UserDashboard

Enotice fix, pledge block on UserDashboard

Merge pull request #25521 from colemanw/actionsMenu

SearchKit - Configurable action menu tasks per-search-display

Smarty notice fix

Merge pull request #25541 from eileenmcnaughton/null_obj

Do not use reference for Object parameter, death to `_NULLObject`

Fix notice, do not pass values that match defaults

Merge pull request #25540 from eileenmcnaughton/world

Remove invalid  'world_region' => 'India' from test

Do not use reference for Object parameter, deathto _NULlObject

Remove invalid  'world_region' => 'India' from test

SearchKit - Configurable action menu tasks per-search-display

Merge pull request #25537 from eileenmcnaughton/database

Fix a couple more places to use mysql DATABASE() function

Merge pull request #25538 from civicrm/5.59

5.59

Merge pull request #25539 from demeritcowboy/query

Typo executequery => executeQuery

typo

Merge pull request #25534 from totten/5.59-test-tz

CiviTestListener - Headless tests should initialize timezone

Merge pull request #25524 from eileenmcnaughton/finder_test

NFC cleanup in test class

Fix a couple more places to use mysql DATABASE() function

CiviTestListener - Headless tests should have initialized timezone

Before
------

* `CiviUnitTestCase` calls `setMySQLTimeZone()`
* Headless tests (which are initialized via `CiviTestListener`) do not call `setMySQLTimeZone()`
* If you have a test like 5.57's flavor of `QueueTest`, then the tests pass or fail depending on happenstance of server timezone configuration.

After
-----

* `CiviUnitTestCase` and `CiviTestListener` both call `setMySQLTimeZone()`
* If you have a test like 5.57's flavor of `QueueTest`, then the tests pass more consistently.

Comments
--------

This branch should work cleanly as PR for `5.59` (rc) or `master`.  I'd
rather like to use this in `5.57-esr` (so that tests run more cleanly).
So maybe `5.59` is the better target (but it doesn't strictly have to be).

Merge pull request #25530 from eileenmcnaughton/database

Use mysql DATABASE() function instead of php code

NFC cleanup in test class

Merge pull request #25531 from civicrm/5.59

5.59

Use mysql DATABASE() function instead of php code

Merge pull request #25528 from eileenmcnaughton/database

Use `DATABASE()` function rather than lossa code

Merge pull request #25525 from seamuslee001/group_filter_acl

dev/core#4068 Prevent adding in addSelectWhereClause for civicrm_group when filtering by a group as unneded for reports

dev/core#4068 Expand test coverage to include the contact summary report which is a non optimised report template