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

Merge pull request #18723 from colemanw/crmSearchCleanup

Search ext: misc cleanup & fixes

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

[NFC] Regen after #18624

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

Merge pull request #18624 from compucorp/CATL-1613-skip-non-case-email

Add options to Mail Account settings to improve inbound mail processing

Merge pull request #18360 from colemanw/fixMultiInProfile

Fix way of identityfing custom serialized fields

[REF] Upgrade DomPDF to v0.8.6

Merge pull request #18324 from mattwire/payproc_configoptional

Display url_site and url_recur based on if the form elements exist

Merge pull request #18701 from eileenmcnaughton/leak7

dev/core#2073 Fix use of legacy leaky method in tested code

Merge pull request #18707 from eileenmcnaughton/notice

Enotice fix

Merge pull request #18702 from eileenmcnaughton/leak8

dev/core#2073 Fix use of legacy leaky method in tested code

Merge pull request #18704 from mlutfy/partListingOutput

ParticipantListing Report: only display the View link for web, unhardcode others

Merge pull request #18699 from eileenmcnaughton/leak5

dev/core#2073 Fix use of legacy leaky method in tested code

Merge pull request #18493 from mlutfy/campaignReportDefaults

Fix default report permissions when creating reports from CiviCampaign

Enotice fix

Merge pull request #18706 from civicrm/5.30

5.30

Merge pull request #18680 from eileenmcnaughton/x

dev/financial#151 Paypal express validation fix on event

Merge pull request #18705 from agh1/5.30.0-releasenotes-final

5.30.0 release notes: updated with late changes

Merge pull request #18700 from eileenmcnaughton/leak6

dev/core#2073 Fix use of legacy leaky method in tested code

Merge pull request #18698 from seamuslee001/test_dev_core_2053

[NFC] dev/core#2053 Extend nextedGroup test to show that it will retu…

5.30.0 release notes: updated with late changes

dev/core#2073 Fix use of legacy leaky method in tested code

This is too low volume to really leak but it uses the leaky legacy method and
has test cover in

CRM_Contact_Import_Form_MapFieldTest.testSubmit with data set basic_data
CRM_Contact_Import_Form_MapFieldTest.testSubmit with data set save_mapping

ParticipantListing Report: only display the View link for web, unhardcode others

Merge pull request #18654 from eileenmcnaughton/token1

dev/core#2079  [REF] Fix some more calls to getTokens to make it clear only the first return value is used

Merge pull request #18653 from eileenmcnaughton/token

dev/core#2079 [Ref] Clarify calls to CRM_Utils_Token::getTokenDetails

dev/financial#151 Paypal express validation fix on event

extension of https://github.com/civicrm/civicrm-core/pull/18459

dev/core#2073 Fix use of legacy leaky method in tested code

Overview
----------------------------------------
Fix use of legacy leaky method in tested code

Before
----------------------------------------

After
----------------------------------------

Technical Details
----------------------------------------
Has test cover in
CRM_Member_Form_MembershipTest.testSubmitUpdateMembershipFromPartiallyPaid
CRM_Member_Form_MembershipTest.testMembershipViewContributionOwnerDifferent

Comments
----------------------------------------

dev/core#2073 Fix use of legacy leaky method in tested code

Overview
----------------------------------------
Fix use of legacy leaky method in tested code

Before
----------------------------------------
dao->query()

After
----------------------------------------
CRM_Core_DAO::executeQuery()

Technical Details
----------------------------------------
This is too low volume to really leak but it uses the leaky legacy method and
has test cover in

CRM_Import_DataSource_CsvTest.testToCsv with data set #0
CRM_Import_DataSource_CsvTest.testToCsv with data set #1
----------------------------------------

dev/core#2073 Fix use of legacy leaky method in tested code

Overview
----------------------------------------
Fix use of legacy leaky method in tested code

Before
----------------------------------------
dao->query()

After
----------------------------------------
CRM_Core_DAO::executeQuery()

Technical Details
----------------------------------------
This is too low volume to really leak but it uses the leaky legacy method and
has test cover in

CRM_Contact_Page_View_UserDashboard_GroupContactTest.testBrowseDisplaysCorrectListOfAddedGroups
CRM_Contact_Page_View_UserDashboard_GroupContactTest.testBrowseDisplaysCorrectListOfAVailableGroups
CRM_Contact_Page_View_UserDashBoardTest.testDashboardContentEmptyContact
CRM_Contact_Page_View_UserDashBoardTest.testDashboardContentContributionsWithInvoicingEnabled
CRM_Contact_Page_View_UserDashBoardTest.testDashboardContentContributions

----------------------------------------

[NFC] dev/core#2053 Extend nextedGroup test to show that it will return child groups that are Mailing groups even when the parent isn't a mailing group

Merge pull request #18667 from eileenmcnaughton/uf

dev/core#2087 Remove extraneous  CRM_Core_BAO_UFMatch::updateUFName call

Merge pull request #18696 from colemanw/btnNoConflict

Greenwich - fix conflict btw bootstrap & jQuery UI button

Merge pull request #18556 from eileenmcnaughton/pcp

dev/core#2056 Only retrieve pcp & soft_credit info when needed

Merge pull request #18695 from colemanw/ids

Exclude api4 from IDS check

Merge pull request #18500 from eileenmcnaughton/loc

dev/core#2047 [Ref] Remove obsolete check

Merge pull request #18650 from eileenmcnaughton/perms

Fix complexity on cache key

