Merge pull request #25620 from mattwire/afformrelationship

Afform: Fix broken syntax for saving reciprocal relationships

Merge pull request #25596 from ixiam/devSK-ShowMoreButtonsAlways

Always show "add more buttons" dropdown in searchkit view

Merge pull request #25622 from braders/unnecessary-entity-property

[REF][PHP8.2] Remove unnecessary dynamic property

Merge pull request #25625 from braders/deprecated-self-callables

[REF][PHP8.2] Fix use of self in callables deprecation

Merge pull request #25626 from braders/groupcontacttest

[REF][PHP8.2] Tidy up properties on api_v3_GroupContactTest

Merge pull request #25623 from braders/event-permissions-test

[REF][PHP8.2] Tidy up properties in CRM_Event_BAO_EventPermissionsTest

Merge pull request #25624 from braders/remove_dedupeRuleGroupID

(REF) Remove test reference to property which no longer exists

Afform: Fix syntax for saving reciprocal relationships

[REF][PHP8.2] Tidy up properties on api_v3_GroupContactTest

[REF][PHP8.2] Fix use of self in callables deprecation

(REF) Remove test reference to property which no longer exists

[REF][PHP8.2] Tidy up properties in CRM_Event_BAO_EventPermissionsTest

[REF][PHP8.2] Remove unnecessary dynamic property

Merge pull request #25619 from eileenmcnaughton/php82_test

Php8.2 test fix, remove one instance of undeclared property

Php8.2 test fix, remove one instance on undeclared parameter

Merge pull request #25617 from eileenmcnaughton/matt_dep

Stop loading unused `relatedObjects`

Merge pull request #25610 from eileenmcnaughton/doc_block

[NFC] doc block fix

Unset ids for clarity

Stop loaded unused relatedObjects

[NFC] doc block fix

Merge pull request #25586 from herbdool/core-4127-6

dev/core#4127 move mailing workflow check to userSystem

Merge pull request #25615 from eileenmcnaughton/matt_dep

Remove function deprecated a year ago

Remove function deprecated a year ago

Merge pull request #25608 from eileenmcnaughton/test_fix

Fix test to test the thing it was written to test

Merge pull request #25607 from eileenmcnaughton/prop

[NFC] php8.2 support in test class `ActivitySearchTest`

Merge pull request #25599 from totten/5.59-patchset-wp

distmaker - Fix export of WordPress patches

Fix test to test the thing it was written to test

Per https://github.com/civicrm/civicrm-core/pull/23929 the point of the test was
to check that any override coming from parameters or the recurring contribution
record would be respected.

Also see
https://issues.civicrm.org/jira/browse/CRM-16397

However, the test did not explicity change the recurring contribution record amount
field, having been written after a hook was added that was changing the recurring amount
based on the received payment. So it was not testing it's goal - that where they differed
then, for single line item contributions, the recurring record took precedence.

This fixes to specifically alter them to be different, thus testing the override works
and does not rely on the hook (which is likely to change it's behaviour)

Merge pull request #25604 from eileenmcnaughton/parse_private

Make private function private

php8.2 support in test class ActivitySearchTest

Merge pull request #25550 from totten/master-abbr2

Disambiguate `Address.state_province_id:abbr`

Make private function private

Merge pull request #25570 from kurund/profile-api-fixes

update entity mapping logic for profile fields to include contact subtypes

Merge pull request #25445 from lemniscus/oauth-allow-client-to-set-redirect-uri

Allow OAuth redirect URI to be overridden in client or provider config

Merge pull request #25322 from braders/unused-params-contribution-soft-test

(REF) Tidy up unused params in api_v3_ContributionSoftTest

Disambiguate `Address.state_province_id:abbr` (MySQL; simpler version)

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 MySQL.  When loading abbreviations, Civi
queries with `ORDER BY abbreviation`.  This is a *typically* stable, but it
has no *guaranteed* resolution.  Adding a secondary sort key makes the
outcome clear/unambiguous.

Merge pull request #25595 from totten/master-api4-cast

APIv4 - Limited support for casting

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)

castTypeSoftly() - Support magic floats and magic ints

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

distmaker - Fix generation of WordPress patches

In `cividist`, the conventional name of the civicrm-wordpress folder is `$SRC/WordPress` not `$SRC/wordpress`.

(REF) Move test for APIv4 casting. Make it easier to mock different types.

Merge pull request #25568 from eileenmcnaughton/search_display_expires

Filter expired searches from search kit results

dev/core#4127 move mailing workflow check to userSystem

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

always show "add more buttons" dropdown in searchkit view

APIv4 - Limited support for casting

Before
------

* `setCheckPermissions(0)` casts the `0` to `false`.
     * This is because it's a concrete setter with explict typing.
* `setUseTrash(0)` does not.
     * This is because it's a magic setter with no typing.

After
-----

* Both `setCheckPermissions(0)` and `setUseTrash(0)` cast the `0` to `false`

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

I initially drafted in a way where `setUseTrash()` performed exactly the same casting
as `setCheckPermissions()`, but there was a countervailing test to assert that
`setDebug('debug')` is invalid, and that seemed fair.

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

update entity mapping logic for profile fields to include contact subtypes

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)