Upgrader - Fix error on Snapshot::cleanupTask

This is a follow-up to #23522.

Before
------

Upgrade works on `cv upgrade:db` but fails on `drush civicrm-upgrade-db`

After
-----

Upgrades on both `cv upgrade:db` and `drush civicrm-upgrade-db`

Merge pull request #23584 from civicrm/5_50_guzzle

[REF] Upgrade Guzzle to 6.5.6

[REF] Upgrade Guzzle to 6.5.6

Merge pull request #23572 from colemanw/resultLength

SearchKit - Fix 'undefined var' error after import

Merge pull request #23522 from totten/5.50-snapshot

Upgrader - Add support for automatic snapshots

SearchKit - Fix 'undefined var' error after import

Merge pull request #23566 from eileenmcnaughton/550

Calculate unit_price based on qty

Calculate unit_price based on qty

Define experimental option CIVICRM_UPGRADE_SNAPSHOT

CRM/Upgrade - Add notice about snap_* and MySQL configuration

CRM/Upgrade - Snapshots should have name `snap_{OWNER}_{VERSION}_{NAME}`

1. Prevents the trigger-based logging system from hitting the snapshots.
2. Makes the de-facto table names shorter (less likely to hit 64 char limit)
3. In practice, it should be easier to skim
4. Use the same pseudo-extension name that we use elsewhere (`civicrm`)

CRM/Upgrade - Snapshots should have name `civicrm_snap_{OWNER}_{VERSION}_{NAME}`

CRM/Upgrade - Store snapshots with compression. More comments.

(REF) SnapshotTest - Split big test in two. Cleanup comments.

CRM/Upgrade - Enable snapshots for 5.50.alpha1

Note: This is being committed during 5.50.beta1.  If anyone previously
executed the upgrade for the RC/beta/alpha, then may not get snapshots.  But
those upgrades are already done; and they're not guaranteed anyway; and the
snapshots aren't mandatory anyway.

CRM/Upgrade - Add basic support for snapshots to the upgrade framework

CRM/Upgrade - Define "Snapshot" helper (with unit-tests)

Merge pull request #23528 from eileenmcnaughton/raw_money

Fix empty money handling

Fix empty money handling

Merge pull request #23499 from totten/master-cdesc

