[REF] Minor simplification - do not use variable variables

This is a minor readabilty improvement because it's awful. Much more could be done....

Merge pull request #16670 from demeritcowboy/audit-tpl-3

dev/core#1366 - case audit printReport parameter is no longer used

Merge pull request #16889 from colemanw/api4ref

Make api4 select query object more sane

Api4SelectQuery - Refactor field handling for looser coupling and more flexibility

Getting ready to support groupBy, having and sql functions

APIv4 - Standardize field sql name conversion

Merge pull request #16920 from colemanw/api4test

[NFC] APIv4 - Add test coverage for expected fields from join

APIv4 - Add test coverage for expected fields from join

Merge pull request #16893 from seamuslee001/nfc_sort_api4_test

[NFC] Add in unit test to ensure that APIv4 Doesn't accept an invalid…

[NFC] Add in unit test to ensure that APIv4 Doesn't accept an invalid sort

Merge pull request #16917 from colemanw/domainId2

APIv4 - Only adjust domain_id if required

Merge pull request #16902 from eileenmcnaughton/setting_ex

[REF] Extract code used to render a pseudoconstant when a table is defined.

APIv4 - Only adjust domain_id if required

Merge pull request #16914 from eileenmcnaughton/act2

Remove unused code

Merge pull request #16913 from eileenmcnaughton/act

[REF] Move form specific handling back to the relevant form.

Remove unused code

If seems to me that 1) if this IF is true then there will be a bounce & it will not proceed annd
2 if it IS true emailAddress does not seem to be applied anywhere

[REF] Move form specific handling back to the relevant form.

The shared function has handling to set ->_single if the class is CRM_Contact_Form_Task_Email -
this seems prety obviously not functionality that is shared so it should be on the CRM_Contact_Form_Task_Email
class

Merge pull request #16895 from eileenmcnaughton/case

Remove assigns for   atypefile

Merge pull request #16910 from eileenmcnaughton/conf

Deprecate use of  in OptionGroup::add() function

Merge pull request #16909 from colemanw/api4sel

[REF] APIv4 - Preserve order when expanding select wildcards

Merge pull request #16863 from seamuslee001/dev_core_561_daterange_deprecation

dev/core#561 Add in deprecation notice on addDateRange funtion

Deprecate use of  in OptionGroup::add() function

We agreed some time back not to use  as part of add, create signatures

APIv4 - Preserve order when expanding select wildcards

Merge pull request #16845 from jitendrapurohit/core-1663

dev/core#1663 - Fix warning on relationship form

Merge pull request #16896 from eileenmcnaughton/act

[NFC] Use strict comparison where  possible

Merge pull request #16904 from colemanw/selectUtil

Improve APIv4 selectUtils to handle join paths in fieldnames.

Merge pull request #16906 from eileenmcnaughton/format

[NFC] Code formatting

Merge pull request #16905 from eileenmcnaughton/acti

[NFC] minor form cleanup.

[NFC] Use strict comparison where  possible

Also fixes a couple of comments & uses of Array::value

[REF] Extract code used to render a pseudoconstant when a table is defined.

This permits us to re-use for settings

[NFC] Code formatting

[NFC] minor form cleanup.

Most notably  fix calling a non-static method statically but also remove comment that
clearly came in by copy & paste,

Improve APIv4 selectUtils to handle join paths in fieldnames.

Merge pull request #16900 from colemanw/api4fix

APIv4 - bugfix in select query

APIv4 - bugfix in select query

Merge pull request #16886 from demeritcowboy/xmlinclude-windows

dev/core#1643 - Make civicrm-setup (especially drupal 8) work on windows

Merge pull request #16897 from eileenmcnaughton/actt2

[REF] Remove & from before variable

Merge pull request #16880 from yashodha/error

log error message instead of throwing exception

Merge pull request #16898 from civicrm/5.24

5.24

Merge pull request #16697 from eileenmcnaughton/deprecate

