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

Merge pull request #18605 from totten/master-pscss

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

Merge pull request #18586 from eileenmcnaughton/blok

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

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

Merge pull request #18447 from mlutfy/inheritLocaleRegression

dev/translation#51 Fix inheritLocale regression

Merge pull request #18567 from eileenmcnaughton/apiv3activity

dev/core#2057 Fix 'sleeper-bug' in apiv4, BAO activity.create

Merge pull request #18591 from eileenmcnaughton/search

[REF] Consolidate retrieval of searchFormValues

Merge pull request #18597 from seamuslee001/core_exts_distmaker

[REF] Include recently added core extensions into distmaker

Fix bug inn primary handling where TRUE rather than 1 used

Merge pull request #18571 from eileenmcnaughton/aipn

dev/financial#148 Duplicate validateData onto A.net

[REF] Include recently added core extensions into distmaker

Merge pull request #18594 from mattwire/repeattransactiondoubleexception

Remove double exception handling in repeattransaction

[REF] Consolidate retrieval of searchFormValues

Currently there are 3 ways in which the values from the searchForm are retrieved

1) the search form sets a value 'searchFormName' on the form during search which can
be later retrieved
2) the search task has an action that reflects the form it came from
3) as per 2 but there is additional handling for the fact it might be
the contact search (knowable because the entity has been set to 'Contact'

I'm not 100% sure at this stage that the selected method (3) in this PR is the best -
but it seems 'as good' as 1 with less form-state stuff and this PR consolidates it
into 1 place so it is handled consistently

Merge pull request #18587 from eileenmcnaughton/prevnext

dev/core#2029 E2E.Core.PrevNextTest.testDeleteByCacheKey More debug attempts

Merge pull request #18595 from adixon/preferred-language-1883

pass in "required" value when building preferred_language form element

pass in "required" value when building preferred_language form element

Remove double exception handling in repeattransaction

Fix MailingJob::writeToDB to handle it's own activityTarget wrangling

Per https://github.com/civicrm/civicrm-core/pull/18567 it makes sense for activity to have basic support for activity targets
but form more nuance the function can DIY

Fix 'sleeper-bug' in api.

It turns out that the Activity.create BAO
- deletes the existing source contact for an activity if the param 'source_contact_id' is set
- deletes existing assignee contacts for an activity if the param 'assignee_contact_id' is empty
- deletes existing target contacts for an activity if the param 'target_contact_id' is empty

The v3 api provides some fairly convoluted wrapping to effectively convert is empty
to isset (via 2 params deleteActivityAssignment, deleteActivityTarget which would only
be anything other than true if the relevant value key is not set or we are operating from
the one place in universe that passes in this param (well actually it's true then too
https://github.com/eileenmcnaughton/civicrm_entity/commit/bd779c19bbf82ca22260615b4f7667d02eee1200
)

The v4 api does no handling and we can expect that on update activity contact records are
likely being lost - ditto with direct BAO calls that were probably QAd at some point but
may have changed over time.

This changes the BAO to treat the assignee & target params the same way as source
- ie do nothing if not set, but empty means to delete them

Remove support for api param deleteActivityAssignment, deleteActivityTarget

I did a universe search & it was only called from one place which
I fixed
https://github.com/eileenmcnaughton/civicrm_entity/commit/bd779c19bbf82ca22260615b4f7667d02eee1200

- it's never been a tested param so technically not supported

This is part of my effort to fix redundant queries in Activity.create regarding
activity contacts

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

This form does extra work in order to call  CRM_Core_BAO_Block::create and CRM_Core_BAO_Address::create - but they have
extra handling that only exists for this code place. So, we are making 2 functions more complex in order to inter-relate them.

This simplifies the CRM_Event_Form_ManageEvent_Location and I will do follow on cleanup once merged. Note I previously
added tests to CRM_Event_Form_ManageEvent_LocationTest specifically to cover this (in a recent previous refactor

Merge pull request #18588 from eileenmcnaughton/phone

dev/core#2046 Fix Phone:add to be a pseudonym for Phone.create

dev/core#2046 Fix Phone:add to be a pseudonym for Phone.create

Per https://lab.civicrm.org/dev/core/-/issues/2046 we have discussed rationalising add vs create

In the case of phone making this change only affects one place - a place that currently has a
bug that relates specifically to the handling of is_primary & which is more solvable by
calling create with it's extra is_primary handling.

(the related bug is that the is_primary handling in Block::create is unreliable &
to mitigate that we are doing an additional 10 queries on every contact create - so
handling is_primary right in the first place helps address that)

Merge pull request #18510 from eileenmcnaughton/preprocess

[Ref] Extract getFormValues

dev/core#2029 More debug attempts

Merge pull request #18583 from colemanw/afformGui

Afform GUI: add bootstrap3

Merge pull request #18397 from swastikpareek/issue-1987-fix-isFrontPage-function

dev/core#1987: Fix Drupal Base 'isFrontEndPage' function to consider Drupal public page for FE theme

Merge pull request #18584 from demeritcowboy/schedmail-targets

[NFC/Test] Unit test for target contacts on Bulk Email when mailing in batches

Merge pull request #18572 from colemanw/concat

APIv4 Search: Improve GROUP_CONCAT with :label prefix

Afform: add bootstrap3 bundle

Merge pull request #18569 from eileenmcnaughton/synt

SyntaxConformance::testSqlOperators cleanup fix - ensure entities are deleted

Merge pull request #18538 from eileenmcnaughton/name

[REF] Remove checks as to whether entityShortName is in the component  array

Merge pull request #18512 from eileenmcnaughton/export_classes

Separate export into separate classes to allow unravelling of component handling (Member)

Afform: code cleanup & remove unused menu item

test for target contacts on Bulk Email for mailing in batches

Merge pull request #18562 from eileenmcnaughton/locit

[Ref] Merge code - Move determination about location type to the getDAOForLocation…

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

Search ext: Expose DISTINCT modifier

APIv4 Search: Improve GROUP_CONCAT with :label prefix

This moves handing of pseudoconstant suffix replacement from the UI level to the API level,
which allows APIv4 to return GROUP_CONCAT results as an array (by default, if no separator specified).

It also improves post-query formatting in general, with finer-grained formating callbacks for sql functions.

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

dev/financial#148 Duplicate validateData onto A.net

Test cleanup fix

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

dev/core#2047 [REF] Move calculation of 'type' to copyDataToNewBlockDAO

[Ref] Move determination about location type to the getDAOForLocationEntity function

This seems more legible than the metadata trick....

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