Merge pull request #21513 from JKingsnorth/core-2846-1-improve-start-end-date-validation

dev/core#2846 Towards validation of start and end dates. Forms.

Merge pull request #23243 from colemanw/reconcile

APIv4 - Add `Managed::reconcile` action.

Merge pull request #23309 from civicrm/5.49

5.49

Merge pull request #23303 from colemanw/afformMetaEvent

Afform - Dispatch event to alter admin metadata; provide mixin

Merge pull request #23280 from eileenmcnaughton/import_job

[REF] [Import] Ensure jobID is set in contact flow

Merge pull request #23293 from eileenmcnaughton/import_deprecated

[REF] [Import] call non-deprecated function instead of deprecated

Merge pull request #23308 from eileenmcnaughton/import_fields

[REF] [Import] [Trivial] Make 'mapper' field available as a submittable value throughout the flow

Merge pull request #23307 from eileenmcnaughton/import_defaults

[REF] [Import] [Trivial] Make mapperKeys parameter optional in construct

Merge pull request #23306 from eileenmcnaughton/import_notice

[Import] Enotice fix on contact import field mapping screen

Merge pull request #22837 from totten/master-authx-ignore-header

(dev/core#3416) authx - If `Authorization:` header is disabled, then ignore it.

[REF] [Import] [Trivial] Make 'mapper' field available as a submittable value throughout the flow

The fields defined here can be accessed from any field in
the flow, removing the need to pass the values from form to form

[REF] [Import] [Trivial] Make mapperKeys parameter optional in construct

[Import] E-notice fix

This fixes 1 of 2 notices on the Contact import field mapping screen.
- instead of assigning a variable for show-hide it uses jquery

Merge pull request #23298 from artfulrobot/artfulrobot-21473-followup

Compare currencies as well as amounts on template Contribution change

Afform - Dispatch event to alter admin metadata; provide mixin

Instead of scanning files for Afform entities, this dispatches an event to allow them to be altered.
It provides a mixin which extensions can use to automatically do the file scanning.

Compare currencies as well as amounts on template Contribution change

Update CRM/Contribute/BAO/ContributionRecur.php

Merge pull request #23300 from eileenmcnaughton/import_extract

[REF] [Import] [Trivial] Extract duplicate match options

Merge pull request #23301 from eileenmcnaughton/549_amount

Fix propertyBag regression on `amount`

authx_civicrm_enable - Don't enable `Authorization:` headers if there's plausible conflict

Some deployments may have two layers of authorization:

(1) A generic/site-wide HTTP basic check (perhaps to prevent search-engines
    from finding the site; perhaps enforced by a reverse proxy)
(2) A Civi or CMS credential (eg session-cookie, ?_authx, or X-Civi-Auth:)

Authx sits in layer 2.  It should enable HTTP `Authorization:` handling
if-and-only-if there is NOT a pre-existing `Authorization:` mechanism.

To test this, I enabled the extension over APIv3 REST -- with/without a superfluous header:

```
curl -X POST -d 'entity=Extension&action=enable&json=%7B%22keys%22%3A%22authx%22%7D&api_key=FIXME_USER_KEY&key=FIXME_SITE_KEY' \
  'http://dmaster.127.0.0.1.nip.io:8001/sites/all/modules/civicrm/extern/rest.php'

curl -X POST -H 'Authorization: Bearer superfluous' -d 'entity=Extension&action=enable&json=%7B%22keys%22%3A%22authx%22%7D&api_key=FIXME_USER_KEY&key=FIXME_SITE_KEY' \
  'http://dmaster.127.0.0.1.nip.io:8001/sites/all/modules/civicrm/extern/rest.php'
```

AllFlowsTest - Make `testJwtMiddleware()` less sensitive to interactions with other tests

AllFlowsTest - If `Authorization:` support is disabled, then it's ignored completely.

This means that the request is effectively anonymous -- the same as it would be if you had not sent an `Authorization:` header.

authx - If `Authorization:` header is disabled, then ignore it.

AllFlowsTest - Add case `testIgnoredHeaderAuthorization()`

Merge pull request #23286 from eileenmcnaughton/mapping

[REF][Test] Update loadMappingTest to use form flow

dev/core#3413 Remove handling of total_amount form propertyBag

This appears to have caused a regression per
https://lab.civicrm.org/dev/core/-/issues/3413

In addition I don't think we should have a 'sloppy alternative'
to  being called with the correct documented
amount property

https://docs.civicrm.org/dev/en/latest/extensions/payment-processors/create/#available-parameters

[REF] [Import] [Trivial] Extract duplicate match options

As part of moving from passing variables from form to form & getting using
'get' we are retrieving onDuplicate via getSubmittedValue(), also extracting
to allow code blocks to clarify better

[REF][Test] Update loadMappingTest to use form flow

This makes it test more accurately what the form is doing....

Merge pull request #23268 from totten/master-ext-load

dev/core#3184 - Definitively load mainfiles during bootstrap - Option 3

Merge pull request #21943 from mattwire/gccacheignore

Simplify and improve performance of query to insert updated cache

Merge pull request #23284 from eileenmcnaughton/import_contact_type

[REF] [Import] Extract getContactType

[REF] [Import] call non-deprecated function instead of deprecated

Merge pull request #23287 from kurund/update-supporter-url

update-supporter-url

Merge pull request #23285 from eileenmcnaughton/import_old_var

Remove an unset variable

update-supporter-url

Merge pull request #23147 from braders/contactimage-accept-attr

Restrict allowed uploads - contact image

Remove an unset variable

[REF] [Import] Extract getContactType

Note that the function on the MetaDataTrait is moved to the
Parser_Class - 3 forms use the trait - a unittest class,
the Parser_Class and MapField.

MapField was not calling the old function but the newly added one
clashes - and highlights that the contents of the function
should differ for the 2 scenarios - hence moving
off the Trait and onto the 3 classes that use it

Merge pull request #23272 from eileenmcnaughton/import_contract

[REF] [Import] Simplify dataSource contract - remove unused preProcess

Merge pull request #23277 from eileenmcnaughton/import_main

Whitespace cleanup in template file

Merge pull request #23278 from demeritcowboy/case-tags-static

Make static function static - processCaseTags

Whitespace cleanup

[REF] [Import] Ensure jobID is set in contact flow

Merge pull request #23275 from eileenmcnaughton/error

Remove another good intention - unused variables

Merge pull request #22683 from mattwire/paymentstatushelpersall

dev/financial#141 - Update return parameters on all payment processors to match expected results

make static static

Merge pull request #23263 from eileenmcnaughton/import_wippier

 [Import] Rename dedupe rule id field from dedupe to dedupe_rule_id field

Remove another good intention - unused variables

100% of the responses from setActiveFields are discarded and 100% of the time
the erroneousField value is not used. Hence it makes sense to remove them....

Merge pull request #23269 from eileenmcnaughton/email

Remove preferred email format from inline edit

Merge pull request #23274 from civicrm/5.49

5.49 to master

Merge pull request #23271 from colemanw/searchKitTestFix

SearchKit - Fix aggregation issues & test regression

Simplify dataSource contract - remove unused preProcess

The choice of required functions was not great - postProcess passes silly
params & preProcess is never used / always empty - this removes preProcess
and gets rid of the abstract class for postProcess (which
will be decommissioned in a follow up)

[Import] Rename dedupe rule id field from dedupe to dedupe_rule_id field

It is still dedupe internally as those places will 'age out'

[REF] [Import] Clean up on map fields & preview templates in Contact import

This provides a bit of sanity to the MapTable.tpl which is included from
MapField and Preview - the focus being around getting
rid of all the section tags which rely on Count being assigned
and getting back to for-eaching the relevant arrays

SearchKit - Fix aggregation issues

In the UI, "Extra" fields like Contact.age_years were not being aggregated properly.

On the server, fixes mismatch when checking an aggregate field with a suffix.
Fixes recent test regression.

Use hard-coding when variable is unchangeable

Merge pull request #23245 from eileenmcnaughton/import_wip

Import - start to use civicrm_user_job to track user imports

Merge pull request #23264 from mlutfy/ui47scroll

dev/user-interface#47 Disable scrolling on time inputs

Merge pull request #23267 from civicrm/5.49

5.49 to master

Merge pull request #23265 from colemanw/pivotTablesFix

SearchKit - Fix selection of fields when creating a Data Segment

Remove preferred email format from inline edit

We removed other places in the UI where this shows up....

dev/core#3184 - Definitively load mainfiles during bootstrap

Merge pull request #23266 from colemanw/castToArray

APIv4 - Avoid php error

APIv4 - Avoid php error

Prevents a php error when calling `array_merge()` on `null`.

Merge pull request #23246 from jmcclelland/CustomCheckboxContributionImport

ensure checkbox custom fields on contributions import properly

SearchKit - Fix selection of fields when creating a Data Segment

Merge pull request #23059 from colemanw/pivotTables

SearchKit - Add data segmentation functionality

dev/user-interface#47 Disable scrolling on time inputs

Merge pull request #23159 from eileenmcnaughton/event

Minor simplification of getAmountLevelText

Merge pull request #23216 from eileenmcnaughton/import_more

[REF] [Import] Use shared file instead of now-identical files

Merge pull request #23190 from eileenmcnaughton/pradeep

Start to fix misnamed FinancialTrxn bao

Merge pull request #23213 from eileenmcnaughton/post

[REF] Do not pass form properties within the form

Merge pull request #23219 from eileenmcnaughton/import_cust

[REF] [Import] Standardise import text for CustomData import

Merge pull request #23224 from totten/master-signing-signage

System Check - Add a reminder about CIVICRM_SIGN_KEYS.

Merge pull request #23185 from eileenmcnaughton/smarty_wiz

Do not escape wizard class - procede straight to Hogwarts

Merge pull request #23195 from eileenmcnaughton/ds3

[Ref] [Import] [csv] Remove non-functional code

Merge pull request #23211 from eileenmcnaughton/event1

[REF] Fix test to use form flow / helper

Start to use user_job to track import

Initial  permission on UserJob

This adds
1) an acl so only the creator can access and
2) permits anyone with access CiviCRM to GET their
own jobs
3) permission on CREATE is set to administer civicrm

