Remove never-true if

isProfile is always false

Stop adding field just so the calling code can strip them

The calling code will not use any field with no WHERE clause

Remove handling for no contact type - it is ALWAYS passed in here

Copy shared ImportableFields to private function, for unravelling

Merge pull request #25854 from eileenmcnaughton/test_tidy

[NFC] Minor tidy up in `MappingTest`

Merge pull request #25852 from eileenmcnaughton/php82_contacttest

[Test][Php8.2] Fix undeclared properties & other housekeeping

Merge pull request #25859 from eileenmcnaughton/dedupe_private

Make `Finder::tableQuery private`, after universe search

Merge pull request #25864 from braders/php8.2-ParticipantPaymentTes

[REF][PHP8.2] Refactor api_v3_ParticipantPaymentTest

Merge pull request #25865 from braders/php8.2-eventid-property

[REF][PHP8.2] Refactor getParticipantOrderParams to never write a dynamic property

[REF][PHP8.2] Refactor getParticipantOrderParams to never write a dynamic property

[REF][PHP8.2] Refactor api_v3_ParticipantPaymentTest

Merge pull request #25860 from eileenmcnaughton/dedupe_query

[REF] Replace use of legacy `$dao->query()` with `CRM_Core_DAO::executeQuery()`

Replace use of legacy dao->query()

Merge pull request #25844 from braders/dynamic-props-ProcessorFormTest

[REF][PHP8.2] Get rid of dynamic properties in CRM_Core_Payment_ProcessorFormTest

Make Finder::tableQuery private, after universe search

Merge pull request #25858 from totten/master-autoclean-explicit

AutoClean - Optionally allow explicit cleanup

[REF][PHP8.2] Get rid of dynamic properties in CRM_Core_Payment_ProcessorFormTest

AutoClean - Optionally allow explicit cleanup

Merge pull request #25856 from totten/test-release-locks

CiviUnitTestCase - During teardown, ensure that we cleanup locks

Merge pull request #25801 from eileenmcnaughton/82_part_test

[PHP8.2] Clean up in `apiv3_ParticipantTest` with view to 8.2 compatibility

Merge pull request #25799 from eileenmcnaughton/php82-invoice_id

[PHP8.2] [Test] Remove use of (often undeclared) property `_invoiceID` in tests & one instance of `paymentProcessorID2`

Merge pull request #25857 from colemanw/unusedVar

Remove unused var

Remove unused var

CiviUnitTestCase - During cleanup, ensure that we have a clean slate (wrt locking)

Merge pull request #25853 from lemniscus/crm-core-form-php8-typeerror

Avoid type error in CRM_Core_Form on php 8

Minor tidy up in MappingTest

avoid type error in CRM_Core_Form on php 8

[Test][Php8.2] Fix undeclared properties & other housekeeping

Merge pull request #25811 from eileenmcnaughton/import_access

dev/core#4184 Permit access to template user jobs (imports)

Merge pull request #25847 from civicrm/5.60

5.60

Permit access to template user jobs (imports)

Merge pull request #25849 from seamuslee001/dev_core_4190

dev/core#4190 Fix issue where defaultValues needs to be an array not …

Merge pull request #25850 from demeritcowboy/vardumper

PHP8 - Update symfony/var-dumper

Merge pull request #25831 from eileenmcnaughton/skip_rest

[PHP8.2] Remove unused setting on undeclared properties

Merge pull request #25830 from eileenmcnaughton/skip_member

[PHP8.2] Remove unused variabls from `Member_Form_Task_PDFLetter::postProcessMembers`

Merge pull request #25829 from eileenmcnaughton/skip

[REF] [php8.2] Remove unused parameters from `Contribute_PDFLetter::buildContributionArray`

update var-dumper for php8

Merge pull request #25776 from eileenmcnaughton/rec-fix

dev/core#4000 Fix (old) regression - ensure only template contribution updates update recur

dev/core#4190 Fix issue where defaultValues needs to be an array not boolean

Merge pull request #25846 from seamuslee001/fix_invalid_soft_credit_table

dev/core#4187 Fix errors when sorting by contributor name or receive …

dev/core#4187 Fix errors when sorting by contributor name or receive date in the soft credit table

Merge pull request #25827 from colemanw/afformCommaFilters

SearchKit - Handle comma-separated values passed from Afform url

Merge pull request #25842 from eileenmcnaughton/import_remove_dep

Deprecated `buildSavedMappingFields` noisily

Merge pull request #25841 from eileenmcnaughton/import_old

Remove un-used, un-displayed button

Merge pull request #25824 from colemanw/dateFields

Afform - Allow contact created/modified date fields to be used as filters

Merge pull request #25843 from eileenmcnaughton/master

5.60 to master

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

Deprecated buildSavedMappingFields noisily

This function was only being used from one place & was
only separate from that function because the Contact DataSource
form used to call it directly. That is no longer
true so I've coped the code into
the calling function & added noisy deprecation, with a view to remove

