Merge pull request #18739 from eileenmcnaughton/ef

[Ref] Minor code extraction

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#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

Merge pull request #18721 from artfulrobot/artfulrobot-relcache-utf8

Remove explicit COLLATE utf8_bin from RelationshipCache trigger

CRM_Mailing_Info - Init Angular data via `settingsFactory` instead of `settings`

CRM_Mailing_Info - Extract method 'createAngularSettings()'

Merge pull request #18731 from colemanw/angularSettingsFactory

AngularLoader: Support 'settingsFactory' callbacks in angular modules.

Replace hard-coded utf8_bin with binary in-use collation

Extract setNextUrl

This code is repeated all over the task sub system. This is a first attempt to
extract it on the parent class in order to start consolidating it

Merge pull request #18747 from eileenmcnaughton/aip2

[REF] Replace long if block with early return

[REF] Replace long if block with early return

Merge pull request #18744 from eileenmcnaughton/aip

dev/financial#152 simplify passed parameters

Merge pull request #18743 from vingle/patch-9

More accurate language around social media sharing

dev/financial#152 simplify passed parameters

Rather than passing objects this switches to specifically passing the 2 objects being used in
the recur function

Merge pull request #18742 from jitendrapurohit/merge-uid

Fix main contact uf url on merge screen

Merge pull request #18738 from eileenmcnaughton/lines

dev/core#2115 remove reference to financialACLs in bounce

Merge pull request #18735 from eileenmcnaughton/aipn

dev/financial#152 simplify interaction with membership, deprecate function

Merge pull request #18736 from eileenmcnaughton/pp

dev/financial#152 simplify passed parameters

Merge pull request #18668 from eileenmcnaughton/no_opt

dev/core#2079 Eliminate unused query on CRM_Core_BAO_CustomQuery::_construct

Merge pull request #18589 from eileenmcnaughton/ex_class

[REF]  Separate export form classes out & simplify task handling

Change in language

Change in language to be more accurate, in relation to this issue: https://lab.civicrm.org/dev/user-interface/-/issues/21#note_48976.

Update Petition.php

Change in language to be more accurate, in relation to this issue: https://lab.civicrm.org/dev/user-interface/-/issues/21#note_48976.

Merge pull request #18555 from eileenmcnaughton/del_primary

dev/core#2047 Fix merge code so that deleted contacts are not left without a primary address

Update Settings.php

Change in language to be more accurate, in relation to this issue: https://lab.civicrm.org/dev/user-interface/-/issues/21#note_48976.

Fix main contact uf url on merge screen

[Ref] Minor code extraction

dev/core#2039 Fix merge code so that deleted contacts are not left without a primary address

This was picked up in tests to ensure that removing a line of code creating excessing queries
would not cause regressions.

https://lab.civicrm.org/dev/core/-/issues/2039

I considered altering the test to exclude deleted contacts but we run the risk that if a contact
is undeleted they will have no primary address so I figured the integrity makes sense.

Note there are a couple of queries in this code that can go (retrieving stuff
we already have) - depending how I go on getting review on this & related tidy up I'll remove them
in a later PR

AngularLoader: Support 'settingsFactory' callbacks in angular modules.

This allows Angular modules with complex/expensive data to provide it with a callback,
which will only be invoked if the module is actively loaded on the page.

Normally, module settings are calculated on every page request, even if they are not needed.

Merge pull request #18737 from eileenmcnaughton/ppp

dev/financial#152 remove unused parameter

dev/core#2115 remove reference to financialACLs in bounce

I believe that the other 2 criteria suffice. allMembershipDetails is built from
CRM_Member_BAO_Membership::buildMembershipTypeValues()  which is altered by
financial acls but also potentially by other extensions so I don't think the bounce
should be restricted to when financial acls are in play

dev/financial#152 remove unused parameter

dev/financial#152 simplify passed parameters

the  function now only requires  so this simplifies so it only receives that

dev/financial#152 simplify interaction with membership, deprecate function

ids['membership'] is only calculated in the Authorize.net IPN flow for the purpose of determining
the autoRenew membership parameter for recurringNotify. This is currently passed to sendRecurringStartOrEndNotification
but that does not add any simplification & cannot be used elsewhere - so deprecating.

This is tested in testIPNPaymentMembershipRecurSuccessNoLeakage

Merge pull request #18651 from eileenmcnaughton/cf

[REF] Minor simplification - don't use a variable for table name

Merge pull request #18728 from eileenmcnaughton/aip

dev/financial#152 Pass contribution directly to completeOrder

Merge pull request #18733 from eileenmcnaughton/aipn

dev/financial#152 [REF] Extract getMembershipID

Merge pull request #18734 from eileenmcnaughton/ppp

dev/financial#152 Remove unreachable code

Merge pull request #18727 from colemanw/entityTest

[Test] Ensure all APIv4 entities have basic info

dev/financial#152 Remove unreachable code

This code cann only be reached if txnType is recurring_payment. Given that we can look
at the earlier switch and see that in that case first must be TRUE or it would have
already returned. Ergo this code is unreachable - and would have been
since the earlier code switched to using the api to handle non-firsts

Merge pull request #18732 from eileenmcnaughton/fail

dev/financial#152 Remove unused parameters from BaseIPN->failed signature

dev/financial#152 [REF] Extract getMembershipID

Note that since NULL is valid we can just return the result of the query. Bypassing for
component=event doesn't provide enough value to justify relying on a url param
as opposed to just looking it up & feels very original-form-flow

Merge pull request #18730 from eileenmcnaughton/pp

dev/financial#152 Remove unused parameters from BaseIPN->cancelled signature

Merge pull request #18729 from eileenmcnaughton/aipn

dev/financial#152 Clean up & test contributionPageID handling

dev/financial#152 Remove unused parameters from BaseIPN->failed signature

Same as https://github.com/civicrm/civicrm-core/pull/18730 but for failed

Test: Ensure all APIv4 entities have basic info

Remove unused parameters from BaseIPN->cancelled signature

dev/financial#152 Clean up & test contributionPageID handling

Merge pull request #18726 from demeritcowboy/static-pdfformat

dev/core#2110 - Warning "Non-static method CRM_Contact_Page_AJAX::pdfFormat() should not be called statically" when changing the page format on print/merge document

dev/financial#152 Pass contribution directly to completeOrder

We have done quite a a bit of cleanup on this and the only value in objects now used is
contribution - this is really explict in the code as we actually unset objects after
extracting contribution.

This alters the function signature such that it receives contribution directly rather than
in an array

Merge pull request #18631 from eileenmcnaughton/ppp

dev/financial#152 [REF] Parse ids before sending to single function (minor simplification)

Merge pull request #18629 from eileenmcnaughton/aipn

dev/financial#152 [REF] Parse ids before sending to recur function (minor simplification)