Merge pull request #18541 from eileenmcnaughton/dep

Fix deprecation notice

Merge pull request #18499 from eileenmcnaughton/merger

dev/core#2047 [REF] Minor code extraction

Merge pull request #18548 from civicrm/5.30

5.30

Merge pull request #18547 from totten/5.30-ml-test

api_v3_MultilingualTest - Skip Cxn entities

Merge pull request #18545 from laryn/patch-3

Check if BACKDROP_ROOT is defined already

api_v3_MultilingualTest - Skip Cxn entities

Merge pull request #18546 from colemanw/getTableNameByEntityName

Fix customGroup getTableNameByEntityName to recognize all entities

Update Backdrop.php

Fix customGroup getTableNameByEntityName to recognize all entities

Check if BACKDROP_ROOT is defined already

Merge pull request #18543 from eileenmcnaughton/line_tax

[REF] Remove now used parameter & make function protected

Merge pull request #18542 from eileenmcnaughton/xml

Fix civi version for greenwich

Merge pull request #18458 from eileenmcnaughton/processor

dev/core#2017 Move function only used by bin/ContributionProcessor to that class

[REF] Remove now used parameter & make function protected

This function used to be called by LineItem api but we removed that when adding v4 line item api

A universe check shows it's not otherwise called so it's only called with the same class, and making
it protected makes that clear. isLineItem is not passed in that one place

Fix civi version for greenwich

We are releasing this as a stable part of 5.31 - xml should reflect that

Merge pull request #18539 from eileenmcnaughton/ppcancel

[NFC] Cleanup in test class

Merge pull request #18451 from eileenmcnaughton/renew

[REF] Consolidate input params that are primarily used for the membership entity action to an array

Merge pull request #18286 from sunilpawar/ui_30

user-interface#30 Ability to Send Invoice with modified subject and CC

Merge pull request #18281 from sunilpawar/report_48

report#48 View Payment owned by Different contact on Membership and Participant View

Fix deprecation notice

User deprecated function: Deprecated function CRM_Core_BAO_Cache::cleanKey, use CRM_Utils_Cache::cleanKey. Array ( [civi.tag] => deprecated ) in CRM_Core_Error_Log->log() (line 58 of /Users/eileenmcnaughton/buildkit/build/dmaster/sites/all/modules/civicrm/CRM/Core/Error/Log.php).

Merge pull request #18537 from eileenmcnaughton/cancel

dev/core#1986 Alter default for  send notification to contributor checkbox on cancel or edit recurring to off

[NFC] Cleanup in test class

Minor cleanup to test class (single quotes, Camel case entities for calling ipn, use getsingle)

dev/core#1986 Alter default for  send notification to contributor checkbox on cancel or edit recurring to off

Per https://lab.civicrm.org/dev/core/-/issues/1986 this got strong support

Merge pull request #18513 from eileenmcnaughton/export_extract

[REF] Extract the code to determine the DAO name into a functions

Merge pull request #18521 from eileenmcnaughton/fin_type

Improve metadata on LineItem DAO

Merge pull request #18465 from totten/master-greenwich-ext

Greenwich: Add BootstrapCSS support via core extension

Merge pull request #18529 from seamuslee001/core_afform_php74_array_string

[REF] Fix deprecated array and string offset access using curly brace…

Merge pull request #18531 from civicrm/5.30

5.30 to master

Merge pull request #18525 from christianwach/lab-core-1997-530

Fix PayPal Standard session loss in WordPress (5.30)

Merge pull request #18524 from MegaphoneJon/core-2052-rc

fix datepickers to show correct minDate

Merge pull request #18530 from seamuslee001/fix_date_relationship

[NFC] Fix dates in RelationshipTest

[NFC] Fix dates in RelationshipTest

[REF] Fix deprecated array and string offset access using curly braces in afform

Merge pull request #18479 from eileenmcnaughton/just_pay

dev/financial#148 Simplify parameters passed to completeOrder

Fix PayPal Standard session loss in WordPress

Merge pull request #18523 from civicrm/5.30

5.30

fix datepickers to show correct minDate