Remove un-used, un-displayed button

Merge pull request #25822 from colemanw/afformValidateSavedSearch

Afform - Fix validateBySavedSearch

Merge pull request #25839 from eileenmcnaughton/master

Merge 5.60 to master

Merge pull request #25832 from colemanw/multiSelectUrl

Afform - set multi-select values from url

Afform - Allow contact created/modified date fields to be used as filters

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

Merge pull request #25835 from eileenmcnaughton/560

Remove require_dev yoast/phpunit-polyfills

Merge pull request #25836 from totten/phpunit-env-var

(REF) PHPUnit - Allow env-var to specify version

Merge pull request #25834 from braders/php8.1-dedupefind

[REF][PHP8.2] Declare property in CRM_Contact_Form_DedupeFind

PHPUnit - Allow env-var to specify version (for tools/scripts/phpunit)

PHPUnit - Allow env-var to specify version (for mixer tests)

Remove require_dev yoast/phpunit-polyfills

[REF][PHP8.2] Declare property in CRM_Contact_Form_DedupeFind

Merge pull request #25818 from totten/master-wf-strict

WorkflowMessage - Enable strict parsing of annotations

Afform - set multi-select values from url

Remove unused setting on undeclared properties

Remove unused variabls from Member_Form_Task_PDFLetter::postProcessMembers

Remove unused parameters from Contribute_PDFLetter::buildContributionArray

Merge pull request #25828 from braders/php8.2-summary-contact-edit-options

[REF][PHP8.2] Don't use a property where a variable will do (CRM_Contact_Page_View_Summary)

Merge pull request #25782 from anemirovsky/fix-profile-listing-column-blank

Fix profile search listings columns are sometimes blank.

[REF][PHP8.2] Don't use a property where a variable will do (CRM_Contact_Page_View_Summary)

SearchKit - Handle comma-separated values passed from aform url

Merge pull request #25825 from civicrm/5.60

5.60 to master

Merge pull request #25806 from mlutfy/core4166rc

dev/core#4166 Fix Contribute Import Parser fatal when soft-credit is empty

dev/core#4166 Fix Contribute Import Parser fatal when soft-credit is empty

Afform - Fix validateBySavedSearch

Merge pull request #25790 from braders/php82-conformance-test-failures

[REF][PHP8.2] Only set properties which exist within ArrayFormatTrait->loadArray()

Merge pull request #25820 from eileenmcnaughton/sql_cache

Do not report missing triggers if the user cannot check them

Merge pull request #25687 from yashodha/dev-4124

(dev/core#4124) Past campaigns are not to be assigned via batch updat…

Remove the word 'Experimental' from setting that has been in prod for 7 years

Do not report missing triggers if the user cannot check them

WorkflowMessage - Enable strict parsing of attributes

Not sure it's a good idea. Let's see what Jenkins thinks.

Merge pull request #25817 from eileenmcnaughton/import_url

Fix typo that causes e-notice in just-merged commit

Fix typo that causes e-notice in just-merged commit

Merge pull request #25802 from eileenmcnaughton/delete_message

[PHP8.2] declare deleteMessage on `entityFormTrait`

Merge pull request #25816 from eileenmcnaughton/import_sk

Add links to search kits to import preview screen, when Civ-Import is enabled

Merge pull request #25815 from civicrm/5.60

Merge 5.60 to master

Merge pull request #25810 from eileenmcnaughton/import_subscriber

dev/core#4184 Fix Import over-authorization

Update CRM/Core/Form/EntityFormTrait.php

Co-authored-by: Tim Otten <totten@civicrm.org>

Add links to search kits to preview screen

dev/core#4184 Fix Import over-authorization

Merge pull request #25759 from eileenmcnaughton/import_notice

Fix really annoying notice on mispelt variable name `$isCheked`

Merge pull request #25814 from seamuslee001/fix_false_ipn_log_paypal

[REF] Fix issue where the strict typing was causing false notices to …

Merge pull request #25813 from samuelsov/5.60

dev/core#4186 regression for Contact Reference Autocomplete

Merge pull request #25812 from braders/remove-unused-param

[PHP8.2] (Ref) Remove unused dynamic property in report form class

[REF] Fix issue where the strict typing was causing false notices to appear in Config And Log

dev/core#4186 regression for Contact Reference Autocomplete

(Ref) Remove unused dynamic property in report form class

Merge pull request #25736 from MegaphoneJon/remove-usps-warning

don't show USPS warning when USPS lookup disabled

Merge pull request #25809 from braders/php-8.2-CaseContactTest

[REF][PHP8.2] Tidy up properties in api_v3_CaseContactTest

Merge pull request #25738 from MegaphoneJon/required-field-separator

field separator is a required field

Merge pull request #25787 from eileenmcnaughton/import_use_uploaded

Import data source Re-use already uploaded data, optionally on use of 'back' button