Merge pull request #18578 from eileenmcnaughton/api42

Simplify CRM_Core_BAO_Location::createLocBlock by moving eventLocation specific handling back to the class

Merge pull request #18566 from eileenmcnaughton/act_contact

dev/core#2057 ActivityContact.create Remove select query that never or almost never finds something.

Merge pull request #18579 from colemanw/select2-bootstrap

Add select2-bootstrap to greenwich theme

ext/greenwich - Improve portability. Consolidate tasks.

1. Using 'php-eval' instead of 'shell' should make it safer to run on Windows.

2. The two steps are very closesly related. Putting them into the same 'task'
   means that it's a bit easier to understand the 'watch' behavior.

composer.lock - Update compile plugin

composer.json - Update compile plugin

Merge pull request #18573 from eileenmcnaughton/nnfc

[NFC] Minor cleanup - use strict comparison where possible

ext/greenwich - Coerce `select2-bootstrap.css` to be handled as SCSS (#9)

Co-authored-by: Tim Otten <totten@civicrm.org>

Merge pull request #18568 from mattwire/membershipdeprecated

Switch to non-deprecated/cached functions for membership pricesets

Merge pull request #18580 from eileenmcnaughton/matt

Unit test for https://github.com/civicrm/civicrm-core/pull/18568

Simplify CRM_Core_BAO_Location::createLocBlock by moving eventLocation specific handling back to the class

On grepping universe we find entity is only passed into
CRM_Core_BAO_Location::createLocBlock from this one place in the code - ergo we can
do the handling for it on that class & not in the shared code

Test cover added in preparation for this to CRM_Event_Form_ManageEvent_LocationTest

Unit test for https://github.com/civicrm/civicrm-core/pull/18568

dev/core#2057 Remove select query that never or almost never finds something.

When it comes to quey efficiency it's good to do an extra select if it saves an update/insert
but not if it never does. To test the theory that this 'never does' I put
removing this select through the entire test suite. No tests failed, meaning
it was unnecessary 100% of the time. Adding the catch ensures
it will still succeed if it the row exists but we have moved our query conservation
from doing an extraneous select in 99.9% of cases to an extraneous update in .1% of cases

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

Merge pull request #18576 from eileenmcnaughton/iso

dev/core#1921 Remove handling for Civi4.2 date bug from paypalProIPN

Add select2-bootstrap to greenwich theme

dev/core#2043 Remove pass by reference

Merge pull request #18575 from eileenmcnaughton/api42

dev/core#2044 Make contact_id optional on v4 api for Address, phone, email

dev/core#1921 Remove handling fo Civi4.2 from paypalProIPN

All the tests in
CRM_Core_Payment_PayPalProIPNTest pass through this line

It used to be necessary to reformat dates on a DAO before resaving
- up until 2014

dev/core#2044 Make contact_id optional on v4 api for Address, phone, email

It's optional in the schema for events etc

[NFC] Minor cleanup - use strict comparison where possible

Merge pull request #18550 from eileenmcnaughton/search

Add bootstrap bundle to search

Merge pull request #18528 from mattwire/inputpaymentprocessorid

Switch to passing payment_processor_id as input param to completeOrder

Switch to passing payment_processor_id as input param to completeOrder

Switch to non-deprecated/cached functions for membership pricesets

Merge pull request #18515 from mattwire/membershipbaodeprecated

Switch membership BAO to use non-deprecated cached functions to get membershipType details

Merge pull request #18558 from eileenmcnaughton/act_type

[REF] Remove wrangling on activityType param

Merge pull request #18565 from eileenmcnaughton/cachekey

dev/core#2029 Extend test debug

Merge pull request #18535 from eileenmcnaughton/first

[REF] Remove unreachable lines

Merge pull request #18504 from demeritcowboy/scrunched-cc

dev/core#2040 - Multiple email activity cc recipients get scrunched together in recorded activity details field

Extend test debug

Maybe this will help us understand why the test is intermittant

Merge pull request #18533 from seamuslee001/dev_core_2030

dev/core#2030 ensure that the Country selector is a Select 2 and ensu…

Merge pull request #18561 from eileenmcnaughton/test_data

Minor test data fix up  - ensure domain contact's email is primary

Merge pull request #18430 from eileenmcnaughton/open

dev/core#2017 Remove unused function OpenID::isAllowedToLogin

Merge pull request #18560 from eileenmcnaughton/test

Minor test fix

Merge pull request #18552 from eileenmcnaughton/ref

[REF] Move daoName generation so we don't need to pass the variable name

Minor test data fix up  - ensure domain contact's email is primary

Minor test fix

Merge pull request #18557 from eileenmcnaughton/dep2

[REF] Finally remove deprecated ids handling

Determine name of DAO when needed rather than passing a variable around

Merge pull request #18554 from civicrm/5.30

5.30

Merge pull request #18553 from seamuslee001/master

[REF] Update composer compile plugin to latest version

[REF] Remove wrangling on activityType param

There is only 1 remaining place that calls this function & does not specifiy activityType. This fixes
that place to pass in activityType and stops attempting to calculate activityType
based on in-function guess work

Merge pull request #18551 from colemanw/dontLoadBootstrap

Remove obsolete load-bootstrap.js

[REF] Finally remove deprecated ids handling

We've been pretty clear for a long time the only supported way to call crud functions is via the api.
We noisily deprecated ids about 9 months ago & it has been not-preferred for much longer
I only found one place in universe where ids IS passed in
https://github.com/lcdservices/biz.lcdservices.movecontrib/edit/master/CRM/LCD/MoveContrib/BAO/MoveContrib.php

Merge pull request #18471 from eileenmcnaughton/error

dev/core#2032 Add potential to vary query log files per process

Merge pull request #18534 from eileenmcnaughton/5.30

Follow up fix on paypal redirect

[REF] Update composer compile plugin to latest version

Merge pull request #18541 from eileenmcnaughton/dep

Fix deprecation notice

Remove obsolete load-bootstrap.js

Add bootstrap bundle to search

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

[REF] Remove unreachable lines

The function recur ONLY calls single if first is TRUE. Once we remove first from the if clause
we can see it applies if recur is TRUE or if recur is FALSE (or any other variant given we are using
truthiness). So the if can go

Follow up fix on paypal redirect

Turns out we did some function renaming & missed a spot before merge.

As an aside I fixed Paypal to call CRM_Core_Config::singleton()->userSystem->prePostRedirect()
directly rather than via system. I'm tempted to take the System function back out
again - I don't think it offers a better one-liner

Merge pull request #18521 from eileenmcnaughton/fin_type

Improve metadata on LineItem DAO

dev/core#2030 ensure that the Country selector is a Select 2 and ensure that if it is not shown on the contribution form that it doesn't prevent form submissions

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

Switch membership BAO to use non-deprecated cached functions to get membershipType details

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