Merge pull request #18520 from colemanw/searchGroupBy

Search ext: Aggregate field fixes

Improve metadata on LineItem DAO

Adds pseudoconstants, labels

Search ext: Ensure all non-grouped columns are aggregated if using GROUP BY

Search ext: When removing a column from SELECT, also remove from ORDER BY

Search ext: Fix field label for aggregate fields

Merge pull request #18427 from mattwire/membershipbackendcached

Switch backend membership form to use non-deprecated/cached functions to get membership types

Merge pull request #18485 from eileenmcnaughton/ref2

dev/core#2043 remove pass-by-reference

Merge pull request #18505 from seamuslee001/afform_container_fix

[REF] Fix visibility of afform_scanner container service for Symfony …

Merge pull request #18511 from totten/master-aff-test

Afform Tests - Fix extension tests when run via `civi-test-run`

[REF] Extract the code to determine the DAO name into a functions

Note per https://github.com/civicrm/civicrm-core/pull/18512 my intent is actually to land on class inheritance
rather than these switches but by starting to functionise them it becomes easier to start to determine what the functions
should be

[REF] Fix visibility of afform_scanner container service for Symfony 3 and 4

Merge pull request #18506 from totten/master-bundle-array

CRM_Core_Resources::addBundle() - Fix handling of array inputs

Merge pull request #18507 from eileenmcnaughton/test_fix

Test fix - use valid membership type

afform_clear() - Fix error when running on non-cached container

After switching the tests from an effective policy of `CIVICRM_CONTAINER_CACHE=auto`
to an effective policy of `CIVICRM_CONTAINER_CACHE=never`, this started to fail:

```
1) api_v4_AfformTest::testGetUpdateRevert with data set #0 ('mockPage', array('', '', 'civicrm/mock-page', 'access Foobar'))
Symfony\Component\DependencyInjection\Exception\BadMethodCallException: Setting service "angular" for an unknown or non-synthetic service definition on a compiled container is not allowed.

/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/vendor/symfony/dependency-injection/ContainerBuilder.php:532
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/ext/afform/core/afform.php:520
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/ext/afform/core/Civi/Api4/Afform.php:99
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/BasicBatchAction.php:88
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/BasicBatchAction.php:68
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Provider/ActionObjectProvider.php:68
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php:150
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/AbstractAction.php:238
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/ext/afform/mock/tests/phpunit/api/v4/AfformTest.php:53
/home/me/bknix/extern/phpunit6/phpunit6.phar:570
```

Container - Disable caching in as many unit-tests as we can

Merge pull request #18503 from colemanw/removeHookCiviCRMTabs

Remove long-deprecated hook_civicrm_tabs

Merge pull request #18488 from eileenmcnaughton/loc_block

dev/core#1973 Fix Email & Phone storage issues in event location

Test fix - use valid membership type

Addresses cause of https://github.com/civicrm/civicrm-core/pull/18427 fail

CRM_Core_Resources::addBundle() - Fix handling of array inputs

