Merge pull request #18493 from mlutfy/campaignReportDefaults

Fix default report permissions when creating reports from CiviCampaign

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

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

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

Remove caching that messes with tests without any time saving

Add supporting test

Merge pull request #18676 from seamuslee001/ref_remove_xml_util

[REF] Remove XML_Util dependancy within ewaysingle extension

[REF] Remove XML_Util dependancy within ewaysingle extension

Merge pull request #18673 from totten/master-gitignore

When exporting for composer-style deployment, exclude the `.gitignore` file

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

This fixes a few things
1) fixes apiQuery to be called statically as it is a static function
2) casts results directly to detail
3) removes unnused variable

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

This is called from Email.create already (which is how this creates emails) - to be cautious an extra
change ensures that if the primary email is deleled & another is promoted it is called

Merge pull request #18675 from eileenmcnaughton/uf2

dev/core#2087 move uf_match check above the primary check

Merge pull request #18674 from seamuslee001/form_buttons_key

[REF] Fix adding in the accessKey based on the button array

dev/core#2087 move uf_match check above the primary check

the uf_match check will result in an early return more often than the other

[REF] Fix adding in the accessKey based on the button array

Merge pull request #18143 from mattwire/membertabbuttons

Move membership tab add/submit membership buttons to PHP layer

Merge pull request #18611 from mattwire/activitytokensmodifieddate

Add modified_date to list of activity tokens

Merge pull request #18610 from demeritcowboy/error-recursion-recursion

dev/drupal#119 - Avoid crash from recursion on unhandled exceptions

gitattributes - When exporting for composer-style tarball, exclude gitignore

Merge pull request #18649 from seamuslee001/error_class_remove

Ensure that the crm-error class on the field is also remove when field is changed

Search ext: rename to Search Kit, mark as beta

Merge pull request #18671 from MegaphoneJon/fix-perm-labels

fix labels on new admin permissions

Merge pull request #17675 from agh1/aria-form-fixes

Select field fixes for screen reader

fix labels on new admin permissions

avoid crash from recursion on exceptions

[REF] Ensure that the crm-error class on the field is also remove when field is changed

Only handle crm-error class

Merge pull request #18644 from colemanw/savedSearch

Search ext: support complex joins & HAVING clause in api4 smart groups

Merge pull request #18670 from totten/master-compup

composer.json - Update compile-lib and compile-plugin

Updates for Angular 1.8 - fix premature access to component properties

Angular 1.8 doesn't seem to like it when you try to access component properties
before the init function.

Merge pull request #18647 from seamuslee001/widget_button_classes

[REF] Add in css classes to make the save and preview button on the C…

composer.lock - Update composer-compile-lib and composer-compile-plugin

composer.json - Update composer-compile-lib

Merge pull request #18669 from demeritcowboy/blank-target-contact

dev/core#2090 - Undefined index on contact's activity tab when there's an activity that has no With Contact

StateProvince DAO title should be State/Province

Select placeholders should always say what kind of thing they select

Cleanup with ??

Attributes for a form element should always be an array

handle missing array member

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

    This fixes
    1) fixes apiQuery to be called statically as it is a static function
    2) casts results directly to detail

This is the copy & paste source of https://github.com/civicrm/civicrm-core/pull/18664 I guess....

Search ext: Fix column appearing in the aggregate list when it is used in GROUP_BY

The bug was that it wasn't recognizing the column name after a function was added to it.

Search ext: Extend api4 smart groups to work with HAVING

Add unit test to cover pcp submission

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

Currently we do 2 queries per contribution create to retrieve pcp & soft credit info. However, the
results of the queries are only used if pcp or soft credit keys exist in the params array

Merge pull request #18666 from civicrm/5.30

5.30 to master

Merge pull request #18648 from demeritcowboy/missing-mail-help

dev/core#1920 - Fix smarty error and missing mail setting help

Merge pull request #18660 from totten/master-bundle-ext

(REF) Make it easier for extensions to define basic bundles

Search ext: Choose smart group contact column

Search ext: fix missing titles in multivalue custom entities