(dev/core#3436) MultipleRecordFieldsListing.tpl - JS strings should us JS escaping

MultipleRecordFieldsListing.tpl - JS strings should us JS escaping

Merge pull request #23497 from totten/5.50-schedrem-bool

Restore `civicrm_action_schedule.limit_to` (5.50)

FiveFortyNine - Tweak task title

FiveFortyNine - Revise message about "limit to"

1. This fixes a problem where the message shows at the opposite-of-correct time
   (ie it should display on builds that have already run 5.49.{beta,0,1}, but
   it actually displayed on builds that have not run 5.49.{beta,0,1})

2. This makes the pre+post messages match.

FiveFortyNine - Don't munge soft credit 'limit-to'

I'm not sure what it even means to limit a scheduled-reminder to a "soft_credit_type".

But it's a thing - I can use the 5.48 GUI to create it. Here's how it looks in the DB (in 5.48):

+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+
| id | title                                    | recipient        | limit_to | recipient_manual | recipient_listing | group_id |
+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+
| 11 | Limit To - Soft Credit                   | soft_credit_type |        1 | NULL             | in_memory_of      |     NULL |
+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+

The `changeBooleanColumnLimitTo()` was coercing down to `limit_to=null` because it lacked `recipient_manual` and `group_id`.

FiveFortyNine - Fix guard on `changeBooleanColumnLimitTo()`

add pre+post upgrade message

FiveFortyNine - Draft message for 5.49.2

Revert civicrm_action_schedule.limit_to boolean column to be NULL

Merge pull request #23481 from alifrumin/rn5.50

[NFC] First pass at 5.50 release notes

[NFC] first stab at 5.50 release notes

Merge pull request #23447 from agh1/5.50.0-releasenotes-initial

5.50.0 release notes initial run

5.50.0 release notes: added boilerplate

5.50.0 release notes: raw from script

Merge pull request #23440 from totten/5.50-since-when

(NFC) mixin/mgd-php@1 - Twiddle `@since`

(NFC) mixin/mgd-php@1 - Twiddle `@since`

Overview
--------

Switch `@since` from "modified" version to "introduced" version.

Comment
-------

The [spec for `@since`](https://docs.phpdoc.org/guide/references/phpdoc/tags/since.html)
is more open ("when an element was introduced _or_ modified"), but this matches similar
usages in the same repo.

Merge pull request #23423 from totten/5.50-mixin-since

(NFC) mixin/**.php - Add @since tags

Merge pull request #23422 from totten/5.50-mgd-ver

ManagedEntities - Update version number for mgd-php

tools/mixin/bin/mixer - Display the `@since` data

(NFC) mixin - Add `@since` tags

ManagedEntities - Update version number for mgd-php

This is a follow-up/correction for #22959 (circa 5.50.alpha1; specifically
fdc67a75b2f20aea9f0961683bd8a59c78755d5f). The prior revision had updated the
behavior of `mgd-php`, so it should have also updated the version-number.

Merge pull request #23412 from colemanw/inputTypes

Afform - Fix RC regression of missing input types

Afform - Fix RC regression of missing input types

This regressed because the client-side variable was renamed as
part of 47332dba8ff22aa6f08d473a0437dc02ca53458d

Merge pull request #23407 from eileenmcnaughton/import_550

[Import] Fix 5.50 error url

[Import] Fix 5.50 error url

Merge pull request #23397 from totten/5.50-e2e-error

(E2E) `testErrorChrome` - Update assertions for page-chrome

Merge pull request #23398 from totten/5.50-e2e-error-bd

(E2E) testErrorChrome - Skip Backdrop and D8+ (much like D7)

Merge pull request #23394 from eileenmcnaughton/import_550

Import 550

Merge pull request #23396 from totten/5.50-upgrade-esc

5.50.alpha1.mysql.tpl - Escape `grouping` (reserved word in MySQL 8.0)

(E2E) testErrorChrome - Revise page-chrome check for all Drupal-style UFs

(E2E) testErrorChrome - Skip Backdrop and D8+ (much like D7)

This test was added during 5.50.alpha (#23257, #22805).  It's purpose is to
assert that the page-chrome/site-wide-template is present on certain
error-messages; however, the underlying behavior was only implemented on
WordPress. It has not yet been implemented on other CMS.

Before
------

* __WordPress__: Runs testErrorChrome. Expected to pass.
* __Drupal 7__: Skip testErrorChrome (mostly) because it is expected to fail.
* __Drupal 8+, Backdrop__: Runs testErrorChrome. Fails.

After
-----

* __WordPress__: Runs testErrorChrome. Expected to pass.
* ___Drupal 7, Drupal 8+, Backdrop__: Skip testErrorChrome (mostly) because it is expected to fail.

Comment
-------

It still runs `testErrorChrome` for `permission` errors; per MM discussion, it appears that this does actually work, but we need
to tune assertion for BD+D8.

(E2E) `testErrorChrome` - Update to work with current wp-demo build

This test was added during 5.50.alpha (#23257, #22805).  It's purpose is to
assert that the page-chrome/site-wide-template is present.  It appears that
the markup from WordPress has changed slightly.  (It may be that I
originally tested against an older version of WP.) In any event, this
relaxes the test so that it also passes on a newer WP build.

5.50.alpha1.mysql.tpl - Escape `grouping` (reserved word in MySQL 8.0.1+)

Import - also export invalid as export error

I found that rows that fail validation are marked 'invalid'
rather than error & won't show on the preview screen.

I guess I could check for invalid vs error separately
- gets kinda weird though - if you leave them
in the file you are importing surely they are still errors
to see on the summary screen even though
you knew they were invalid on the Preview?

Merge pull request #23389 from colemanw/getFieldsContactTypeFix

APIv4 - Fix fatal error on getFields when passing in contact id

APIv4 - Fix fatal error on getFields when passing in contact id

Merge pull request #23387 from totten/5.50-cacert

Update expired root certs (5.50)

update expired root certs

Set version to 5.50.beta1

Merge pull request #23343 from eileenmcnaughton/import_parser_job

[REF] [Import] Stop passing submittedValues as parameters

Merge pull request #23380 from eileenmcnaughton/import_conflict

[REF] [Import] Remove another good intention - import 'conflicts'

Merge pull request #23381 from colemanw/searchKitDateFields

SearchKit - Add date pseudo-fields

Merge pull request #23371 from eileenmcnaughton/import_valid

[REF] [Import] extract validation functionality

[REF] [Import] Remove another good intention - conflict

There is all sorts of handling for 'conflict' - but a conflict return code is never
generated

[REF] [Import] Stop passing submittedValues as parameters

These parameters are all availabe via

Extract validateValues

Merge pull request #23378 from colemanw/searchKitNow

APIv4 - Add NOW() date function

Merge pull request #23379 from samuelsov/tsOnOff

Translation - distinguish between 'On' for date and 'On'/'Off'

SearchKit - Add date pseudo-fields

Merge pull request #21993 from mattwire/aclcachequery2

ACLCache: Efficiency. Fix duplicate inserts

Translation - distinguish between 'On' for date and 'On'/'Off'

Merge pull request #23369 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variables

APIv4 - Add NOW() date function

Adds a function for NOW which returns the full date+time, and distinguishes
it from CURDATE which just returns the date part.

Merge pull request #23377 from colemanw/month

APIv4 - Add MONTH sql function

Merge pull request #23375 from eileenmcnaughton/ref

[Ref] Get rid of private deprecated legacy function

Merge pull request #23282 from eileenmcnaughton/import_default_location_type

[REF] Do not treat site having a default location type as optional

Merge pull request #23292 from eileenmcnaughton/import_output_table

[Import] Output errors, duplicates csvs directly from the user job table

APIv4 - Add MONTH sql function

[Ref] Get rid of private deprecated legacy function

It now doesn't have enough code to justify it's own function

Merge pull request #23368 from eileenmcnaughton/ref

[REF] Minor code simplifcation - reduce silly lines

Remove conflict download from preview

I can't see where these would arise - but definitely
not on the preview screen - which only runs basic validation

Add output form for csv-on-the-fly

Make tablename a property of the datasource

The datasource stores the table name on the job - we don't need to pass it around.

Also - stop cleaning up the temp table at the end - we want it to
output results but will add a cleanup routine later

Move adding status fields on import table to datasource

Cleanup cleanup on old tables for form re-submission

[REF] [Import] Remove always NULL parameter from private function

Merge pull request #23374 from eileenmcnaughton/549_up

549 to master

Merge pull request #23356 from eileenmcnaughton/pp

Ensure comma not formatted into amount

Merge pull request #23366 from eileenmcnaughton/import_tpl

[REF] [Import] Remove unused tpl assign showColNames

Ensure comma not formatted into amount

Probably the function doesn't add anything as params['amount']
should be correctly formatted - however,
it shouldn't add a comma - per
https://github.com/eileenmcnaughton/nz.co.fuzion.omnipaymultiprocessor/issues/227

Merge pull request #23370 from eileenmcnaughton/import_catch

[Import] [Ref] [Minor] Simplify error handling with a try-catch block

[REF] [Import] Trivial - simplify variables

Simplify error handling with a try-catch block

Merge pull request #23367 from eileenmcnaughton/import_prop

[Ref][Import] Remove another good intention - random email counting

Merge pull request #23363 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variables

Merge pull request #23336 from colemanw/customGroupExtends

CustomFields - Improve metadata about which custom groups belong to which entities

[REF] Minor code simplifcation - reduce silly lines

Merge pull request #23364 from eileenmcnaughton/rel

Minor function simplification in private function

[Ref][Import] Remove another good intention - random email counting

[REF] [Import] Trivial - simplify variables