Merge pull request #25291 from demeritcowboy/sk-update-57

searchkit - fix broken update task - 5.57 backport

fix broken update task

Update 5.57.0.md

Update 5.57.0.md

Set version to 5.57.0

FiveFiftySeven - Switch upgrade step from #25170 (5.57.beta1 => 5.57.0)

This was split apart from 25170 to simplify to some other aspects of the concurrent merges
required for the security-update. Those are done, and now we can proceed.

This will ensure that anyone who ran 5.57 RC will also run the step.  The
step was merged in very, very recently (like 10-20 min ago). For all other
purposes, it's the same as before.

release-notes/5.57.0.md - Add security advisories

AssetBuilder - Validate checksum for requested parameters

CiviEvent - Escape parameters in help text

security/core#120 Use Rich's patch (ArtfulRobot) with minor variation to try to resolve the issue and add unit tests

Fix Regex and include test cases as per Rich's comments

Merge pull request #25170 from demeritcowboy/sortable

dev/core#4013 dev/core#4022 - Broken quicksearch options setting

Merge pull request #25278 from totten/5.57-rn

release-notes/5.57.0.md - Add recent update

release-notes/5.57.0.md - Add recent update

Merge pull request #25273 from gellweiler/5.57

Add gellweiler to contributor-key.yml

Merge pull request #25272 from eileenmcnaughton/557

dev/core#4049 Nested tags are not showing under some circumstances.

Add gellweiler to contributor-key.yml

Fix #4049

This commit fixes #4049: Nested tags are not showing under some circumstances.

Merge pull request #25231 from alifrumin/rn5.57

[NFC] first pass at 5.57 release notes

[NFC] first pass 5.57 release notes

Merge pull request #25225 from agh1/5.57.0-releasenotes-initial

5.57.0 release notes initial run

5.57.0 release notes: added boilerplate

5.57.0 release notes: raw from script

Merge pull request #25213 from demeritcowboy/fract-qty

dev/core#4039 - Restore ability to have fractional quantity for text price fields

restore ability to have fractional quantity

Merge pull request #25204 from kurund/contribution-import-fix

dev/core#4038 import contribution fixes in update mode

import contribution fixes in update mode

Merge pull request #25197 from colemanw/fixOptions

SearchKit - Fix option loading & rendering for in-place edit

SearchKit - Fix option loading & rendering for in-place edit

Fixes dev/core#4037

Merge pull request #25189 from MegaphoneJon/batch-form-entity

Fix 'cannot determine default entity' error on New Batch page

Fix 'cannot determine default entity' error on New Batch page

Merge pull request #25177 from totten/5.57-testing

Fix miscoordination between CiviUnitTestCase/CiviEnvBuilder. Fix flaky ConformanceTest.

AutocompleteTest - Use SK as part of the baseline environment

ConformanceTest - Remove override, which causes failures in `ConformanceTest`

CiviUnitTestCase - Fix miscoordination with CiviEnvBuilder

Background
----------

Recall that `CiviUnitTestCase` and `CiviEnvBuilder` are two different
implementations of a similar concept.

* `Concept`: If two tests use the same baseline DB/environment, and if they
  preserve the baseline, then you don't need to reset everything in between
  tests.  But if they change, then you need to reset.

* `CiviUnitTestCase`: The older rendition.  Tightly coupled.  Only allows
  one specific baseline.  Mingled with a lot of unrelated functionality in
  `CiviUnitTestCase`.

* `CiviEnvBuilder`: The newer rendition.  Loosely coupled.  Allows different
  baselines.  Can be mixed into any plain-old `TestCase`.  Used by
  `Civi\Test::headless()` and `Civi\Test::e2e()`.

Problem Scenario
----------------

Suppose you have a mix of different tests running with the same DB -- e.g.

1. Run some tests based on `CiviEnvBuilder`
2. Run some tests based on `CiviUnitTestCase`
3. Run some tests based on `CiviEnvBuilder`

This wasn't originally anticipated, but it can happen -- either because
the test-suite is large+mixed, or because a developer is manually
running specific tests (which happen to be written differently).

The problem goes like this:

