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

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

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

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

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

Search ext: Ability to load saved searches by id

(NFC) Update docblock in CRM_Core_Resources_Common

(DX) Resources - Add helper for creating new bundles

(REF) Convert hidden function 'useRegion' to public 'fillDefaults'

This mostly affects a hypothetical bundle defined outside of core's `Common.php`:

* Before: You cannot re-use `useRegion()`. You'd have to write it again.

* After: You can re-use and/or override `fillDefaults()`.

Merge pull request #18635 from colemanw/angular-1.8

dev/core#1818 Upgrade AngularJS from 1.5.11 to 1.8

Merge pull request #18636 from demeritcowboy/test-scalar-target

dev/core#2057 - Failing unit test for passing in activity contacts as scalars to Activity::create()

Merge pull request #18659 from totten/master-cssmin

dev/core#1393 - distmaker - Skip new file "vendor/bin/cssmin"

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

Our standard expectation is that each BAO will have a create action that handles a single entity. For legacy
reasons Address.create has special handling for multiple addresses. This extracts that handling
into a separate function (legacyCreate) and updates the (very few) places currently calling create to
call that.

With this merged apis v3 & v4 can call Address.create  - currently v3 is hard coded to call add and
v4 bypasses the BAO altogher. The v4 api bypass means we are not managing is_primay integrity on v4 api calls

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

The del function on each of the location DAO ensures that after a delete at least one phone (etc) is
marked is_primary. This blockDelete does not do that & per the test it's possible to call this
in such a way that no addresses are primary. Whether this actually happens is a bit unknown.

However, this cleans up block delete in a way where we can demonstrate the tests work

dev/core#1393 - distmaker - Skip new file "vendor/bin/cssmin"

Overview
----------------------------------------

This fixes the broken distmaker builds in `master`.

The file `vendor/bin/cssmin` was recently added by way of `composer-compile-lib => cssmin`.
We don't need it in the tarball, and it's a symlink, so trying to include breaks some
consumers (like in https://lab.civicrm.org/dev/core/-/issues/1393).

Technical Details
----------------------------------------

* Before (last release): `vendor/bin/cssmin` does not exist
* Before (current master): `vendor/bin/cssmin` is created as symlink, copied to tarball
* After (this commit): `vendor/bin/cssmin` is created on dev-builds but omitted from tarball

Merge pull request #18625 from eileenmcnaughton/activi

dev/core#2057 Remove extraneous activity contact queries on activity update

Merge pull request #18643 from eileenmcnaughton/cache

Fix cache bypass

Merge pull request #18656 from eileenmcnaughton/apiq

dev/core#2079 [REF] Remove reference to second param returned from CRM_Contact_BAO_Query::apiQuery

Merge pull request #18606 from eileenmcnaughton/create

dev/core#2046 Ensure location BAOs have create actions, deprecate add…

Merge pull request #18608 from eileenmcnaughton/block

[REF] Follow up cleanup from Event Location

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

The second value from apiQuery is calculated using an extraneous uncached query. This function
is the only function that calls that function AND puts that value into a variable. This change clarfies that
this function does not require thhe variable

[REF] Remove reference to second param returned from CRM_Contact_BAO_Query::apiQuery

Remove problematic tests

This removes 2 tests that broke with the upgrade to Angular 1.8.
Neither test was covering much of anything useful.

dev/core#1818 Upgrade AngularJS from 1.5.11 to 1.8

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

The biggest breaking change between 1.5 and 1.8 was the default hashPrefix.
See https://github.com/angular/angular.js/blob/master/CHANGELOG.md#location-due-to-1

Our solution is to force crmApp to use the previous default. This ensures that existing modules
which rely on crmApp for routing (which is nearly all of them) will be unaffected.

New modules like Afform which use their own routing should use the new default recommended by Angular.

Merge pull request #18655 from eileenmcnaughton/token2

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

Merge pull request #18652 from eileenmcnaughton/silly

Fix patently silly code

dev/core#2057 Remove extraneous activity contact queries on activity update

This changes it so that instead of doing either

1) if passing in an array of activity contacts for a given type then attempt
to delete existing contacts, regardless of whether they are the same as the
existing ones, create new ones

OR

2) if the param is an int then do a lookup, per record type, and update if needed to

- Do one query to find existing records of types to be deleted (for all 3 record types).
Only delete activity contact records that are not to be kept and
don't alter those records that should be retained

[REF] Fix some more calls to getTokens to make it clear only the first return value is used

Fix patently silly code

Fix insufficiently distinct cache key

Fix cache bypass

If the contact is not impacted by any acls then ids will be an empty array. However, currently
an empty array is treated as 'uncached' rather than 'cached by empty' which
would be represented by null

Merge pull request #18640 from eileenmcnaughton/syn

dev/core#2073 Improve cleanup on syntax conformance test

Merge pull request #18416 from seamuslee001/dev_core_2003