Add some deprecation notices, stop calling PrevNext::cleanupCache

[REF] Remove & from before variable

I couldn't see how this & could do anything useful given it's an object & testing shows the form is unchanged

Merge pull request #16842 from demeritcowboy/lang-install-crash

dev/translation#40 - Fix crash when change language on installer page

Remove assigns for   atypefile

This is old code - I  traced back it's introduction to  https://github.com/civicrm/civicrm-svn/commit/fb3038000356998e43b0c9aea67f0186f7e41516
which relates to https://issues.civicrm.org/jira/browse/CRM-3829?page=com.atlassian.streams.streams-jira-plugin:activity-stream-issue-tab

At that time the template used the variable- ie https://github.com/civicrm/civicrm-svn/commit/fb3038000356998e43b0c9aea67f0186f7e41516#diff-6fc7e9552ffce6cf4cb7680854ef7b70R4

But that is no longer the case

Merge pull request #16894 from seamuslee001/core/issues/923

Fixed fatal error for Event Participants custom search if price set f…

Merge pull request #16869 from colemanw/del

APIv3 - Use new DAO::deleteRecord method

Merge pull request #16867 from colemanw/dashboard

APIv4 - Add Dashboard & DashboardContact entities

Merge pull request #16892 from civicrm/5.24

5.24

Fixed fatal error for Event Participants custom search if price set field option is disabled after a registrant has already selected it

Fix test failure and parametise queries

Merge pull request #16891 from seamuslee001/5_23_4_release_notes

Add release-notes/5.23.4.md

Merge pull request #16887 from totten/5.24-dbl-path

dev/joomla#26 - Fix path derivation when CMS is rooted in a subdir

Add release-notes/5.23.4.md

(NFC) Update comments in PathsTest.php

log error message instead of throwing exception

log error message instead of throwing exception

Merge pull request #16872 from colemanw/arrayValue8

[REF] Replace \CRM_Utils_Array::value with ??

Merge pull request #16873 from colemanw/api4ArrayValue

[REF] Civi/Api4 - Refactor out 'use CRM_Utils_Array'

Merge pull request #16874 from colemanw/contactCleanup

[REF] BAO_Contact - Remove CRM_Utils_Array::value and other unnecessary code

Merge pull request #16884 from civicrm/5.24

5.24

dev/joomla#26 - Fix path derivation when CMS is rooted in a subdir

Overview
--------

CiviCRM is deployed inside a CMS. The CMS is usually deployed at the HTTP root (`http://example.org`),
but it is sometimes deployed in a subdirectory (`http://example.org/my-cms`).

Some asset URLs are computed using the variables `[civicrm.bower]`, `[civicrm.packages]`, and `[civicrm.vendor]`, which
are derived from the value of `[civicrm.root]`.  However, if the site is deployed in a subdirectory, and if using v5.23,
then the computation of `[civicrm.bower]` (etc) can misbehave.

Before
------

When the URL for `[civicrm.bower]` (or similar) is derived, it goes through multiple filters - first, from absolute to
relative, and then later from relative back to absolute.  In the process, the base is inadvertently changed.

After
-----

When the URL is derived, it is computed in absolute format - and simply kept that way.

Comments
--------

Regarding test coverage, there are two relevant unit-tests. This PR only updates one.

* `E2E\Core\PathUrlTest`: This is a more concrete smoke test which demonstrates functional problems with variables like
  `[civicrm.bower]`.  It should already catch problems like dev/joomla#26...  but only if you run the E2E suite on a system
  where the CMS was deployed to a subdirectory.  `civibuild` doesn't currently include such a build-type.
* `Civi\Core\PathsTest`: This is a more abstract, headless test to examine edge-cases in `Civi\Core\Paths`. It does not
  specifically check for `[civicrm.bower]` or similar variables (b/c the URL routing is ill-defined in a headless context).
  However, I've updated it to compare/contrast the working and non-working ways to derive variables.

make install work on windows

Merge pull request #16878 from colemanw/api4Sql

