Enotice fixes on ContributionPage dash

Merge pull request #22287 from eileenmcnaughton/process

[REF] Stop passing this as form, set in function

[REF] Stop passing this as form, set in function

Merge pull request #21611 from eileenmcnaughton/cancel

Add workflow template for recurring cancelled

Merge pull request #22229 from colemanw/searchKitExport

SearchKit - Import/Export saved search configuration

Merge pull request #22284 from eileenmcnaughton/process

[REF] Remove now non-variable variables from previously shared code

Merge pull request #22283 from eileenmcnaughton/notice

Cleanup Smarty e-notices on campaign dashboard

[REF] Remove now non-variable variables from previously shared code

SearchKit - Add Import dialog

Move crmDialog angular utility from message_admin extension to core for reusability

SearchKit - Rename searchList template to match controller, move "New Search" button to top

SearchKit - Add Export dialog

Afform - Be more forgiving about input format when saving an afform

SearchKit - Faster delete/revert actions in search listing

SearchKit - Use dropdown for actions in search listing

The list of actions is getting too big for a row of buttons...

Merge pull request #22276 from eileenmcnaughton/process

[REF] Duplicate & unshare processFormContribution

Cleanup Smarty e-notices on campaign dashboard

Merge pull request #22282 from colemanw/cgExtends

CustomGroup - Add pseudoconstant for extends column

CustomGroup - Add pseudoconstant for extends column

Merge pull request #22265 from eileenmcnaughton/ver

dev/core#2996 Increment php recommndations

Merge pull request #22275 from eileenmcnaughton/recur_test

dev/core#567 Add test shell

Merge pull request #22278 from demeritcowboy/unused-function

Remove unused function CRM/Utils/Mail/EmailProcessor::cleanupDir

Merge pull request #22279 from eileenmcnaughton/refs

Remove unnecessary pass-by-ref

Remove unnecessary pass-by-ref

unused function

dev/core#567 Add test shell

[REF] Duplicate & unshare processFormContribution

The only way I've managed to break up these big toxic functions is divide & conquer. While it seems
counter-intuitive to not doing things in more than one place in practice much of the
code is form-relevant and the the code before, during and after these functions
does a whole lot of extra work to share stuff that they don't really have in common.

This dates back pre-api when the only copy of the business logic often was on the forms....

Merge pull request #22272 from eileenmcnaughton/anet

[NFC] Cleanup in Authorize.net test class

Merge pull request #21927 from eileenmcnaughton/trxn

Move two more functions to financialProcessor

Merge pull request #22233 from colemanw/apiExplorerArgs

APIv4 Explorer - Make selected language and format bookmarkable

Add workflow template for recurring cancelled

This is a simple template, with a slightly smaller subset of values than the exisiting one.

It pulls out the recurring part to a trait...

[NFC] Cleanup in Authorize.net test class

The main goal in this update is to switch the contribution form to use the full form flow.
Other changes are just code style

Merge pull request #22266 from eileenmcnaughton/cont_tests

Fix test to use full form flow

Merge pull request #22269 from demeritcowboy/make-clean

[NFC] - Try to work around failing tests

Merge pull request #22271 from civicrm/5.45

5.45

Merge pull request #22270 from colemanw/relatedContactsLabel

Fix related contacts label in search kit

Fix related contacts label in search kit

work around previous test fails

Fix test to use full form flow

Move two more functions to financialProcessor

Increment php recommndations

Merge pull request #22107 from demeritcowboy/test-isdir

dev/core#2927 - Avoid warnings for is_dir() when open_basedir is in effect

Merge pull request #22236 from mlutfy/angularCacheCodeLanguage

Resources: set language-specific cacheCode, fixes AngularJS translation with multilingual

Merge pull request #21762 from jitendrapurohit/job-alert

Add system status warning to display scheduled job failures

add ts

Merge pull request #22261 from eileenmcnaughton/prof

[REF] Remove more params that are unused now function is not shared

[REF] Remove more params that are unused now function is not shared

Merge pull request #22260 from eileenmcnaughton/prof

Remove handling for always-truthy var being false

Merge pull request #22259 from eileenmcnaughton/update

Rename protected function

Remove handling for always truthy var being false

In a previous unshared life the code in this function supported contact being null

However, in it's unshared version contactID is always an integer (in the merge context). This strips
out the now-irrelevant code.

Further cleanup to follow ... it's not clear this function is doing
anything of value at all but some unwravelling required

Merge pull request #22258 from eileenmcnaughton/prof

Remove never passed variables

Rename protected function

This function was originally shared & hence the name made sense but in the context of dedupe
it is only used to do updates so renaming to make that clear & making required nature
of contactID clear too

Remove never passed variables

This function is no longer shared so is only called from one place. That place doesn't use
contactDetails so don't return. Also never sets skipCustom so stop checking