dev/core#2003 Use Brick/Money to ensure that we display all possible …

Merge pull request #18646 from totten/master-complib

(REF) Switch to composer-compile-lib

dev/core#2073 Improve cleanup on syntaxconformance test

Cleaning up all objects seems a bit too much - but ensuring we
re-set the DAO we create in this class should be a memory-friendly action

https://github.com/civicrm/civicrm-core/pull/18639

fix missing mail setting help

Greenwich - Switch to composer-compile-lib

[REF] Add in css classes to make the save and preview button on the Contribution Page widget settings form look consistant with the other buttons on the page

Merge pull request #18607 from eileenmcnaughton/website

dev/core#2046 Rationalise add vs create on website BAO

Merge pull request #18642 from seamuslee001/dev_core_339_custom_field

dev/core#334 Use the current in use collation and character sets when…

composer.json - Switch to composer-compile-lib. Add more helpers.

The overall effect of this change is to get access to a few more helpers.

Before: Load `composer-compile-plugin`, `scssphp`, and `php-autoprefxer` as
three distinct packages.

After: Load `composer-compile-lib`, which transitively requires all those others
*and* includes some helpers.

Sea also

* https://github.com/civicrm/composer-compile-plugin
* https://github.com/civicrm/composer-compile-lib

dev/core#2003 Use Brick/Money to ensure that we display all possible digits stored in the database for the option amount

Add in noisy notice about fall back due to missing intl extension

Merge pull request #18621 from eileenmcnaughton/prefix

dev/membership#27 Update outdated membership statuses in preProcess rather than submit

Merge pull request #18632 from eileenmcnaughton/leak

dev/core#2073 Memory drain

dev/core#334 Use the current in use collation and character sets when creating new custom value tables

Merge pull request #18599 from seamuslee001/frontend_group_fields

[REF] Add in frontend fields for title and description of group Schem…

[REF] Add in frontend fields for title and description of group Schema only change

Add in upgrade step to expand title column to be 255 as well

Update langauge as per Artfulrobot and require group title field

Merge pull request #18641 from eileenmcnaughton/will-the-real-slim-leaky-please-stand-up

dev/core#2073 Fix a real live leak

dev/core#2073 Fix a real live leak

I used the same methods I had been playing with on the test class on a contribution
import script and found that the object that was increasing in memory in tandem with
me increasing iterations was an instance of DB_result

I tracked it down to here. In essence our main DAO functions clean up after themselves - e.g
when the DAO is destructed it cleans up the globals it has created.

However, this older ->query() function does not do that and each time is is called increases it's memory hold.

It's also worth better caching in this function...

Merge pull request #18638 from eileenmcnaughton/test_perf

[Civi\Test] Fix test output noise

Merge pull request #18637 from demeritcowboy/no-details

dev/core#2075 - E_NOTICE viewing an activity that has no details contents

[Civi\Test] Fix test output noise

Seeing this is test logs

Warning: Invalid argument supplied for foreach() in /home/jenkins/bknix-dfl/build/core-18634-8zcf7/web/sites/all/modules/civicrm/Civi/Test/Schema.php on line 29

Call Stack:
    0.0007     530944   1. {main}() /home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar:0
    0.0498    9999808   2. PHPUnit\TextUI\Command::main() /home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar:615
    0.0498   10008600   3. PHPUnit\TextUI\Command->run() phar:///home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar/phpunit/TextUI/Command.php:162
    1.1684  104654872   4. PHPUnit\TextUI\TestRunner->doRun() phar:///home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar/phpunit/TextUI/Command.php:206
    1.2327  104791840   5. PHPUnit\Framework\TestSuite->run() phar:///home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar/phpunit/TextUI/TestRunner.php:652
 3300.8182  675188928   6. PHPUnit\Framework\TestSuite->run() phar:///home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar/phpunit/Framework/TestSuite.php:746
 3300.8199  675188928   7. CiviUnitTestCase::setUpBeforeClass() phar:///home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar/phpunit/Framework/TestSuite.php:703
 3300.8199  675188928   8. CiviUnitTestCase::_populateDB() /home/jenkins/bknix-dfl/build/core-18634-8zcf7/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:311
 3300.8199  675188928   9. Civi\Test\Data->populate() /home/jenkins/bknix-dfl/build/core-18634-8zcf7/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:305
 3300.8199  675189248  10. Civi\Test::asPreInstall() /home/jenkins/bknix-dfl/build/core-18634-8zcf7/web/sites/all/modules/civicrm/Civi/Test/Data.php:45
:

handle missing details

test activity contacts as scalars

Merge pull request #18596 from MegaphoneJon/core-2063

core#2063: Don't throw a warning when public ext directory isn't found

Merge pull request #18564 from seamuslee001/dev_core_1838

dev/core#1838 Ensure that no fatal error is triggered if you try to a…