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

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

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

Merge pull request #23228 from eileenmcnaughton/549_m

Fix malformed query when user with no access to any financial acls accesses civimember search

[REF] [Import] [Test] switch test to use full form flow

Remove non-functional code

Instantiating the importJob with a table name and no createTable
does nothing but set the name so that getTableName() gets it
- import_table_name is always set so these lines do nothing

[REF] Minor clean up

[Import] Simplify and limit dataSource retrieval

Remove another good intention....

Fix malformed query when user with no access to any financial acls accesses civimember search

Fix sql error on acl user

Merge pull request #23217 from eileenmcnaughton/import_act

Standardise activity datasource text

Merge pull request #23218 from eileenmcnaughton/import_part

Standardise participant import text

Merge pull request #23227 from eileenmcnaughton/inbetween_4

[REF] [Import] Remove last in-between parser class

Merge pull request #23221 from eileenmcnaughton/mapField

[Ref] [Import] Remove another 'in-between' class

Merge pull request #23223 from eileenmcnaughton/in_between2

[REF] [Import] [Contact] Remove in-between class

Merge pull request #23232 from braders/nodefaults-tab-links

Apply nodefaults to contact tabs for escape-on-output

Merge pull request #23231 from civicrm/5.49

5.49

Apply nodefaults to contact tabs for escape-on-output

Merge pull request #23229 from pradpnayak/limitcontacts

Limit issue fix

Merge pull request #23226 from MegaphoneJon/individual-employers

restore using non-organizations as employers

Merge pull request #23220 from totten/master-defn

(REF) CryptoRegistry - Fix type declaration

Merge pull request #23184 from totten/master-bd-drush

(NFC) Skip CliRunnerTest on php80+drush+Backdrop

Limit issue fix

Merge pull request #23080 from eileenmcnaughton/isset

Always assign pay_later_receipt

Merge pull request #23199 from eileenmcnaughton/imp

dev/core#1307 Add civicrm_user_job table

Merge pull request #22664 from braders/membershipview-default-values

Set default values on MembershipView to stop PHP notices

allow non-organization employers

Remove last in-between parser class

Unlike the others this does have non-core refs in civihr & csvimport -
I suspect these are copy & paste rather than functional but will patch

Merge pull request #23207 from braders/row-action-nodefaults

Apply nodefaults to row actions

Merge pull request #23208 from braders/access-wp-nodefaults

Add nodefaults to access.tpl

Merge pull request #23225 from braders/edit-group-nodefaults

Add nodefaults to group edit form

Add nodefaults to group edit form

Overview
--------

The setting `CIVICRM_SIGN_KEYS` was introduced circa 5.36.  However, it is
defined in `civicrm.settings.php`, which makes it difficult to reliably
configure in an automated upgrade.  Consequently, some sites may not have
this setting, and we must rely on the sysadmin to provide it.

The setting is required for the `crypto.jwt` API (which in turn is used by
some core extensions, like `authx` and `afform`).

Before
------

There is a pre-upgrade message when somebody passes through v5.36.

If you missed the message in 5.36, then you would be unaware of the missing
setting (until you hit some failure because you use some new/update
code-path that relies on it).

After
-----

There is a system status-check.  If you don't have `CIVICRM_SIGN_KEYS`, then
it will show a link to https://docs.civicrm.org/sysadmin/en/latest/setup/secret-keys/.

Remove another 'in-between' class

As with Contribution & Membership there is a pointless in-between class that
is only inherited by one class...

Remove in-between class

A universe search finds no other references

(REF) CryptoRegistry - Tighten type declaration

Standardise import text for CustomData import

Standardise participant import text

Standardise activity datasource text

Use shared file instead of now-identical files

Merge pull request #23186 from eileenmcnaughton/import_more

Standardize DataSource file for Contribution & Membership

dev/core#1307 Add civicrm_import_job table

Take a reality check on moneyCleaning in test

Merge pull request #23209 from civicrm/5.49

5.49

[REF] Do not pass form properties within the form

This function was obviously separated from shared code - so
now we can use this within the function....

Merge pull request #23212 from colemanw/noMoreDrupal6

Remove code only needed by Drupal6

Always use cancelSubscription form for cancelling recurring contributions

This is https://github.com/civicrm/civicrm-core/pull/18196 without the bit I was uncomfortable with

Remove code only needed by Drupal6

Followup from https://github.com/civicrm/civicrm-core/pull/22960

Merge pull request #23206 from braders/feature/pledge-payment-nodefaults

Don't overescape links in pledge tab

Fix test to use form flow / helper

Merge pull request #23160 from eileenmcnaughton/ev

(REF) Fix test to use full flow

Merge pull request #23205 from braders/tablink-nodefaults

Don't overescape tab links

Merge pull request #23198 from eileenmcnaughton/549

Move 5.49 update to beta