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

APIv4 - Add Managed::reconcile() action.

Adds APIv4 wrapper around ManagedEntities->reconcile();

[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

SearchKit - Move loadFieldOptions function to searchMeta service for reusability

SearchKit - Support operators, custom fields & multiple fields in a searchSegment clause.

APIv4 - Pass entire Api4SelectQuery object to expression when rendering

This permits SearchSegment pseudo-fields to call $query->getField() on multiple
fields, including custom fields, while rendering.

SearchKit - Fix ORDER BY calculated fields when using GROUP BY

Before: Calculated field functions were repeated verbabim in the SELECT, ORDER BY & GROUP BY clause,
this would cause mySql error when using FULL GROUP BY mode.

After: Calculated field function defined in SELECT clause, alias used in ORDER BY & GROUP BY.
No error when grouping by an ordering by the same calculated field.

Side-benefit: When selecting and ordering by RAND(), the selected random numbers will be in order,
as it's now the same random function used for both instead of a different one.

SearchKit - Add basic SearchSegment functionality for numeric range groups

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

Add nodefaults to access.tpl

Merge pull request #23204 from totten/master-ext-msg

(dev/core#3171) "Manage Extensions" - Tweak messaging for "Install" screen

Apply nodefaults to row actions

Don't overescape links in pledge tab

Don't overescape tab links

"Manage Extensions" - Fix title display

1. Extensions sometimes display the `key` rather than the `<name>`.
   This happens because it only reads titles from the remote/download feed.
2. When the key is used, put some styling on it.
3. Escape from Los Smartalous

dev/core#3171 - Tune-down the pre-"Install" message

Standardize DataSource file for Contribution & Membership

SearchKit - Add SearchSegment entity & table

Merge pull request #23194 from eileenmcnaughton/ds2

Remove unused parameter from private function

Merge pull request #23200 from eileenmcnaughton/unused

Remove unused function

Merge pull request #23157 from eileenmcnaughton/anet

Minor readability improvement