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

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

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

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)

Merge pull request #17969 from seamuslee001/widget_normal_route

dev/cloud-native#16 Migrate Contribution Page widget extern url to us…

Merge pull request #18725 from civicrm/5.31

5.31

Merge pull request #18717 from eileenmcnaughton/static2

dev/core#2108 fix deprecation notice take 2

Merge pull request #18713 from eileenmcnaughton/531

dev/core#2102 Fix hang on event edit

make routing endpoint static

Merge pull request #18723 from colemanw/crmSearchCleanup

Search ext: misc cleanup & fixes

dev/cloud-native#16 Migrate Contribution Page widget extern url to use the normal CMS routing mechanism

Update test to test both endpoints

Search ext: Ensure all entities have titlePlural to fix sort/display

Merge pull request #18719 from JMAConsulting/core_2105

core#2105: Groups children now get shown with SPAN CSS error

Merge pull request #18716 from eileenmcnaughton/static

dev/core#2108 fix deprecation notice

Search ext: misc cleanup & fixes

- Fixes white on white button text by adding btn-default class
- Fixes missing field names in the bulk-update action
- Decouples searchActions from the main search controller
- Misc cleanup

Merge pull request #18720 from artfulrobot/artfulrobot-core-issue-1428

dev/core#1428 Add test for Activity::update()

Merge pull request #18715 from seamuslee001/dev_financial_11

dev/financial#11 Expand unit test to ensure that the correct currency…

Add test for Activity::update()

core#2105: Groups children now get shown with SPAN CSS error

dev/core#2108 fix deprecation notice take 2

Loosens the deprecation notice - alternative to https://github.com/civicrm/civicrm-core/pull/18716 but
we should merge both IMHO

dev/core#2108 fix deprecation notice

This addresses the deprecation warning by moving the text to the tpl layer - we could ease up
on the deprecation for type = 'static' but the pattern didn't seem to make much sense or to be
consistent with how we normally do things

Merge pull request #18714 from MegaphoneJon/financial-111

Financial#111: Contribution tokens always display amount with default…

dev/financial#11 Expand unit test to ensure that the correct currency is outputted in the token

Merge pull request #18688 from seamuslee001/dompdf_upgrade

[REF] Upgrade DomPDF to v0.8.6

dev/core#2102 Fix hang on event edit

Per https://lab.civicrm.org/dev/core/-/issues/2102

Financial#111: Contribution tokens always display amount with default currency

Set version to 5.32.alpha1

Set version to 5.31.beta1

Merge pull request #18710 from eileenmcnaughton/locev2

dev/core#2096 Fix in-master-only regression on creating new events

Merge pull request #18708 from eileenmcnaughton/locev

Add unit test that ultimately failed to hit the desired code but does add cover

Merge pull request #18711 from seamuslee001/move_to_alpha1_stage

Move to alpha1 stage

[REF] Move recently added upgrade steps from 5.31.0 to 5.31.alpha1 step

dev/core#2096 Fix in-master-only regression on creating new events