[NFC] minor form cleanup.

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

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

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.

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

Merge pull request #16865 from yashodha/typo-fixes

fix typo

Merge pull request #16834 from colemanw/apiGroup

APIv4 - Improve saveSearch popup in Explorer

Merge pull request #16866 from colemanw/versionThreeAndFour

API tests - remove unnecessary check for v4

APIv3 - Use new DAO::deleteRecord method

Fix display of payment processor title in cancelSubscription form

API tests - remove unnecessary check for v4

APIv4 - Improve saveSearch popup in Explorer

fix typo

Merge pull request #16862 from eileenmcnaughton/build

[REF] Remove redundant call to build permissions

Merge pull request #15709 from seamuslee001/report_jcalender_upgrade

dev/core#561 Convert Reports from jcalender to datepicker format

dev/core#667 Add test to demonstrate entity tags are not being deleted

Merge pull request #16846 from jitendrapurohit/core-1664

dev/core#1664 - Fix errors recorded in log message on Manage Case scr…

[REF] Remove redundant call to build permissions

Per the comments I added to the function the output of this permissions clause is overwritten. I stepped through
this in the process of reviewing the datepicker PR & confirmed it

Convert Reports from jcalender to datepicker format

Add in unit test of report upgrade process

Fix time handling as per review from Eileen and fix query for time handling

dev/core#1664 - Fix errors recorded in log message on Manage Case screen and case dashboard page

[TEST] Add test for to time processing

Merge pull request #16850 from colemanw/bool5

Cleanup reverse boolean expressions

Merge pull request #16852 from colemanw/bool1

CRM/Core - Cleanup boolean expressions

Merge pull request #16859 from civicrm/5.24

5.24 to master

Merge pull request #16858 from eileenmcnaughton/5.24

dev/core#1662 - Saved mappings doesn't work for all use cases

Merge pull request #16851 from colemanw/bool2

CRM/Utils - Cleanup boolean expressions

Merge pull request #16854 from colemanw/bool4

CRM - Cleanup boolean expressions

Merge pull request #16836 from tunbola/modify-form-values

dev/core#1569: Increase Data Size for `form_values` column on civicrm_report_instance table

Merge pull request #16856 from colemanw/writeRecord

Add DAO::writeRecord and DAO::deleteRecord methods

dev/core#1662 - Saved mappings doesn't work for all use cases

Merge pull request #16848 from colemanw/profilePerms

Fix checking permissions in api3 profile get

Merge pull request #16715 from mattwire/cancelsubscriptiongeneratetext

Generate text for cancelSubscription form in payment class

Merge pull request #16855 from ray-wright/patch-2

Fix batch contact/activity update with radio options

Merge pull request #16762 from eileenmcnaughton/mem_allan

dev/core#1630 Partial fix on renew membership missing

Add DAO::writeRecord and DAO::deleteRecord methods

Merge pull request #16849 from colemanw/ufMatch

UFMatch - deprecate unused functions

Wrap value in single quotes

Wrap radio value in single quotes to account for options with spaces.

Cleanup reverse boolean expressions

CRM - Cleanup boolean expressions

CRM/Contribute - Cleanup boolean expressions

CRM/Utils - Cleanup boolean expressions

CRM/Core - Cleanup boolean expressions

UFMatch - deprecate unused functions

Fix checking permissions in api3 profile get

The boolean being passed to the $skipPermission param was reversed.

Merge pull request #16835 from colemanw/removeOldStuff

Remove old stuff

Merge pull request #16839 from mlutfy/fixCustomDataViewMemo

Fix CustomDataView.tpl plain empty memo field.

Merge pull request #16844 from eileenmcnaughton/coleman

CRM/Contact - Cleanup boolean expressions

CRM/Contact - Cleanup boolean expressions

Merge pull request #16843 from seamuslee001/master

5.24