Merge pull request #25887 from eileenmcnaughton/dep_dn

Remove 2021 noisily deprecated pdfCommon function

Merge pull request #25891 from eileenmcnaughton/map_field

[NFC] Add metadata about import jobs to `MapField` form

Merge pull request #25890 from eileenmcnaughton/dep_mail_format

Remove references to deprecated `preferred_mail_format` in tests

[NFC] Add metadata about import jobs to MapField form

This extra metadata has been part of a whole lot of PRs & somehow always
winds up in conflict or otherwise so I'm adding it by itself, wihout any usage of it.

Note that
- this brings MapField into line with DataSource
- there is nothing calling these functions as of this PR - it's just
an attempt to get the extra metadata added to the forms/ parsers without
conflict

Remove references to deprecated preferred_mail_format in tests

Merge pull request #25889 from braders/remove-unused-print-property

[REF][PHP8.2] Remove property which is no longer used

[REF][PHP8.2] Remove property which is no longer used

Remove 2021 noisily deprecated pdfCommon functions

Merge pull request #25885 from totten/master-cache-unique

civicrm_cache - Replace wonky index

civicrm_cache - Add index on `expired_date`

Upgrader - Dedupe caches. Failsafe to ensure ugprade runs.

civicrm_cache - Replace wonky idx (group,path,date) with sensible idx (group,path)

Note: On the PR, I'll give a more thorough discussion for why the old index seems nonsensical.

Merge pull request #25855 from totten/setup-order

CiviUnitTestCase (etal) - Resolve spooky interaction

Merge pull request #25883 from eileenmcnaughton/mail_format

Remove preferred_mail_format from tokens

Merge pull request #25884 from lemniscus/tagset-markup

don't create tagset markup if there's no tagset

Remove preferred_mail_format from tokens

This also removes the handling from action schedule - there
might be a test checking it's presence to putting up
to adapt tests

Merge pull request #25673 from jmcclelland/transaction-lock

ensure delays from transactions don't cause cache race condition

don't create tagset markup if there's no tagset

Merge pull request #25877 from eileenmcnaughton/submitted

Move getSubmittedValues to `CRM_Core_Form`

Merge pull request #25882 from lemniscus/tagset-tpl

Skip adding script when it won't have anything to run on

Merge pull request #25881 from colemanw/dropTable

SchemaHandler - Add IF EXISTS to dropTable function

skip adding script when it won't have anything to run on

SchemaHandler - Add IF EXISTS to dropTable function

Merge pull request #25880 from mattwire/caseactivity

Use API4 to create 'Assign Case Role' activity

Use API4 to create 'Assign Case Role' activity

Merge pull request #25402 from colemanw/deprecatedActions

APIv4 - Show actions as deprecated in the Explorer

Merge pull request #25461 from mattwire/dedupepermission

Afform: Fix dedupe rules for anonymous users

Merge pull request #25779 from eileenmcnaughton/unreachable

Add deprecations to unreachable code in previously shared function

Merge pull request #25878 from eileenmcnaughton/dep_individual_format

Remove do-nothing noisily deprecated formatting code

Remove do-nothing formatting code

I've been convinced that dates are already handled long before this point
(& loading from the database) so nearly a year ago I added deprecation
notices to fire if this code ever actually did anything. No reports.

It can go:

Add deprecations to unreachable code in previously shared function

Move getSubmittedValues to CRM_Core_Form

Merge pull request #25875 from eileenmcnaughton/php82_report

[PHP8.2] Declare outputHandler property in reports

[PHP8.2] Declare outputHandler property

I did a universe search to check private is OK

Merge pull request #25872 from totten/master-release-mysqlver

CiviUnitTestCase - Degrade gracefully on older DBMS

CiviUnitTestCase - Allow relaxed enforcement on older DBMS

Merge pull request #25862 from eileenmcnaughton/dedupe_finder2

Post split code cleanup

Merge pull request #25869 from eileenmcnaughton/import_contact_pre

[REF] Simplify import preProcess

Merge pull request #25868 from eileenmcnaughton/import_switch

Switch out the switch

[REF] Simplify import preProcess

Remove require_onces

Switch out the switch

Merge pull request #25866 from eileenmcnaughton/import_contribution_mapfield

[REF] Minor code simplification - extract `getHighlightedFields` in Contribution import `MapField`

Merge pull request #25867 from eileenmcnaughton/import_unused

[NFC] Remove some no-longer-used properties on Contribution import parser

Remove some no-longer-used properties

[REF] Minor code simplification - extract getHighlightedFields

Merge pull request #25851 from eileenmcnaughton/pledge

dev/core#4079 Add Pledge Tokens

Remove fake if

Remove duplicate layer of caching

This is cached within a cache - the load of calling it is not
especially high and it is somewhat unlikely the cache will be populated
anyway so let's simplfiy...

Remove last non-variable variables

Hard-code cacheKey as variables are not variable

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

CRM_Core_Lock - Deferred release should work same for commit or rollback

ensure we properly pass the arguments needed.

ensure delays from transactions don't cause cache race condition

https://lab.civicrm.org/dev/core/-/issues/3988

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)

Normalize order of setUp()/useTransaction() - batch 6

Normalize order of setUp()/useTransaction() - batch 5

Normalize order of setUp()/useTransaction() - batch 4

Normalize order of setUp()/useTransaction() - batch 3

Normalize order of setUp()/useTransaction() - batch 2

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

CiviUnitTestCase - Complain if setup steps are flipped around

test/phpunit/api/v3 - Fix spooky interaction

This fixes an issue observed in review of 25673. With that patch included,
`JobProcessMailingTest` would still pass... but only in isolation. When
combined with `ExceptionTest`, it would fail.

I tried disabling the entire substance of `ExceptionTest`, and it still
failed. Say what?

The problem was _the order_ in which `ExceptionTest` does its initialization
(`parent::setUp()` and `$this->useTransaction()`). I suppose that order
determines the effectiveness of the transaction.

Most tests call `setup()` then `useTransaction()`. I looked for
outliers which flipped the order -- and found `EntityTagACLTest`
has the same init and the same spooky interaction.

After fixing the outliers, I'm getting the expected results on
`JobProcessMailingTest`.

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)

dev/core#4079 Add Pledge Tokens

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