Merge pull request #18818 from civicrm/5.31

5.31

Merge pull request #18822 from totten/master-prevnexttest

(NFC) dev/core#2029 - Make assertions in PrevNextTest more skimmable

Merge pull request #18740 from eileenmcnaughton/hook

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

(NFC) dev/core#2029 - Make assertions in PrevNextTest more skimmable

Overview
--------

The test appears to have a random failure. Making it more readable may help figure out why.

Before
------

`testDeleteByCacheKey()` and the related `testFillArray()` both have some assertions in these two forms:

```
// Form #1, more verbose
$all = ...getSelections($cacheKey, $action);
$this->assertEquals([...expected...], array_keys($all)...);

// Form #2, more pithy
$this->assertSelections([...expected...], $action, $cacheKey);
```

After
-----

The verbose form is replaced with the pithier form.

In the pithier form, some of the default inputs are made explicit.

Comment
-------

It is confusing that the method `getSelections()` has a parameter `$action`
which can be `get` or `getall` -- and that `getall` does not (in fact) return
selections.  It returns all!  (The fact that the contract is weird makes the
unit-test helpful imho...)

Merge pull request #18815 from eileenmcnaughton/recursoft

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

Merge pull request #18812 from eileenmcnaughton/en

dev/core#927 Enable contribution cancel actions extension

Merge pull request #18817 from seamuslee001/5-30-1-notes

release-notes/5.30.1.md

release-notes/5.30.1.md

Merge pull request #18805 from eileenmcnaughton/date

Rationalise date formatting

Enable contribution cancel actions

Once completed this extension will be unhidden and available to disable

Merge pull request #18811 from colemanw/savedSearchIndex

SavedSearch: add UI_name index to upgrade script

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

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

Merge pull request #18786 from eileenmcnaughton/cancelextwtest

dev/core#927 add first test on cancel

SavedSearch: add UI_name index to upgrade script

Merge pull request #18806 from eileenmcnaughton/constant

Update CRM_Utils_Constant::value to support env variables

Merge pull request #18804 from eileenmcnaughton/manual

Declare support for cancelRecurring in manual processor

Add initial test to the new extension

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

Merge pull request #18803 from eileenmcnaughton/canceltest

Remove always-true IF

Merge pull request #18784 from eileenmcnaughton/cancelext

dev/core#927 Add shell extension to move contribution cancel actions into

Rationalise date formatting

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

Merge pull request #18809 from colemanw/savedSearchName

SavedSearch - Add name and label columns

SavedSearch - Add name and label columns

Update CRM_Utils_Constant::value to support env variables

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)

Merge pull request #18645 from seamuslee001/frontend_group

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

Declare support for cancelRecurring in manual processor

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

Remove always-true IF

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'

Merge pull request #18801 from eileenmcnaughton/canceltest

Remove always true if

Merge pull request #18800 from agh1/5.31.0-releasenotes-initial

5.31.0 release notes initial run

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

Merge pull request #18790 from eileenmcnaughton/canceltest

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

Merge pull request #18798 from eileenmcnaughton/memtest

Replace BAO calls with api calls in test class

Merge pull request #18797 from eileenmcnaughton/apim

Switch to calling api

5.31.0 release notes: added boilerplate

5.31.0 release notes: raw from script

Merge pull request #18796 from eileenmcnaughton/api3

Switch to calling the api

Merge pull request #18785 from eileenmcnaughton/status

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

Merge pull request #18789 from jitendrapurohit/emailcc

Fix sendconfirmation api to override receipt params

Replace BAO calls with api calls in test class

Merge pull request #18795 from eileenmcnaughton/api

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

Switch to calling api

Test cover in
CRM_Member_BAO_MembershipTest::testMembershipUpdateDoesNotDeleteRelatedMembershipsByMistake

Merge pull request #18791 from eileenmcnaughton/statusm

Remove instances of variable variables

Switch to calling the api

Test cover in
api_v3_ContactTest.testMergeOrganizations

Merge pull request #18739 from eileenmcnaughton/ef

[Ref] Minor code extraction

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

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

Remove instances of variable variables

Minor code readability tidy up covered by EntryTest:testProcessMembership

Fix sendconfirmation api to respect cc and bcc set in params

Merge pull request #18788 from civicrm/5.31

5.31

Merge pull request #18540 from eileenmcnaughton/paypal

dev/core#2034 Fix paypal standard cancel url

Merge pull request #18766 from eileenmcnaughton/basic

Fix unreleased regression on search tasks from basic search

Merge pull request #18773 from eileenmcnaughton/url

dev/core#2066 Use shared function to set the next url

