[TESTS] Fix some campaign cleanup

Merge pull request #25979 from eileenmcnaughton/civi_import_notice_fix

Fix enotice when using Civi-import

Merge pull request #25939 from eileenmcnaughton/template_search

Civi-Import - Add Template search

Merge pull request #25971 from colemanw/acitivityCruft

Activity BAO - Remove unused classes/functions

Merge pull request #25973 from colemanw/savedSearchUpdate

BAO - deprecate create for writeRecord: SavedSearch

Merge pull request #25976 from colemanw/deprecateSetIsActive

BAO - Deprecate `setIsActive` BAO functions

Merge pull request #25986 from civicrm/5.60

5.60

Merge pull request #25977 from eileenmcnaughton/import_edit_cond

Civi-Import - improvements to the search display for imported rows

Merge pull request #25980 from eileenmcnaughton/import_js

Minor fixes to Civi-Import screen, populating defaults, missing `ts`

Merge pull request #25981 from eileenmcnaughton/560

Fix error when importing a soft credit AND a note when currency not provided

Merge pull request #25985 from civicrm/5.60

5.60 to master

Merge pull request #25982 from demeritcowboy/fileoncase

dev/core#4217 - Fix file on case search by id and broken entityref pagination

Merge pull request #25983 from demeritcowboy/casejson

[NFC] Don't have null values for required fields in Case test fixture

Extend Generic\CheckAccess to reduce boilerplate

Minor fixes to Civi-Import screen, populating defaults, missing ts

BAO - Deprecate setIsActive BAO functions

don't be null

add test

fix file on case search by id and broken pagination

Fix error when importing a soft credit AND a note when currency not provided

Fix enotice when using Civi-import

Merge pull request #25974 from colemanw/updateSubHistory

BAO - deprecate create for writeRecord: SubscriptionHistory

Merge pull request #25978 from civicrm/5.60

5.60 to master

Merge pull request #25967 from alifrumin/5.60rn

[NFC] 5.60 release notes

Merge pull request #25970 from colemanw/aclWriteRecord

BAO - deprecate create for writeRecord: ACL

Merge pull request #25975 from colemanw/factory

Demolish the factory 🏭💥

Move entity_id before status_message

Removed edit-in-place options from Import views on imported rows.

It's confusing to the user to be able to edit the records when they
are already imported. This is not a real security thing - more
of a UI thing - but Coleman recommends implementing through
checkAccess. I did leave open the back door of being able to edit it
if the status is now present as there isn't a real reason they shouldn't -
just that it is confusing to present edit-in-place, except when they could
edit & re-try the import

Merge pull request #25969 from colemanw/entityValues

SearchKit - Pass entire record into `checkAccess`

Merge pull request #25972 from colemanw/batchUpdate

BAO - deprecate create for writeRecord: Batch

Remove unused CRM_Contact_DAO_Factory

BAO - deprecate create for writeRecord: SubscriptionHistory

BAO - deprecate create for writeRecord: SavedSearch

BAO - deprecate create for writeRecord: Batch

Activity BAO - Remove unused classes/functions

BAO - deprecate create for writeRecord: ACL

SearchKit - Pass entire record into `checkAccess`

For efficiency it's best to pass all available data into the `checkAccess` action to avoid unnecessary
db lookups. This ensures the full record is passed in when checking access before enabling in-place edit.

Merge pull request #25956 from totten/master-tap

(dev/core#4188) PHPUnit - Port TAP printer to v9.x

[NFC] 5.56 release notes first pass

Merge pull request #25965 from eileenmcnaughton/dep_tax

Remove noisily deprecated code

Merge pull request #25966 from civicrm/5.60

5.60 to master

Merge pull request #25963 from seamuslee001/dev_core_4029

dev/core#4029 Ensure that Pan Truncation and card type id is recorded…

Merge pull request #25964 from eileenmcnaughton/mem_regress

Fix master-only regression on loading membership changes

Remove noisily deprecated code

Fix master-only regression on loading membership changes

A recent commit caused _mid to be set to FALSE to denote
it had been looked up & was empty (as opposed to not yet checked)
but these places are using isset.

The change fixes them all to call the function.

Merge pull request #25962 from eileenmcnaughton/import_url

Upgrade fix for url change

dev/core#4029 Ensure that Pan Truncation and card type id is recorded when processing front end contributions

Upgrade fix for url change

Merge pull request #25952 from seamuslee001/dev_core_3133

dev/core#3133 - Extend Authx functionality to support validation of e…

Merge pull request #25958 from colemanw/javascript

CustomFields - drop unused 'javascript' column

Merge pull request #25953 from demeritcowboy/too-loggy

Duplicate logging not needed for PEAR_Errors anymore

Merge pull request #25955 from colemanw/groupEdit

Manage groups - add dedicated url for edit/delete group

Merge pull request #25944 from colemanw/writeRecordRef

[REF] Refactor location-related BAOs to use `writeRecord`

Merge pull request #25959 from eileenmcnaughton/import_links

Flush cache that holds schema links information when saving UserJob

Manage groups - add dedicated url for edit/delete group

Merge pull request #25957 from eileenmcnaughton/import_template

Import templates Set created_id, expires_date appropriately

Flush cache that holds schema links information when saving UserJob

CustomFields - drop unused 'javascript' column

Import templates Set created_id, expires_date appropriately

This tweaks just-merged userJob functionality.

When creating an import job it is not correct to use
the created_id or expires_date from the template - but
these should be set appropriately, rather than left blank

Make sure search_kit is installed first

Add Template search

tools/scripts/phpunit - Enable tap for phpunit9

Testing - Port TAP printer to PHPUnit 9

(REF) Testing - Extract class-file "TAP7"

BAO - Refactor Address add/create to use writeRecord

BAO - Deprecate email/im/phone/website/openId create/add functions in favor of writeRecord

Merge pull request #25954 from demeritcowboy/case-dashboard-recent

dev/core#3804 - Fix recently performed activities on case dashboard when no upcoming exist

fix recently performed activities on case dashboard

Merge pull request #25917 from eileenmcnaughton/smarty_pcp

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

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

Merge pull request #25906 from herbdool/issue-4200

dev/core#4200 use org ID rather than assume it

logging not needed here anymore - still gets logged later if not caught

Merge pull request #25726 from ixiam/devCrmMoneyLocale

add locale param to smarty crmMoney()

authx - Simplify service declaration

dev/core#3133 - Extend Authx functionality to support validation of externally generated JWTs

Migrate legacy rest to appropriate place

Remove legacy auth as not needed due to it self registering now

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