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

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 participant import text

Standardise activity datasource text

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

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

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

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

Merge pull request #23203 from civicrm/5.49

5.49 to master

Merge pull request #23177 from JMAConsulting/booleanFields_1

(REF) Schema - Correct boolean fields in various tables

Fix button breakage on viewContribution

Merge pull request #23201 from civicrm/5.49

5.49