Extend Generic\CheckAccess to reduce boilerplate

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

BAO - deprecate create for writeRecord: Batch

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

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

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

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