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

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

Compare currencies as well as amounts on template Contribution change

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 #23286 from eileenmcnaughton/mapping

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

[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

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

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

Merge pull request #23237 from eileenmcnaughton/data_source

[REF] [Import] Stop overloading dataSource form

Merge pull request #23235 from eileenmcnaughton/acl

Fix sql error on acl user

Merge pull request #23242 from eileenmcnaughton/imp_var

[REF] [Import] Remove meaningless code

Merge pull request #23156 from eileenmcnaughton/pppro

Paypal pro, a.net ipn - do not update start date, status

Merge pull request #22959 from colemanw/managedRefactor

ManagedEntities - Allow targeted reconciliation. Add hook parameter. Simplify logic.

Merge pull request #23210 from eileenmcnaughton/cancel

Always use cancelSubscription form for cancelling recurring contributions

ManagedEntitiesTest - Add coverage for filtering by module

Merge pull request #23244 from eileenmcnaughton/wow

I have no words

I have no words

Merge pull request #23241 from eileenmcnaughton/import_abstract

[REF] [Import] remove pointless fini functions.

Merge pull request #23240 from eileenmcnaughton/import_codes

[REF] [Import] Remove warning count - never returned

[REF] [Import] remove pointless fini functions.

These never do anything - there are there only through copy & paste + a the abstract class requiring them.

In fact all the abstract functions are dumb - the are only called internally
within the classes so I have gotten rid of the abstractness so they
can be removed at will. import makes sense - but the signature doesn't...

Remove meaningless code

Merge pull request #23239 from colemanw/afformLineup

Afform - Improve dragging into dropzones

hook_managed - Accept option module filter

Merge pull request #23236 from eileenmcnaughton/mapField

[Ref] [Import] Remove another good intention....mapField

REF - Refactor ManagedEntities with the ability to reconcile a single module

Reconciling a single module (e.g. Afform) is more efficient.
Further refactoring of ManagedEntities for code simplification and efficiency.
Instead of fetching Managed records 3 times (for enabled, disabled and missing modules),
now they are fetched only once and sorted into the correct category.

[REF] [Import] Remove warning count - never returned

Afform - Improve dragging into dropzones

Stop overloading dataSource form

SearchKit - Refresh fields after saving a SearchSegment

SearchKit - Add UI for editing SearchSegments