Check presence of field, rather than extra smarty variable for whether to display the field

Merge pull request #25951 from civicrm/5.60

5.60 to master

Merge pull request #25950 from demeritcowboy/upgrade-fk

Make 5.60 upgrade rerunnable

make upgrade rerunnable

Merge pull request #25946 from totten/master-phpunit9

(dev/core#4188) phpunit.xml.dist - Make file compatible with phpunit8 + phpunit9

Merge pull request #25949 from totten/master-phpunit9-crm

(dev/core#4188) Update CRM tests for phpunit9 compatibility

Merge pull request #25948 from totten/master-phpunit9-api3

(dev/core#4188) Update APIv3 tests for phpunit9 compatibility

Merge pull request #25947 from totten/master-phpunit9-api4

(dev/core#4188) Update APIv4 tests for phpunit9 compatibility

(dev/core#4188) Update CRM tests for phpunit9 compatibility

(dev/core#4188) Update APIv3 tests for phpunit9 compatibility

These tests failed on `phpunit9` because upstream swapped out around the semantics of `assertContains()`.

The specific issue is whether an array like `["1","2","3"]` contains the value `2`.

* `assertContains()` in phpunit8 says yes.
* `assertContains()` in phpunit9 says no.
* `assertContainsEquals()` in both phpunit8+phpunit9 says yes.

(dev/core#4188) Update APIv4 tests for phpunit9 compatibility

These tests failed on `phpunit9` because upstream swapped out around the semantics of `assertContains()`.

The specific issue is whether an array like `["1","2","3"]` contains the value `2`.

* `assertContains()` in phpunit8 says yes.
* `assertContains()` in phpunit9 says no.
* `assertContainsEquals()` in both phpunit8+phpunit9 says yes.

phpunit.xml.dist - Make file compatible with phpunit8 + phpunit9

Merge pull request #25873 from colemanw/ppUpdate

PaymentProcessor - Enter title instead of name on form, call hooks

PHP 8.2 - declare property _fields

Really we want to merge this into entityFields but it wasn't as easy as I hoped

Fix incorrect redirect when saving with bad config

Since the config is checked after saving we want to go to the
edit form, not the new form

Fix form to stop messing name field for test processor

Consolidate loading parameters from PaymentProcessorType

Present frontend_title for Payment processor

PaymentProecssor - Add frontend_title & make title required

Clarifies the role of 'title' as "for admins" and adds a new "frontend_title" for end-users.

Updates the PaymentProcessor form/dao to use hooks instead of an overloaded "create" function,
and only allows the title and frontend_title instead of the name to be entered on the form.

The name will be autogenerated from the title in the DAO::writeRecord function.
The frontend_title is required in the database but will be filled from `title` if not supplied during `create`

Updates tests, sql and sample data with required field

Merge pull request #25927 from colemanw/entityRefFix

SearchKit - Fix handling of new Custom EntityReference fields

Merge pull request #25937 from colemanw/emailWriteRecord

Refactor CRM_Core_BAO_Email to use DAO::writeRecord

Merge pull request #25933 from mlutfy/logReminders

Log sending of Scheduled Reminders

Merge pull request #25941 from eileenmcnaughton/nfc_null

[NFC] Remove extraneous NULL

Merge pull request #25943 from totten/master-mfw

Merge forward 5.60-rc=>master

Merge remote-tracking branch 'origin/5.60' into master-mfw

Merge pull request #25942 from totten/5.60-cache-test

Simplify steps of loading SimpleCacheTest from cache/integration-tests

Merge pull request #25940 from eileenmcnaughton/upgrade_bug

Fix bug in just-merged upgrade code

[NFC] Remove extraneous NULL

It confuses code analysers & Coleman says it's silly

(NFC) SimpleCacheTest - Adapt license

The copied declaration references a file that doesn't exist. Just inline it.

remove dependence on cache/integration-tests

Fix bug in just-merged upgrade code

Undeclare cache/integration-tests

Refactor CRM_Core_BAO_Email to use DAO::writeRecord

Merge pull request #25808 from eileenmcnaughton/import_template

dev/core#4130 add template support for imports

Merge pull request #25936 from civicrm/5.60

5.60 to master

Merge pull request #25932 from ufundo/api-event-create-from-template

Fix for creating new events from template with api

Merge pull request #25879 from eileenmcnaughton/import_hooks

dev/core#3514 Define hook to alter data once it has been mapped but before work is done on it.

Merge pull request #25886 from eileenmcnaughton/import_contribution_api4

Update Contribution Import to use apiv4 field names, prior to adding hooks

Cast metadata to string, as it would hit deprecation notice on null

Add url to UserJob data & declare import Type on MapField screens

The DataSource screen currently declares the type so this makes it available on MapField

Minor tidy up/ standardisation on Contact defaultValues

This also allows the parent to override the values, e.g if loaded from a template

Fix BAO to support Template UserJobs

- do not save non-template fields
- delete templates on Mapping deletes

Add support for UserJob templates at the form level

Fix for strict typing

fix creating new event from template

make test for creating event from template a bit more difficult

Merge pull request #25934 from colemanw/usageGen

Regenerate DAOs with usage from #25874

Regenerate DAOs with usage from #25874

Log sending of Scheduled Reminders

They are a pain to debug when they fail because of errors with smarty or tokens

Merge pull request #25408 from artfulrobot/artfulrobot-speedup-searchkit

Speed up SearchKit results loading by 800ms

Merge pull request #25754 from eileenmcnaughton/quick_config

Extract `isMembershipPriceSet` (`useForMember`)

Merge pull request #25929 from eileenmcnaughton/mem_test_nfc

Add unit test cover for test offline mail receipt

Merge pull request #25912 from eileenmcnaughton/560

dev/core#4197 Ensure parameters are correct type

Merge pull request #25907 from eileenmcnaughton/url

Fix import contribution url

Merge pull request #25833 from eileenmcnaughton/mid

[php8.2] Create getter for `renewalMembershipID`, remove if

Merge pull request #25454 from eileenmcnaughton/grant_notice

CiviReport - Remove duplicate external identifier key in contact fields array, reduce notices

Add unit test cover for test offline mail receipt

Merge pull request #25928 from eileenmcnaughton/mem_test_nfc

[NFC] Test cleanup, prefer full form test flow

[NFC] Test cleanup, prefer full form test flow

APIv4 - Prevent errors during upgrade when loading custom field metadata

Merge pull request #25924 from eileenmcnaughton/mem_test

Fix intermittant test fail

SearchKit - Fix handing of new EntityReference custom fields

Fixes the metadata for the new EntityReference custom fields to correctly display.

Removes redundant function SpecFormatter::customFieldHasOptions which was completely
identical to CRM_Core_BAO_CustomField::hasOptions except for one line which was incorrect.

EntityRef fields don't get returned and can't be joined in API4

Merge pull request #25926 from braders/entity-form-trait-properties

[REF][PHP8.2] Declare properties directly on CRM_Core_Form_EntityFormTrait

Merge pull request #25925 from braders/membership-test-variable

[REF][PHP8.2] Convert dynamic property into a variable

[REF][PHP8.2] Declare properties directly on CRM_Core_Form_EntityFormTrait

[REF][PHP8.2] Convert dynamic property into a variable

Merge pull request #25923 from eileenmcnaughton/mem_offline

Stop passing `$this` as `$form` to previously shared, non-static, function

Fix intermittant test fail

Stop passing this as form to previously shared, non-static, function

Merge pull request #25918 from mlutfy/core4204

dev/core#4204 Show the Payment Agreement of the selected processor

Merge pull request #25921 from braders/unused-ufmatchid

[REF][PHP8.2] Remove unused dyanamic property

[REF][PHP8.2] Remove unused dyanamic property

dev/core#4204 Show the Payment Agreement of the selected processor

Merge pull request #25916 from civicrm/5.60

5.60 to master

Merge pull request #25910 from eileenmcnaughton/regression_dd

dev/core#4189 Fix indiscriminate display of direct debit agreement

Merge pull request #25900 from eileenmcnaughton/import_contact_custom_bool

dev/core#3937 Remove legacy code causing custom boolean import bug

Merge pull request #25913 from mlutfy/flushCacheButton

UpdateConfigBackend: use api4 to flush the cache

UpdateConfigBackend: use api4 to flush the cache

Merge pull request #25780 from yashodha/dev-4176

(dev/core#4176) Allow to search on participant id

(dev/core#4176) Allow to search on participant id

dev/core#4197 Ensure parameters are correct type

Merge pull request #25876 from eileenmcnaughton/custom_number

dev/core#4152 Fix custom data code to not cast to a float

dev/core#3514 Define hook to alter data once it has been mapped but before work is done on it.

Minor tidy up in test class

Update Contribution Import to use apiv4 field names, prior to adding hooks

Merge pull request #25911 from eileenmcnaughton/master

5.60 to master

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

Merge pull request #25897 from eileenmcnaughton/import_contact_id

Fix for issue when optional contact_id is blank importing contributions

Merge pull request #25905 from colemanw/getSetApiValues

Api4 - Prevent developer error mixing up `addValue` with `addWhere`

Fix import contribution url

dev/core#4189 Fix indiscriminate display of direct debit agreement

Merge pull request #25909 from totten/5.60-vertest

(NFC) Version Check Test - Fix multi-user and concurrent execution

dev/core#3937 Remove legacy code causing custom boolean import bug

(NFC) Version Check Test - Fix multi-user and concurrent execution

This is a small change to the way the tests run. It doesn't change the meaning of the test.

Before
------

* Writes example file to `/tmp/versionCheckTestFile.json`.
* File left behind at end of test.
* If two users runs the same test at different times, they could interfere with each other.
* If two processes run the same test at the same time, they could interfere with each other.

After
-----

* Writes example file to `/tmp/VersionCheck-NNNNNNN/versionCheckTestFile.json`
* Temp folder cleaned up at end
* If two users or two processes run the test, they use separate folders

Merge pull request #25672 from colemanw/joinStuff

Note API - Fix viewing, joining to, and updating Notes

Merge pull request #25903 from colemanw/hookCleanup

Code cleanup on aisle Hook

Merge pull request #25895 from eileenmcnaughton/url

Fix recently added import url

Merge pull request #25896 from eileenmcnaughton/import_label

Show friendlier labels when Civi-Import enabled

Merge pull request #25898 from eileenmcnaughton/import_contact_custom

Fix Contact Import tests custom date testing, remove duplicate handling

Api4 - Prevent developer error mixing up the `addValue` and `addWhere` functions

Code cleanup on aisle CRM_Utils_Hook

Merge pull request #25902 from demeritcowboy/that

[NFC] hulk smash

hulk smash