Attributes for a form element should always be an 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

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

[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

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…

Merge pull request #18544 from magnolia61/participant_status_notification_default

Set participant status notification to false by default

dev/core#1838 Ensure that no fatal error is triggered if you try to access a merged contact and the contact that it was merged into has been permanently deleted

Move exception capturing into the page layer and extend unit test to cover page function

Update status message as per Mikeý's comment

Merge pull request #18633 from eileenmcnaughton/leak2

dev/core#2073 just tweak a memory leak

dev/core#2073 Memory leak fix

dev/core#2073 just tweak a memory leak

This one is like https://github.com/civicrm/civicrm-core/pull/18632 but only
seems to have switched us from an xml leakage to a string leakage - it still
seems slightly better from a code quality point of view

Merge pull request #18630 from eileenmcnaughton/pps

[REF] Parse ids before sending to single function (minor simplification)

Merge pull request #18619 from demeritcowboy/contact-test

[NFC/Test] Unit test activity-contact variations

Merge pull request #18570 from mlutfy/pcpLinksHook

dev/core#2061 PCP: call the hook_civicrm_links hook for PCP page user actions

Merge pull request #18626 from demeritcowboy/why-no-id

[Test framework] Wrong group id in mailing test setup

Merge pull request #18627 from civicrm/5.30

5.30

Merge pull request #18628 from seamuslee001/dev_core_2035

dev/core#2035 Add in upgrade step to fix the template copy on current…

[REF] Parse ids before sending to single function (minor simplification)

Overview
----------------------------------------
Minor simplification - parse ids into valid format & pass them into single, rather than passing them in wrong & 'asking' singleto parse them

Before
----------------------------------------
parsing of ids into usable array done at the last minute, other values passes around unnecessarily

After
----------------------------------------
parsing done close to the point where they are derived

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

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

Merge pull request #18613 from mattwire/moveloadrelatedobjects

Simplify call to loadRelatedObjects in repeat/completetransaction

dev/core#2035 Add in upgrade step to fix the template copy on current installs

Merge pull request #18602 from eileenmcnaughton/msg

dev/core#2035 Invoice template - show Amount paid even when it is fully paid

dev/core#2061 PCP: call the hook_civicrm_links hook for PCP page user actions

missing id

Merge pull request #18559 from agileware/CIVICRM-1567

Update the post-upgrade thank you message to include URLs to CiviCRM contributors, CiviCRM members and minor rewrite

Simplify call to loadRelationObjects in repeat/completetransaction

Merge pull request #18590 from eileenmcnaughton/taskname

[REF] Remove unused taskName variable

Merge pull request #18600 from eileenmcnaughton/aipn

[REF] IPN - move unshared chunk of code out of shared function

Merge pull request #18536 from eileenmcnaughton/main

[REF] Paypal std ipn Move not-actually shared-code out of shared code function

Merge pull request #18614 from eileenmcnaughton/mem

[REF] Remove some unused params, move one to where it is used

[REF] Remove some unused params, move one to where it is used

Merge pull request #18617 from eileenmcnaughton/ord

[Ref] Extract getOrderParams

Merge pull request #18620 from eileenmcnaughton/notice

Enotice fix

Merge pull request #18622 from civicrm/5.30

5.30

Merge pull request #18615 from eileenmcnaughton/530

Fix regression on back-office membership renewal by credit card

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

Per https://lab.civicrm.org/dev/membership/-/issues/27 the function fixMembershipBeforeRenew is agreed to be a useful part of this flow
(ie we should set to expired before renewing if that is the correct pre-renewal status). As discussed on that
issue this moves that handling to the pre-process function.

In the process I hit what appears to be an unreleased regression affecting fixMembershipBeforeRenew borking
when changeDate = NULL from https://github.com/civicrm/civicrm-core/commit/2cb6497039fde605f5d64b2fe4fa0548cd5d1e07#diff-f43c8498e32f5b2d68ab27bcd243ca36L1136

The regression only affects master

Enotice fix

test activity contact variations

Merge pull request #18609 from eileenmcnaughton/act2

dev/core#2057 Reduce queries when calling activity.create

Fix regression or back-office membership renewal by credit card

Looking at the code I realised a mistake was made in  https://github.com/civicrm/civicrm-core/pull/17398 whereby
the actual membership id would not be passed through correctly

[Ref] Extract getOrderParams

Overview
----------------------------------------
Minor extraction

Before
----------------------------------------
Code not re-usable

After
----------------------------------------
Code is re-useable, basis for further fix

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

Comments
----------------------------------------
Note this ties into https://lab.civicrm.org/dev/membership/-/issues/28

Merge pull request #18593 from mattwire/joblog

Add an 'Execute Now' button to the job log

[REF] Follow up cleanup from Event Location

CRM_Core_BAO_Block::create & CRM_Core_BAO_Address::create are no longer called with the entity param
- remove support

dev/core#2057 Reduce queries when calling activity.create

This reduces queries in 2 ways
1) if the action is 'create' then existing activity contacts cannot exist, so do not try to delete them
2) if the action is 'edit' then look for & delete for all 3 record types at once.

There are still questions about how we can improve this further but it should remove up to 3 queries per
Activity.create call

Merge pull request #18604 from eileenmcnaughton/act_cont

[REF] Very minor cleanup

Add an 'Execute Now' button to the job log

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

2 years ago we deprecated 'calling website create in weird ways' - it seems like now we can remove that
handling & standardise on recommended create function & deprecate add

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

Merge pull request #18605 from totten/master-pscss

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

dev/core#2046 Ensure location BAOs have create actions, deprecate add, simplify block code

Per https://lab.civicrm.org/dev/core/-/issues/2046 we have a goal (long term) to consolidate create & add functions into one function - with the functionality moved from add into create & add becoming a deprecated wrapper around create.

This makes that change on the IM & OpenID in line with the previous change on Phone & Email, leaving just Address & Website which are
both a bit trickier. The code in Block runs on the 4 that are updated so this simplifies & confirms all 4 are calling
functions that handle primary. We could switch to an apiv4 call - I just want to be a bit more sure we've handlled primary there
& also that we look into Address & website

Merge pull request #18586 from eileenmcnaughton/blok

[REF] Fix Event location to create it's locations directly rather than via shared methods

dev/core#2035 Invoice template - show Amount paid even when it is fully paid

In https://github.com/civicrm/civicrm-core/pull/16680/files?w=1#diff-b211e0dac858fdad5d480c118d645e0bR129 there was a change from
'show amount paid & due depending on pay-later status' to 'show amount paid & due depending on whether payment is due'

This has been experienced as a regression and, being an invoice, there seems no reason to only change it sometimes.

This change just alters the template that new installs will receive. If approved another step is needed to update existing
installs (unless they have customised the template)

Merge pull request #18598 from eileenmcnaughton/prim

Fix bug in primary handling where TRUE rather than 1 used

Merge pull request #18601 from eileenmcnaughton/apiv3activity

Test cleanup fix

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

This file was recently added by way of `scssphp`.  We don't use it, and
trying to bundle the symlink into the tarball breaks some consumers.

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

Minor code cleanup

Remove unused variable

Test cleanup fix

Note it's better practice to clean up in the tear down but it looks like someone
tried & failed before so let's see....