Merge pull request #18846 from eileenmcnaughton/ornery

dev/core#2029 exclude ornery test

Merge pull request #18837 from eileenmcnaughton/minor

[REF] Determine values where they are needed rather than passing them  around (in tested function)

[REF] Determine values where they are needed rather than passing them around

The membershipTypes array is constructed, but barely used. One place it is used
is in this function - which is explictly tested by CRM_Member_Form_MembershipTest::testSubmit

This fixes it to get the values when needed rather than construct a variable miles away from
where it is used - no more pass the parcel

Merge pull request #18847 from civicrm/5.31

5.31 to master

Merge pull request #18799 from deb1990/CPS-332-support-new-button-markup

CPS-332: Button markup fix in PremiumBlock.tpl

Merge pull request #18843 from bluehorndigital/dev/core-2140

dev/core#2140 Fix site path for Drupal 8 and 9

dev/core#2029 exclude ornery test

E2E.Core.PrevNextTest.testDeleteByCacheKey

dev/core#2140 Fix site path for Drupal 8 and 9

This fixes hardcoded assumptions of the site path. This breaks testing
integration, which has sites in a subdirectory of `simpletest`

Merge pull request #18839 from civicrm/5.31

5.31

Merge pull request #18821 from eileenmcnaughton/531im

dev/membership#30 switch Membership import to use the v3 api

Merge pull request #18836 from civicrm/5.31

5.31

Merge pull request #18834 from demeritcowboy/file-add

Fix deprecated passing of string attributes to CRM_Core_Form::add for contact import

Merge pull request #18831 from seamuslee001/dev_core_2136

dev/core#2136 Fix the loading of groups on the contact Individual screen

Merge pull request #18820 from eileenmcnaughton/531

More buttonrama fallout

Merge pull request #18829 from eileenmcnaughton/aip

[REF] Minor extraction

deprecated string

Merge pull request #18826 from eileenmcnaughton/extract

Remove a few lines of deprecated code

Fix the loading of groups on the contact Individual screen

More buttonrama fallout

Merge pull request #18827 from yashodha/activity_report

(core/dev#2104) Add birth date column for target contact to activity …

Minor extraction

Merge pull request #18825 from seamuslee001/en2

[REF] Include contributioncancelactions extension in dismaker and reg…

Merge pull request #18814 from eileenmcnaughton/recur_out

dev/core#927 Remove code to update recur on cancel & fail where there is no recur

Merge pull request #18746 from eileenmcnaughton/aip

[REF] Extract handling for loading contribution recur object.

(core/dev#2104) Add birth date column for target contact to activity report

Remove a few lines of deprecated code

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

[REF] Include contributioncancelactions extension in dismaker and regenerate civicrm_generated

Merge pull request #18740 from eileenmcnaughton/hook

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

dev/membership#30 switch Membership import to use the v3 api

ITturns out the addition of a unique name at some point caused is_override not to be passed in
inn a way the BAO recognises - the api has handling for this...

(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

dev/core#927 Remove code to update recur on cancel & fail

I've been racking my brains as to why we do this and it just feels like copy & paste that has survived forever

This code is called only from the 2 PaypalIPN classes. In both cases the classes follow the same flow - ie

if the payment is part of a recurring series, then call recur
if the payment is NOT the first in the series then call repeattransaction
only proceed to the 'single' function if we are confirming the
first recurring payment or it is not recurring

This means the cancelled & failed functions can only be reached when the flow is not
recurring OR the payment is the first - in neither case does it make sense to copy date
from one payment in the series to others - I'm pretty sure this precedes many many code changes

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)

CPS-332: Button markup fix in PremiumBlock.tpl

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