(This fixes a small bug in a new function added during this release -- part of #18247.)

The signature of `addBundle()` optionally accepts an array/iterable -- if
given, then it should add all the items from the array. For example:

```php
Civi::resources()->addBundle(['foo', 'bar']);
```

Before
------

It adds `foo` but then bails out on `bar`.

After
-----

It adds both `foo` and `bar`.

Merge pull request #18501 from eileenmcnaughton/acltest

dev/core#2039 Fix test to use set up method with more data integrity

dev/core#1973 Fix Email & Phone storage issues in event location

Merge pull request #18492 from colemanw/moveDaoFunctionsToBaseClass

Ensure DAO base class contains functions to be removed from generated files

Merge pull request #18490 from eileenmcnaughton/emailadd

Remove deprecated code lines

Remove deprecated hook_civicrm_tabs

This hook has been deprecated for 5 years. Time to remove it.

Merge pull request #18498 from eileenmcnaughton/opennp

dev/core#2039 Fix OpenID::add to ensure is_primary is set

dev/core#2039 Fix test to use set up method with more data integrity

Using the api rather than CRM_Core_DAO::createTestObject to create the phone ensures it is created with is_primary.

We might need to fix CRM_Core_DAO::createTestObject but for this case simply using the api suffices

[REF] Minor code extraction

This extracts code to the MergeHandler class for deduping locations. We have a LOT of
test cover here with all the scenarios in jobTest::testBatchMergesAddresses & other functions
using that same data provider.

The goal is to refactor the code onto the MergeHandler class, with a view to
doing less variable passing & more property-using to help clean it up -
in this case it's already visible that 2 variables can be derived in the mergeHandler:

dev/core#2039 Fix OpenID::add to ensure is_primary is set

In my efforts to determine the the non-performant line of code that makes sure IM is set is
not required I edited tests to track down when is_primary is not correct & determined it
is missing from the add function - same as https://github.com/civicrm/civicrm-core/pull/18489 for IM

Merge pull request #18489 from eileenmcnaughton/im

dev/core#2039 Add check to ensure all have a primary address, fix identified but in IM.add

composer.lock - Update composer-compile-plugin

composer.json - Move greenwich-specific rules into the greenwich folder

composer.json - Update composer-compile-plugin

greenwich - Add to civicrm_generated.mysql

greenwich - Activate during upgrade

greenwich - Activate on new builds

composer.lock - Update for Greenwich/SCSS libraries and compilation task

greenwich - Make a more useful readme

greenwich - Quick/weak stab at slightly-less glaringly-mismatched but…

greenwich - Declare theme. Use the new bootstrap3.css

greenwich - Define compilation task

composer.json - Add scssphp, php-autoprefixer, and compile-plugin

greenwich - Download bootstrap-sass

greenwich - Remove unnecessary hook stubs

Import skeletal "greenwich"

Merge pull request #18497 from eileenmcnaughton/open_primary

dev/core#2039 [NFC] Code cleanup in test class

Merge pull request #18496 from eileenmcnaughton/nfc

dev/core#2039 [NFC] Minor cleanup in tests

Merge pull request #18495 from eileenmcnaughton/email_add

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

[NFC] Code cleanup in test class

Merge pull request #18494 from eileenmcnaughton/test_clean

dev/core#2039 [NFC] Test cleanup to stop calling Email::add action

[NFC] Minor cleanup in tests

Merge pull request #18484 from eileenmcnaughton/ref

dev/core#2043 remove pass-by-ref

Merge pull request #18491 from totten/master-ref-gendata

(REF) GenerateData - Make it possible to call this via PHP

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

This folds the 2 functions into one, deprecating add

[NFC] Test cleanup to stop calling Email::add action which does not handle primary

Note I feel like we should further
1) move the contents of email add into create
2) make add a deprecated wrapper for create

Whatever the vision of different add vs create functions it didn't turn out in practice and seems to
only generate confusion. We recommend apiv4 as the preferred create methodology

Ensure DAO base class contains functions to be removed from generated DAO files.

The functions fieldKeys(), getLog(), and getTableName() are redundant in the generated DAO files,
we don't need 50 copies of each function.

This preliminary commit ensures they are present in the parent class, and in the future we can
regenerate the DAOs without them.

dev/core#2043 remove pass-by-ref

Also shortening of if clause

Merge pull request #18470 from JKingsnorth/dev/core#2027-2

dev/core#2027 Add/update to UK county list

Merge pull request #18423 from eileenmcnaughton/afform

dev/core#2000 Move afform into core extensions

dev/core#2027 Add/update to UK county list

(REF) GenerateData - Allow external control randomness

(REF) GenerateData - Use overrideable 'echo'. Use exceptions.

This removes elements of GenerateData which perform direct output to the user. there
are basically two cases:

1. Outputting advisory/status info (e.g. "echo"). This now goes through a method, which
   can be overriden/replaced.

2. Displaying errors from DAO-calls. Now-a-days, DAO-calls can raise their own exceptions,
   so we don't need to sprinkle it without error-outputs.

(REF) GenerateData - Extract method 'generateAll()'

GenerateData - Make the generated sample data more stable (wrt shuffle+array_rand)