1. Run some tests based on `CiviEnvBuilder`
    * This resets the DB and also stores a DB signature. ("Here is how
      we setup the DB...")
2. Run some tests based on `CiviUnitTestCase`
    * This resets the DB, but leaves the old DB signature in place.
3. Run some tests based on `CiviEnvBuilder`
    * This sees the old DB signature and wrongly concludes that we
      still have the DB from step (1).

Solution
--------

Whenever one resets the DB, it should update the DB signature.

(REF) CiviEnvBuilder - Simplify `$name` calculation

Tweaking the `$name` is theoretically useful if you have entirely
independent data-sets that happen to live in the same DB.  In practice, we
only use CiviEnvBuilder to manage Civi-related data-sets.

fix borked quicksearch setting

Merge pull request #25160 from totten/5.57-file-test

CRM_Utils_FileTest - Fix compatibility with private data folders

Merge pull request #25145 from MegaphoneJon/core-4027-rc

fixes core#4027 - don't crash when a contribution has no line items

Merge pull request #25158 from totten/5.57-entity-stat

getSiteStats - Use consistent report format

CRM_Utils_FileTest - Fix compatibility with private data folders

For example, on my local, the `templates_c` folder lands at:

$BUILDKIT/build/dmaster/.civibuild/private/default/civicrm/templates_c

which is not a descendent of

$BUILDKIT/build/dmaster/web

getSiteStats - Use consistent report format

When general usage stats are enabled, the format should be consistent across
versions.

Before (4.7 - 5.56): Report stat as 'Delivered'
Before (5.57.beta1): Report stat as 'MailingEventDelivered'
After (5.57.beta1): Report stat as 'Delivered'

Merge pull request #25149 from larssandergreen/allow-dashes-in-tracked-URLs-for-text-in-flexmailer

Flexmailer: Prevent broken urls containing hyphens when click tracking is enabled for plain text mailings

allow hyphens in tracked URLs in text emails

Merge pull request #25150 from demeritcowboy/sitestats

[NFC] Update testGetSiteStats to match earlier name change

update to match earlier name change

fixes core#4027 - don't crash when a contribution has no line items

Set version to 5.57.beta1

Merge pull request #25108 from eileenmcnaughton/pdf_depp

Remove noisly deprecated function PDFLetterCommon::postProcess

Merge pull request #25140 from eileenmcnaughton/class

Fix enotice on user dash - when user has membership/s & debugging enabled

Merge pull request #25137 from eileenmcnaughton/import_loc

Stop using single hard-coded value array (clean up from previous refactors)

Fix enotice on user dash

Merge pull request #24798 from jmcclelland/currency-additional-payment

ensure currency is included to avoid translation problems

Merge pull request #25139 from christopher-yu/master

Cast membership object as array before passing to getMembershipStatusByDate()

Merge pull request #25138 from demeritcowboy/regen

[NFC] Update instructions for running regen online

Cast membership object as array before passing to getMembershipStatusByDate()

update instructions

Merge pull request #25126 from colemanw/stripTags

SearchKit - Include html columns in spreadsheet download

re-generated sql file.

Merge pull request #25136 from eileenmcnaughton/import_count

Remove always-true ifs

Merge pull request #24925 from eileenmcnaughton/token_dep

Fold `processGreetingTemplate` function only called from one place into caller, deprecate

Stop using single hard-coded value array

Remove always-true ifs

Merge pull request #25129 from MegaphoneJon/oauth-accounting-batches-cont

Fix core#7016 in a second location

Merge pull request #25110 from eileenmcnaughton/mandatory_flex

Make flexmailer mandatory

Merge pull request #25134 from seamuslee001/master

#24911 Forward port to master / 5.57

Merge pull request #25125 from MegaphoneJon/cmsuser-profile-cleanup

always define showCMS

Merge pull request #25131 from totten/master-symbols-test

(NFC) dev/core#4008 - Afform - More coverage for symbol-scanner

Merge branch 'php81_null_string'

Merge pull request #25132 from totten/master-wp-baseurl

CRM_Utils_System_WordPress - Drop unused variable

Merge pull request #25130 from civicrm/5.56

5.56

Merge pull request #25133 from colemanw/searchSortTest

SearchKit - Fix unit test so sort param actually does something

[REF][PHP8.1] Fix a couple of issues where passing in NULL in string functions is deprecated in PHP8.1

SearchKit - Fix unit test so sort param actually does something

CRM_Utils_System_WordPress - Drop unused variable

This is an off-shoot from #24911.

Before
------

`WordPress` implements the method `mapConfigToSSL()`, but ti doesn't do anything.

To see this, you can search for 'base_url' across the entire-source of a Civi-WP installation.
Specifically, I used

```
grep -lri 'global.*base_url'
```

The references appeared unrelated -- some references to Drupal/Backdrop adapters, and
some references to minified JS files. But I couldn't see anything that would in PHP on WP.

After
-----

Don't bother implementing the method. Might as well inherit the base method (which is empty).

Merge pull request #25128 from seamuslee001/5.56

Output of npm audit fix

Merge pull request #24998 from demeritcowboy/oauth-name

Avoid symbol scan errors in oauth-client help text

(NFC) dev/core#4008 - Afform - More coverage for symbol-scanner

Add more test-coverage.  This reproduces the failure-scenario that was
addressed by #25067.  (When combined with 25067, it passes.  But before
25067, it fails.)

Merge pull request #25067 from demeritcowboy/invalid-html

dev/core#4008 - alternate to 24998 - Avoid symbol scan errors

Fix core#7016 in a second location

Output of npm audit fix

Merge pull request #25124 from MegaphoneJon/profile-cleanup-1

remove unused template vars from Profile Form

ensure currency is included to avoid translation problems

See https://lab.civicrm.org/dev/core/-/issues/411

Merge pull request #25119 from demeritcowboy/current-revision

dev/core#1532 - Upgrade message about old civicase activity revisions

SearchKit - Include html columns in spreadsheet download

This adds html-type columns to the spreadsheet download, by formatting them as plain text.

Merge pull request #25122 from MegaphoneJon/no-perl

remove civimail-spooler script

always define showCMS

remove unused template vars from Profile Form

Merge pull request #25121 from MegaphoneJon/no-drupalCms

remove unused code in Profile template

remove civimail-spooler script

remove unused code in Profile template

prevent deletion of current revisions

don't delete the current revision if you delete old revisions

upgrade message

Merge pull request #25098 from JKingsnorth/dev/core#4020

dev/core#4020 Remove old recaptcha options

Merge pull request #25113 from colemanw/hideDeprecatedFields

Hide deprecated fields from SearchKit & Afform

Merge pull request #25118 from civicrm/5.56

5.56 to master

Merge pull request #25102 from alifrumin/5.56rn

[NFC] 5.56 release notes

Merge pull request #25117 from civicrm/5.56

5.56

Make flexmailer mandatory

Merge pull request #25111 from colemanw/autocompleteCleanup

APIv4 Explorer - autocomplete cleanup