I am thinking that create might be too strict but
it might be better to start this way. Likewise we
probably want sysadmins to be able to access other
people's jobs but unless we have a plan now for
what permissions we want we can punt the question
by keeping these strict for now

[REF] [Import] Strict typing on function that throws an exception if null

Merge pull request #22805 from braders/permission_denied_wordpress_improvement-alternative

WordPress - Display site-theme/decorations on error screens

Merge pull request #23258 from civicrm/5.49

5.49 to master

Merge pull request #23251 from colemanw/managedEntityCleanup

Managed entity cleanup

Merge pull request #23257 from totten/master-error-test

E2E_Core_ErrorTest - Demonstrate whether error pages are well-formed

Merge pull request #23256 from colemanw/fixDownloadInvoiceButtonPort

Fix dev/core#3168 broken download contribution invoice button

Merge pull request #23164 from mlutfy/eventCancelWaitlist

Event self-service cancel: allow cancelling Waitlist registration

ensure checkbox custom fields on contributions import properly

Merge pull request #23247 from colemanw/searchKitAggFix

SearchKit - Prevent errors trying to orderBy nonaggregated columns

E2E_Core_ErrorTest - Demonstrate whether error-pages are well-formed

Fix dev/core#3168 broken download contribution invoice button

Merge pull request #23202 from eileenmcnaughton/pay_links

Fix button breakage on viewContribution

Merge pull request #23253 from eileenmcnaughton/imp_if

[REF] [Import] Obsolete if removal

Merge pull request #23222 from eileenmcnaughton/in_between

[REF] [Import] [Custom] Remove another in-between class

(REF) HttpTestTrait - Promote `assertBodyRegexp()` from `AllFlowsTest` to `HttpTestTrait`

SearchKit - Prevent errors trying to orderBy nonaggregated columns when using groupBy

Remove another in-between class

A universe search shows no other references

[REF] [Import] Obsolete if removal

This if was left in in the last patch for readability

ManagedEntities - Refactor class variable $declarations to local variable

Merge pull request #23183 from eileenmcnaughton/notice

Ensure tags fields are assigned to tpl for email form test

ManagedEntities - Refactor class variable $plan to local variable

Merge pull request #23215 from eileenmcnaughton/test_amount

Take a reality check on moneyCleaning in test