Merge pull request #18783 from eileenmcnaughton/distinct

dev/core#2066 Further cleanup on search actions

dev/core#927 Add shell extension to move contribution cancel actions into

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

dev/core#2066 Further cleanup on search actions

Most of the noise is in the shared pre-process code so my focus is on paring that down

Merge pull request #17568 from agileware/CIVICRM-1496

Contribution confirmation page should not display the name of payment processor type

Merge pull request #18778 from colemanw/plurals

Fix pluralize function for words like 'display'

Fix pluralize function for words like 'display'

Merge pull request #18777 from civicrm/5.31

5.31

CIVICRM-1496 Reword text on the contribution confirmation page

Merge pull request #18776 from seamuslee001/dev_core_2125

dev/core#2125 Ensure that the id that is used in the field is the gro…

Merge pull request #18775 from colemanw/searchRefactor

[REF] Search ext: Reorganize code into modules

dev/core#2125 Ensure that the id that is used in the field is the group id not the id of array key as that has changed from being a keyed array to a 2D array

Merge pull request #18411 from MegaphoneJon/pcp-wysiwyg

CRM-9362: PCP 'Your Message' should use WYSIWYG

Merge pull request #18400 from aydun/class_api_tweak_2

class.api.php: In remote api calls, allow referer and useragent to be set.

Merge pull request #18774 from civicrm/5.31

5.31 to master

Merge pull request #18765 from totten/5.31-res-alias

dev/core#2117 - Add compatibility aliases for 'scriptFile' and 'styleFile' resources

Merge pull request #18761 from MegaphoneJon/event43-test

Test for event#43

dev/core#2066 Use shared function to set the next url

Extends https://github.com/civicrm/civicrm-core/pull/18767 to more places

Search ext: Support is_deleted field but not api_key or hash

Search ext: Add settingsCallback to searchAdmin module

Search ext: Reorganize into searchAdmin and searchActions modules

Merge pull request #18770 from civicrm/5.31

5.31

Merge pull request #18768 from eileenmcnaughton/ids

dev/core#2066 Extract getSelectedIDs

Merge pull request #18767 from eileenmcnaughton/task

dev/core#2066 Use shared function to set the next url

Merge pull request #18763 from eileenmcnaughton/531

dev/core#2119 Fix mailing code to exclude deleted recipients

Merge pull request #18449 from colemanw/multiValueAutocomplete

Allow custom fields of type Autocomplete-Select to be multivalued

Merge pull request #18764 from civicrm/5.31

5.31

Merge pull request #18581 from demeritcowboy/requirements-check

dev/drupal#137 - Alternate PR - On drupal status report need different check when civi is already installed

dev/core#2066 Extract getSelectedIDs

dev/core#2066 Use shared function to set the next url

Reduces copy & paste - there are some more places like this to sort out after ....

Fix unreleased regression on search tasks from basic search

Test for event#43

Merge pull request #18762 from demeritcowboy/all-ALL

Typo in call to fixSchemaDifferencesForAll

Merge pull request #18758 from mlutfy/event43rc

dev/event#43 Contact Dashboard: fix Event Participations

dev/core#2119 Fix mailig code to exclude deleted recipients

dev/core#2117 - scriptFile, styleFile - Automatically register URL aliases

This allows backward compatibility for customizations which targetted these
resources by their URL.

dev/core#2117 - Resources - Allow each snippet to have `aliases`

__Before__: Every snippet/resource has a singular name. If the name ever changes,
it becomes a backward-compatibility break.

__After__: Every snippet/resource has a singular name, and it may optionally
have a list of aliases.  If the name ever changes, you can leave an alias
for backward-compatibility.

make sure we're `all` on the same page

Merge pull request #18757 from civicrm/5.31

5.31

Merge pull request #18754 from colemanw/angularPerms

Angular Loader: Allow modules to specify permissions to add client-side

dev/event#43 Contact Dashboard: fix Event Participations

Merge pull request #18753 from agh1/placeholderfix-5.31

dev/core#2118 supply better automatic placeholders for select (5.31 intra-rc)

Merge pull request #18756 from civicrm/5.31

5.31 to master

Merge pull request #18751 from demeritcowboy/utf8convert-triggers

Rebuild triggers after utf8mb4 conversion

Merge pull request #18750 from eileenmcnaughton/weird

Extract setNextUrl

Angular Loader: Allow modules to specify permissions to add client-side

Merge pull request #18749 from totten/master-crmMailing-factory

crmMailing - Only load Angular settings if they're needed

dev/core#2118 supply better automatic placeholders for select

rebuild triggers after utf8mb4 conversion