Cleanup relationships

Remove obsolete code

This was actually obsolete before the apiv4 switch I think - but it DEFINITELY is now

More better fix on population on cached organization name

Fix for handling of FALSE now we are getting apiv4 results

As exposed in api_v3_JobTest.testBatchMergeWorks

Adjust to APiv4 returning label as a field

As tested by CRM_Dedupe_MergerTest::testGetRowsElementsAndInfoSpecialInfo

Do not consider greeting display names in merges

As surfaced by api_v3_ContactTest::testMergeGetConflicts
these are basically caches of calculated fields

Fix employer not carrying over correctly in merge

This addresses a deduper bug where selecting the employer_id to
carry over in the form does NOT result in the newly
merged contact having that organization's name
in it's organization_name field

Merge pull request #26008 from colemanw/fixEntityRefDefaults

Afform - Fix showing default values for EntityRef fields

Set version to 5.61.beta1

Merge pull request #26013 from eileenmcnaughton/test_strict

Type hints & strictness fixes within tests

Type hints & strictness fixes within tests

Merge pull request #26012 from eileenmcnaughton/old

Remove an attention-span's worth of cvs info

Merge pull request #26011 from eileenmcnaughton/strict_test

[Test] Strictness fixes - `CiviUntTestBaseClass`

Remove an attention-span's worth of cvs info

[Test] Strictness fixes

Merge pull request #26005 from eileenmcnaughton/strict_test

Strictness cleanups in test class

Afform - Fix showing default values for EntityRef fields

There was some back-and-forth with this. #25374 removed the change() trigger, which fixed
the recursion bug described in https://lab.civicrm.org/dev/core/-/issues/4083 but broke
setting defaults, prompting #25465 which proposed to change it back.

This fix walks the line by triggering the change event but only when the value has actually changed.
Fixes dev/core#4110

Strictness cleanups in test class

Merge pull request #25175 from colemanw/afformRepeatRelationships

Afform - Support repeatable relationships

Merge pull request #26004 from eileenmcnaughton/dedupe_old

Remove upgrade custom field handling from Dedupe code - previously shared function

Merge pull request #26003 from eileenmcnaughton/dedupe_strict

Clarify `entityID` types in Merger class

Clarify entityID types in Merger class

Merge pull request #26001 from demeritcowboy/undef

Avoid potentially undefined var

Remove upgrade custom field handling from Dedupe code

Merge pull request #25988 from eileenmcnaughton/import_campaign

When importing invalid campaigns they pass validation (only to fail on import)

avoid potentially undefined var

Merge pull request #25647 from mattwire/disableinvalidrelationship

Allow to disable an invalid relationship (eg. contact subtype was changed so no longer valid)

Merge pull request #25997 from agileware/CIVICRM-2120

🥚 Payment API creates Activity with subject: "$x Offline Payment for Contribution" even when a payment was captured online

Merge pull request #25991 from eileenmcnaughton/import_label

Use label, where present for import field labels

Merge pull request #25995 from colemanw/uniqueNameIndex

Campaign - Make name index unique

Campaign - Allow writeRecord to autogenerate name from title, fix tests

The Campaign BAO was autogenerating a name from the title but without the dedupe checking feature
performed by writeRecord, so it was getting in the way of that safeguard.

CIVICRM-2120 Payment API creates Activity with subject: "$x Offline Payment for Contribution" even when a payment was captured online

Campaign - Make name index unique

Merge pull request #25996 from eileenmcnaughton/test_clean

[TESTS] Fix some campaign cleanup

[TESTS] Fix some campaign cleanup

Use label, where present

Label is preferred over title - when the metadata comes from the
DAO it is in html['label'], but if it comes from apiv4 it
seems it is mapped higher up...

Merge pull request #25990 from eileenmcnaughton/import_user

Fix url from just-merged PR

Fix url from just-merged PR

Fix to validation error for invalid campaign

[REF] Add in Test for importing a deliberartly bad campaign id

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

Afform - Support repeatable relationships

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