Merge pull request #18663 from eileenmcnaughton/blockdel3

dev/core#2039 Remove extraneous location queries from contact.create->Location::block path

Merge pull request #18662 from eileenmcnaughton/blockdel2

dev/core#2017 Deprecate an unused function

Merge pull request #18693 from eileenmcnaughton/leak4

dev/core#2073 Fix memory leak in well tested code

Merge pull request #18683 from colemanw/searchClause

Search ext & APIv4 Explorer: Refactor clause from directive to component

Greenwich - fix conflict btw bootstrap & jQuery UI button

This calls noConflict on the Bootstrap3 $.fn.button() method, essentially getting rid of it,
as it is not used by CiviCRM, whereas the jQuery UI $.fn.button() is used.

If we needed the boostrap button function, we could rename it with something like:
`$.fn.btn = $.fn.button.noConflict();`

Exclude api4 from IDS check

Search ext & APIv4 Explorer: Refactor clause from directive to component

Improves compatibility with Angular 1.8; uses the more modern component system
with less coupling and cleaner interfaces.

Merge pull request #18140 from mattwire/removetimeoutpermlocation

Remove ajax timeout from contribution page on behalf of

Merge pull request #18682 from eileenmcnaughton/add4

dev/core#2046 Fix api to call Address.create now it is standardised

Merge pull request #18690 from seamuslee001/dev_translation_54

dev/translation#54 Ensure that Component titles are translated in lin…

Merge pull request #18692 from eileenmcnaughton/leak1

dev/core#2073 Remove memory leak in heavily tested (merge) code

CATL-1613: Add new field to the mail account form - Skip emails which do not have a Case ID or Case token

CATL-1614: Add new field to the mail account form - Do not create new contacts when filing emails

Merge pull request #18694 from seamuslee001/master

5.30

Merge in 5.30

Merge pull request #18689 from seamuslee001/nfc_dao_update

[NFC] Update Checksum on CustomField DAO

dev/core#2073 Fix memory leak in well tested code

Example test

api_v3_ContactTest::testSortLimitChainedRelationshipGetCRM15983

Merge pull request #18685 from seamuslee001/ab_test_fix

[REF] Fix error when creating new AB test mailing because domain_id i…

dev/translation#54 Ensure that Component titles are translated in link text

[NFC] Update Checksum on CustomField DAO

dev/core#2073 Remove memory leak in heavily tested (merge) code

There are dozens of tests that pass through these lines - I used api_v3_JobTest:testBatchMerge to step through it
- note the dao->affectedRows uses a query to determine that - so it is no more or less accurate than before

Merge pull request #18623 from eileenmcnaughton/rene

dev/core#1790 - Contact Card - Email Links

[REF] Fix error when creating new AB test mailing because domain_id is a required field

Merge pull request #18684 from eileenmcnaughton/block

dev/core#2039 Call del directly - rather than now-deprecated blockDelete

Merge pull request #18672 from colemanw/searchKit

Search ext: rename to Search Kit, mark as beta

dev/core#2046 Fix api to call Address.create now it is standardised

Switches  api from calling Address.add to Address.create as part of BAO standardisation on create.

This has no material impact - just standardisation

dev/core#2039 Call del directly - rather than now-deprecated blockDelete

blockDelete is now just wrapper for del so this is really NFC

dev/core#2046 Remove extraneous primary handling

This builds on https://github.com/civicrm/civicrm-core/pull/18661 and removes extra primary handling

We can feel confident this is not needed as we have done a handful of rounds of tidy up & added tests
to ensure that for all 4 entities (email, phone, im & openid) potentially created through this
function both the create and the delete functions take action to ensure that the contact
is left with exactly one primary address at the end of this process

dev/core#2039 Remove extraneous location queries from contact.create->Location::block path

Overview
----------------------------------------
Removes 10 queries from each contact  create that are otherwise dealt with elsewhere
Updated version of https://github.com/civicrm/civicrm-core/pull/18477

Before
----------------------------------------
After creating location blocks 2 queries are done for each of the 5 entities to ensure is_primary fields have not been messed up (ie contact has 1 or more phones but none are marked primary or contact has more than one primary phone).

After
----------------------------------------
Queries are removed as location entities are already created correctly

Technical Details
----------------------------------------
This PR is part of a series of PRs to make this possible - best documented at https://lab.civicrm.org/dev/core/-/issues/2039#note_47758

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

Merge pull request #18679 from seamuslee001/vardump_d9_compatability

[REF] Fix compatability with Drupal 9 installing of var_dumper

Merge pull request #18658 from eileenmcnaughton/address

dev/core#2046 Migrate BAO_Address::create towards standardisation

Merge pull request #18661 from eileenmcnaughton/blockdel

dev/core#2046 Fix blockDelete to delete while ensuring is_primary is valid

Merge pull request #18664 from eileenmcnaughton/list

dev/core#2079 [REF] clean up call to apiQuery

Merge pull request #18677 from pradpnayak/float

change civicrm_price_set.min_amount to float

[REF] Fix compatability with Drupal 9 installing of var_dumper

Merge pull request #18678 from demeritcowboy/class-typos

dev/core#2093 - Fix red error box on new individual form and ltrim typos and doubling-up of class attribute

ltrim typos

change civicrm_price_set.min_amount to float

Merge pull request #18665 from eileenmcnaughton/list2

dev/core#2079 [REF] clean up call to apiQuery

Merge pull request #18657 from eileenmcnaughton/token_2

dev/core#2079 do not return never-used, query-wasting variable