#2115 Move some more financialacl code to a hook within the extension

financial#152 move code only relevant to repeattransaction into that function

#927 Enable contribution cancel actions extension

Rationalise date formatting

Once completed this extension will be unhidden and available to disable

SavedSearch: add UI_name index to upgrade script

Per https://github.com/civicrm/civicrm-core/pull/18814 I dug into this addRecurSoftCredits and
it simply copies soft credits from the first transaction in the series to the new one. As such
it belongs in repeattransaction not the main body of completeOrder

#927 add first test on cancel 

Update CRM_Utils_Constant::value to support env variables

Declare support for cancelRecurring in manual processor

I made a small change to support the test - causing Membership.create not to set the
stattus_id if it is specified.

Remove always-true IF

#927 Add shell extension to move contribution cancel actions into

This came out of discovering a place in the code was 'accidentally formatted correctly'
and finding it far too hard to figure out the right way to specify the date formatting -
something that seems confirmed by finding various places with hard-coded date formattin
rather than using the setting

SavedSearch - Add name and label columns

This is something I discussed with @totten earlier but as the rc was being cut at the time
we went for the more conservative approach of only adding env support to thhe debug_query
function.  This extends to all defines - meaning they can be defined at the script level e.

env CIVICRM_DEBUG_LOG_QUERY=backtrace drush cvapi

(that already works)

Display public title and description on profiles and unsubscribe/subs…

Reviewer's partial from https://github.com/civicrm/civicrm-core/pull/18196

As can be seen by searching for calls to recurLinks recurID is not
an optional parameter for the call - so the IF is extraneous.

I've updated almost all the places that call it to
pass strict-type-casted int (or in one place ensured it already is)
as a bit of 'tightening'

Remove always true if

Since paymentProcessorObj is loaded as the manual processor where another does not exist
the processor condition is always true - so this if is obsolete

Add test for recurring links and clean up method of retrieving recurring

Replace BAO calls with api calls in test class

Switch to calling api

Switch to calling the api

unit test for #18306 - order create api test for contribution

Fix sendconfirmation api to override receipt params

[Test] - Fix some tests to call API not BAO

Test cover in
CRM_Member_BAO_MembershipTest::testMembershipUpdateDoesNotDeleteRelatedMembershipsByMistake

Remove instances of variable variables

Test cover in
api_v3_ContactTest.testMergeOrganizations

[Ref] Minor code extraction

Minor code readability tidy up covered by EntryTest:testProcessMembership

5.31