Merge pull request #25457 from eileenmcnaughton/notice_cont_view

Move form specific code to the form

Merge pull request #25522 from eileenmcnaughton/dedupe

Make dedupe exceptions searchable in SearchKit

Merge pull request #25612 from eileenmcnaughton/ical_notice

E-notice fix Ical display

Merge pull request #25392 from eileenmcnaughton/prev_next_divide

dev/core#4112 Privatise `prevNextCache` functions

Merge pull request #25446 from lemniscus/oauth-reduce-dns-queries

Reduce unneeded DNS queries during OAuth flow

Merge pull request #25631 from eileenmcnaughton/poly

composer require --dev yoast/phpunit-polyfills

Merge pull request #25511 from eileenmcnaughton/dep_token6

Extra deprecation for clarity on token function `convertPseudoConstantsUsingMetadata`

Merge pull request #25630 from colemanw/phpQueryEncodeBug

Angular Coder: Fix unescaping of quotes breaking attributes

Merge pull request #25632 from colemanw/formatCustomValue

EntityRef - Format custom field display value on QuickForms

EntityRef - Format custom field display value on QuickForms

Angular Coder: Fix unescaping of quotes breaking attributes

Fixes the bug described in https://github.com/civicrm/civicrm-core/pull/25471#issuecomment-1423233785

Merge pull request #25471 from colemanw/entityReferenceField

Add new EntityReference custom field type

composer require --dev yoast/phpunit-polyfills

Merge pull request #25441 from MegaphoneJon/status-is-a-string

getStatus() should be returning a string

getStatus() should be returning a string

CustomFields - Add EntityReference field type based on APIv4 autocomplete

Merge pull request #25444 from michaelmcandrew/dev/core#4100

Add setting to disable Smarty in scheduled reminders

Merge pull request #25616 from jitendrapurohit/parse_address_fix

Regression: Fix DB syntax error on Parse address scheduled  job

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

Fix DB syntax error on Parse address sched job

Remove function deprecated a year ago

E-notice fix Ical display

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

Add setting to disable Smarty in scheduled reminders dev/core#4100

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()