(will remove the other non-passed soon)

Merge pull request #22254 from eileenmcnaughton/prof

[REF] Duplicate function to allow us to work it out of the code

Merge pull request #22257 from aydun/mr-info

(NFC) APIv4: Add help info for multi-record custom field sets

Add help info for multi-record custom field sets

Merge pull request #22183 from eileenmcnaughton/smarty9

Smarty variables]  Remove another isset - deferredFinancialType

Merge pull request #22256 from eileenmcnaughton/order

Do not default-escape weight field on order

Do not default-escape weight field on order

Merge pull request #22253 from eileenmcnaughton/rel5

[REF] Minor parameter simplification

[REF] Duplicate function to allow us to work it out of the code

Merge pull request #22251 from eileenmcnaughton/rel3

[NFC] Test cleanup

[REF] Minor parameter simplification

The only value in this array used in the function is 'custom' which should be an array. I think it woul
d only ever be an array or NULL (which would then be an array with this change) but I left
a deprecation notice in case. Pass-by-ref isn't needed

Merge pull request #22249 from eileenmcnaughton/rel

[NFC] Minor cleanup in test class

Merge pull request #22252 from eileenmcnaughton/escapeit

Move require_once for smarty modifier due to order issues

[NFC] Test cleanup

This biggest part of this is relying on the tearDown to do the cleanup more. This is the right
way because if the test fails the cleanup won't happen if it is in the test itsef

Merge pull request #22243 from demeritcowboy/quickfix-reportedby

dev/core#2825 - Make source contact required for activities on the form

Move require_once for smarty modifier due to order issues

[NFC] Minor cleanup in test class

Merge pull request #22248 from MattTrim1/patch-1

Contributor key: Add Matt Trim

Merge pull request #22244 from MattTrim1/flip-default-display-custom-group-options

dev/core#2977 - For custom group creation, flip the default display settings

Contributor key: Add Matt Trim

Merge pull request #22246 from mattwire/money0.5

Update Brick/Money to 0.5.3

Merge pull request #22247 from mlutfy/fixConfirmRecur545

Contribution Confirm: fix wording on the recurring

Contribution Confirm: fix wording on the recurring

Update brick/money to 0.5.3

For custom group creation, flip the default display settings

make source contact required

Resources: set a cacheCode that is language-specific, fixes AngularJS translation with multilingual

Merge pull request #22240 from colemanw/nirodaonline-master

Contact/BAO/Query.php: fix searching for whitespace

avoid flooding logs when open_basedir is used

Merge pull request #22213 from colemanw/afformApi4

[REF] Afform - Use APIv4 for managed dashboard

Contact/BAO/Query.php: fix searching for whitespace

Condition was never true

Add Unit test for space wildcard replacement on sort_name

fix failing test

Merge pull request #22096 from yashodha/dev-2962

(dev/core#2962) get contact id for the mailing along with display nam…

Merge pull request #22225 from colemanw/upgradeTemplateFix

Simplify autogenerated boilerplate upgrader class

Merge pull request #22238 from colemanw/upgradeVerBump

Upgrade - Bump minimum upgradable version to 4.6.12

(NFC) CRM_Upgrade_Incremental_Base - Expand docblocks. More references.

Remove unnecessary legacy version check

Now that the minimum upgradable version is > 4.6, this check is not needed.

Merge pull request #22186 from eileenmcnaughton/smarty12

Smarty variables]  Another isset bites the dust

Bump minimum upgradable version to 4.6.12

Merge pull request #22237 from totten/master-verifydb

(REF) CRM/Upgrade - Remove unused entrypoint `verifyPreDBstate()`

(REF) CRM/Upgrade - Remove unused entrypoint `verifyPreDBstate()`

Overview
--------

In olden times, upgraders sometimes implemented the method `verifyPreDBstate()`. However, this appears to be unused.
Additionally, it's not clear (from the documentation) how you would use it (if you wanted to).

So...  simplify the code....

Before
------

Function exists. But it does nothing. Because no subclasses implement it.

After
-----

Function does not exist. It still does nothing.

Simplify autogenerated boilerplate upgrader class and clarify comments

Merge pull request #22187 from eileenmcnaughton/smarty13

Smarty variables]  Remove isset from batch form

Remove isset from batch form

Affects url civicrm/financial/financialbatches?reset=1&batchStatus=1

Merge pull request #22234 from colemanw/afformDateRange

Afform - Set date-range defaults

Merge pull request #22232 from civicrm/5.45

5.45

Merge pull request #22226 from colemanw/deleteBoilerplate

NFC - Delete boilerplate comments and empty functions from upgrade classes

Merge pull request #22193 from eileenmcnaughton/smarty18

[Smarty variables] remove isset from merge screen

Merge pull request #22231 from colemanw/apiDisabledComponent

dev/core#2984 - Clarify API error when component is disabled