Merge pull request #25792 from braders/php82-imtest

[REF][PHP8.2] Refactor api_v3_ImTest with primary aim of improving PHP 8.2 compatiability

Merge pull request #25793 from braders/PriceFieldValueTest

[REF][PHP8.2] Refactor api_v3_PriceFieldValueTest with primary aim of improving PHP 8.2 compatiability

Merge pull request #25794 from braders/unused-onpopupclose

(REF) Remove unused onPopupClose variables

Merge pull request #25791 from braders/deprecated-self-callables-dedupe

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

Remove unused onPopupClose variables

[REF][PHP8.2] Refactor api_v3_PriceFieldValueTest with primary aim of improving PHP 8.2 compatiability

[REF][PHP8.2] Refactor api_v3_ImTest with primary aim of improving PHP 8.2 compatiability

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

Merge pull request #25786 from totten/master-setver-warning

set-version.php - Fix warning on PHP 8.1

Merge pull request #25788 from braders/errortest-php82

[REF][PHP8.2] Declare property on CRM_Core_ErrorTest

Merge pull request #25789 from braders/participantTest-dynamic-property

[REF][PHP8.2] Declare property on CRM_Event_Import_Parser_ParticipantTest

[REF][PHP8.2] Declare property on CRM_Event_Import_Parser_ParticipantTest

[REF][PHP8.2] Declare property on CRM_Core_ErrorTest

Merge pull request #25657 from eileenmcnaughton/dynamic

dev/core#4144 Fix smarty notice on isDuplicate

Merge pull request #16 from lemniscus/smarty-notices-profile-dynamic

fix notice in non-dialog mode too, make var name clearer, fix another notice

eliminate notice in non-dialog mode too, make var name clearer, fix another couple notices

set-version.php - Fix warning on PHP 8.1

The `$sqlFile` may or may not be needed, which produces a warning:

```
Deprecated: file_exists(): Passing null to parameter #1 ($filename) of type string is deprecated in
/.../tools/bin/scripts/set-version.php on line 105
```

Merge pull request #25781 from colemanw/deprecateConstants

APIv4 - Deprecate unnecessary constants

Merge pull request #25783 from seamuslee001/php82_mailing_contact

[NFC] Fix PHP8.2 issues in the Mailing Contact Test

[NFC] Fix PHP8.2 issues in the Mailing Contact Test

APIv4 - Deprecate unnecessary constants

Merge pull request #25774 from eileenmcnaughton/import_cache

Fix cache clearing when import table is changed

Fix cache clearing when import table is changed

Merge pull request #25527 from eileenmcnaughton/finder_build

Fix dedupe finder performance issue on looking up table size

Merge pull request #25712 from eileenmcnaughton/import_link

Import link - fix search Import SearchDisplays to link to the created contact

Merge pull request #25474 from eileenmcnaughton/civi-import

Hyphenate Civi-Import in the title/label

Civi-Import Add fk joinable to api spec

Merge pull request #25187 from totten/master-civiunit-more

CiviUnitTestCase - More aggressive simplifications

Merge pull request #25677 from colemanw/noisyDeprecate

BAO - Noisily deprecate and stop using deprecated functions

Merge pull request #25749 from eileenmcnaughton/retreive

[REF] Improve function signature for `retrieve()` in `PaypalProIPN` , add test for when trxn_id is present

BAO - Noisily deprecate and stop using deprecated functions

Merge pull request #25770 from colemanw/replaceWithMatch

APIv4 - Support `match` in replaceAction

Merge pull request #25768 from eileenmcnaughton/dep2

Remove function deprecated noisly in 2020

Merge pull request #25028 from MegaphoneJon/bad-pseudoconstants-api4

Don't crash API4 if pseudoconstant lookups return nothing

Merge pull request #25773 from colemanw/allYourBase

All your base are deleted

All your base are deleted

Merge pull request #25767 from eileenmcnaughton/dep

More deprecations of `del`

Merge pull request #25771 from eileenmcnaughton/datasource

Consolidate `DataSource.tpl`

Merge pull request #25772 from lemniscus/smarty-notice-merge

Smarty notice fix on Merge

Smarty notice fix on Merge

Consolidate DataSource.tpl

APIv4 - Support `match` in replaceAction

This allows the replaceAction to support non-id matching, the same way as the saveAction does.
Fixes dev/core#4168

Merge pull request #25763 from eileenmcnaughton/coleman

Deprecate pledge_payment::del

Merge pull request #25766 from eileenmcnaughton/coleman2

Remove deprecated calls from contribute, financial, remove ManagePremiums deprecated BAO

Remove function deprecated noisly in 2020

More deprecations

Remove deprecated calls from contribute, financial, remove ManagePremiums deprecated BAO

Deprecate pledge_payment::del

Merge pull request #25764 from eileenmcnaughton/coleman2

Deprecations on del functions

Merge pull request #25765 from eileenmcnaughton/coleman_rel

Deprecate relationship::del

Deprecate relationship::del

Deprecations on del functions

Merge pull request #25762 from colemanw/noisyDep

Noisily deprecate unused function

Merge pull request #25666 from kurund/file-validations-formbuilder

dev/core#4121 Afform: add validation for file fields in formbuilder

Noisily deprecate unused function

Merge pull request #25674 from colemanw/apiUpdateTest