[REF] APIv4 - Restructure the way get query objects are constucted

Merge pull request #16883 from demeritcowboy/blank-dashboard-notice

dev/core#1668 - Avoid E_NOTICE when removing last dashlet

Merge pull request #16876 from colemanw/ssFix

Fix undefined index warning in SavedSearch api

Merge pull request #16659 from demeritcowboy/name-label-print-report

Alternate to #16650 - On Case Audit/Print Report richtext details field is getting escaped when system is non-english

avoid notice on removing last dashlet

Merge pull request #16591 from ixiam/dev/core#1575

Fix use col "Soft Credit Amount Stats" and any filter of contribution…

DashboardContact - Change column_no field from bool to int

Cleanup Dashboard BAO

APIv4 - Generic handler for domain_id field.

This field should be treated the same across all entities for consistency.

Dashboard - serialize field in metadata

APIv4 - Add Dashboard & DashboardContact entities

Merge pull request #16870 from seamuslee001/pear_mail_patch_port

[REF] Use composer patches to apply CiviCRM Customisations for CRM-1367 and CRM-5946

Merge pull request #16881 from wmortada/core#1331-4

dev/core#1331 Add missing parameter to translation string

Merge pull request #16875 from colemanw/arrayValue9

[REF] Cleanup is_array(CRM_Utils_Array::value()) pattern

Deprecate redundant api3 util function

APIv4 - Cleanup and restructure the way get query objects are constructed

Add missing parameter to translation string

Merge pull request #16832 from eileenmcnaughton/deltest2

dev/core#667 Fix bug where entity_tags are not deleted by 'soft FK'

BAO_Contact - Remove CRM_Utils_Array::value and other unnecessary code

Merge pull request #16785 from demeritcowboy/weird-casetype-check

dev/core#1652 - Check for change case type activity on case activity form doesn't work

Merge pull request #16674 from alexymik/patch-2

Add privacy fields to ContributionDetail report

Merge pull request #16787 from eileenmcnaughton/dead

Follow up on fix on updating memberships when a contact is deceased

Merge pull request #16879 from civicrm/5.24

5.24

Merge pull request #16702 from seamuslee001/5.24

bin/*, extern/* - Fix leak of "$config" in global namespace

Merge pull request #16857 from mattwire/cancelsubscriptionprocessorname

Fix display of payment processor title in cancelSubscription form

Merge pull request #16853 from colemanw/bool3

CRM/Contribute - Cleanup boolean expressions

Follow up on fix on updating memberships when a contact is deceased

This removes the now-duplicate call from the Contact edit form & fixes it such that the message would appear
for any logged in action that causes this (notably the 2 ways of editing the contact at the moment.)

Note it might change the way the messages look from one message to more than one - this  feels like
it's arguable enough that it's not worth worrying about

dev/core#667 Fix bug where entity_tags are not deleted by 'soft FK'

Merge pull request #16480 from eileenmcnaughton/comp

dev/financial#87 make it possible to add a payment to a fully paid contribution

Fix undefined index warning in SavedSearch api

Merge pull request #16742 from jaapjansma/dev-1640

dev/1640 New feature - adds a  checkbox to the  update contribution  status  form  to determine  if  a receipt  is  sent

Cleanup is_array(CRM_Utils_Array::value()) pattern

Issue #1640 including a test

Merge pull request #16871 from colemanw/arrayValueTest

Civi/Test - Refactor out CRM_Utils_Array::value

Civi/Api4 - Refactor out 'use CRM_Utils_Array'

Civi - replace CRM_Utils_Array::value with ??

Civi/Test - Refactor out CRM_Utils_Array::value

Use composer patches to apply CiviCRM customisations for pear_mail

Rename file to be standard patch file

Merge pull request #16609 from wmortada/core#1331-3

dev/core#1331 Don't freeze fields for auto-renew memberships

Merge pull request #16861 from eileenmcnaughton/dep

[TEST] Add test for to time processing