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

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

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)

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