APIv4 - Add 'Update' to conformanceTest

Merge pull request #25752 from eileenmcnaughton/cust_it

Remove remaining non-variable variables

Merge pull request #25761 from seamuslee001/regen_561

Regen after civicrm-core#24735

Regen after civicrm-core#24735

Merge pull request #24735 from agileware/CIVICRM-2052

🪥 Fix inconsistent terminology used when referring to Member Since, Membership Start Date and Membership Expiration Date

Merge pull request #25760 from civicrm/5.60

5.60 to master

Merge pull request #25758 from eileenmcnaughton/560

Load submitted `fieldSeparator` on back on `Import Datasource` screen

Merge pull request #25681 from agileware/CIVICRM-2103

dev/core/-/issues/4149 Change CKEditor 4 default to not encode HTML entities by default which breaks Smarty syntax and prevents use of crmDate function

Merge pull request #25757 from eileenmcnaughton/import_table

Code cleanup: Use `parent::buildForm` in contact import dataSource screen

Load submitted fieldSeparator on back on Import Datasource screen

Merge pull request #25751 from MegaphoneJon/relationship-cache-check

Relationship Cache check

Use parent::buildForm in contact import dataSource screen

Merge pull request #25700 from eileenmcnaughton/address

Change title of is_primary fields consistent with Email

Merge pull request #25750 from colemanw/ParticipantStatusType

APIv4 - Add ParticipantStatusType entity

Hyphenate Civi-Import in the name

dev/core#4144 Fix smarty notice on isDuplicate

Fix performance on deciding which tables to query

Merge pull request #25294 from colemanw/afformFileReupload

Afform - Show file field values for existing entities

Merge pull request #25756 from demeritcowboy/separator

revert 25735

Merge pull request #25755 from civicrm/5.60

5.60

revert 25735

Merge pull request #25753 from MegaphoneJon/two-field-separators-3

Remove the correct extraneous field separator field

Merge pull request #25719 from eileenmcnaughton/quick_config

Unconditionally assign `quickConfig`, add helper for `getPriceSetID`

Remove the correct extraneous field separator field

Relationship Cache check

Remove remaining non-variable variables

APIv4 - Add ParticipantStatusType entity

Merge pull request #25747 from eileenmcnaughton/c2

Stop using deprecated methods, `ufField::del`

Merge pull request #25746 from eileenmcnaughton/colemanw

Remove some deprecated del calls

Merge pull request #25745 from eileenmcnaughton/cust_it

Remove unvariable variables `toReturn` & `subName` from `Merger`

CiviUnitTestCase - Remove redundant reset

In CiviUnitTestCase, there are methods `setUpBeforeClass()` and `setUp()`.
Both methods reset the database unconditionally.

The general flow is like this:

  1: setUpBeforeClass();
  2: setUp(); testApple(); tearDown();
  3: setUp(); testBanana(); tearDown();
  4: setUp(); testCherry(); tearDown();
  5: tearDownAfterClass();

You might say: "Aha!  You need the call in `setUp`, because it runs more
frequently!" But not really - because `setUp()` has an extra guard
(`static::$dbInit`).  In effect, you perform DB-reset at step 1 -- and then
immediately again on step 2 -- and then never again (steps 3+4+5).

It is conceivable that a subclass might override these methods and inject
additional initialization steps -- in which case, this patch may affect
their behavior. We'll just have to run the test-suite to find out.

Final thought - If `static::$dbInit` were manipulated by other code, then
`setUp()` would be much more interesting.  But it's counterfactual -- grep
finds nothing that manipulates `dbInit`.

CiviUnitTestCase - Remove inert property `$DBResetRequired`

This property is assigned by ~30 tests, but it is never used.

CiviUnitTestCase - Display more accurate debug note

If you skimmed, you might say: It prints a console message when it resets
the DB.  That seems like useful information for someone who wants to
understand the overall behavior, so good.

The reality is different.  `setUpBeforeClass()` and `setUp()` both reset the
DB.  `setUpBeforeClass()` does it for every test-class, but it never says so.
`setUp()` does it for the first test-method (and then no others).

So when it showed the message "Installing database", it should have really said:

> I just installed the database a minute ago without telling you.
> Now I'm installing it a second time, and you should know that.
> In the future, I will install twenty more times, but you don't need to know that.

That's a silly thing to say. Better to say when it's happening for realz.

Fix retrieve function, add test for when trxn_id is present

Merge pull request #25748 from civicrm/5.60

5.60 to master

Merge pull request #25739 from colemanw/monthOptions

SearchKit - Add select options for MONTH date filter

Merge pull request #25743 from colemanw/fixFkEntity

APIv4 - Don't link to non-existent API entities

Merge pull request #25724 from eileenmcnaughton/crazy_complete

PayPal Prp IPN - Fix incorrect option_group update

Stop using deprecated methods, ufField

Remove some deprecated del calls

Remove unvariable variables toReturn & subName

These were established as unchanging here
https://github.com/civicrm/civicrm-core/pull/25676

Merge pull request #25676 from eileenmcnaughton/cust_it

Stop passing invariable variables to private function

SearchKit - Add select options for MONTH date filter

APIv4 - Don't link to non-existent API entities

Merge pull request #25740 from colemanw/nextBirthday

SearchKit - Add next_birthday calc field