dev/user-interface#14 Fix membership_status_id url handling (recent regression).

The right fix here is to fix the xml to the  field setting it to export & adding a uniquename.

However, with our search focus moving on to the new search functionality in the pipeline & us having had
a hiatus on dealing with  search  code I think this more conservative change + a rc focus makes sense
at the  moment

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

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

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

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 #16876 from colemanw/ssFix

Fix undefined index warning in SavedSearch api

Merge pull request #16702 from seamuslee001/5.24

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

Fix undefined index warning in SavedSearch api

Merge pull request #16834 from colemanw/apiGroup

APIv4 - Improve saveSearch popup in Explorer

APIv4 - Improve saveSearch popup in Explorer

Merge pull request #16858 from eileenmcnaughton/5.24

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

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

Merge pull request #16841 from seamuslee001/ckeditor_4_14_upgrade

Update CKEditor 4.14

fix crash when change language on installer page

Update CKEditor 4.14

Merge pull request #16830 from alifrumin/rn-5.24

[NFC] release-notes 5.24 first pass

release-notes 5.24 first pass

Merge pull request #16752 from eileenmcnaughton/ext24

Fix fatal error on loading extension page when an extension has been deleted

Merge pull request #16796 from seamuslee001/notes_5_23_3

release-notes/5.23.3.md

release-notes/5.23.3.md

Merge pull request #16791 from totten/5.24-dot

(REF) dev/core#1637, dev/core#1651 - Restore format of packagesBase

(NFC) Document format of packagesBase, resourceBase.

Try to prevent future bounciness in changing these variables.

dev/core#1637 - Restore format of packagesBase

These use-cases had been tested during PR dev for 5.23.alpha, but they
regressed in 5.23.1.  In 5.23.1's #16735, note item (5) and the flip-flop on
`/.` Item (5) references some greps to find references `/.` For obscure
reasons, the  file `l10n.js.tpl` didn't match the greps.

Revert "inline editing not working and other js packages issues"

This reverts commit c68b3313741a367e2510ea6be54895405e4438ca.

Merge pull request #16763 from eileenmcnaughton/mem_dash

dev/user-interface#14 Fix regression whereby links from membership dashboard don't work for export

Merge pull request #16775 from demeritcowboy/fin-fatals

dev/core#1645 - (Test for) Fatal error when assigning account to financial type

Merge pull request #16776 from eileenmcnaughton/fform

dev/core#1645 fix  regression  by removing form inheritance

Merge pull request #16779 from demeritcowboy/inline-edit

dev/core#1651 and dev/core#1637 - Inline editing not working on many admin screens and other js packages issues

inline editing not working and other js packages issues

dev/core#1645 fix  regression  by removing form inheritence

The Assign  Account form is failing to load because it inherits indirectly from CRM_Admin_Form which
now has a different visibility on the _id property.

I  took a look and there really is no reason for this complex  inheritence - this is a standalone form
and the setDefaults of  the parent seems of no use, let alone the parent's parent.

Assign, edit & browse seem to still work fine

fatal error when assigning account to financial type

Merge pull request #16766 from seamuslee001/5_23_2_release_notes

release-notes/5.23.2.md

release-notes/5.23.2.md

Merge pull request #16761 from eileenmcnaughton/joomla

dev/financial#120 - Fix CiviCRM base url on Joomla frontend

dev/user-interface#14 Fix regression whereby links from membership dashboard don't work for export

Note it  also applies to membership_status_id but the lack of a uniquename makes the fix trickier  so I'm reluctant to target the
rc

Alternate Joomla! fix - move to CMS override

Fix fatal error on loading extension page when an extension has been deleted

Since we added the concept of 'hidden' tags we are loading them in a way where the exceptions are not caught. This means
that the whole page cannot load - which is needed to disable the missing extension

Merge pull request #16750 from eileenmcnaughton/sgroups

Fix smart group custom field  check to cope  with api error

Fix smart group custom field  check to cope  with api error

As we have added new fields to the saved_search table this will lead to a fatal if it runs before
the database update. This catches that error and handles it

Merge pull request #16748 from seamuslee001/5_23_1_release_notes

5 23 1 release notes

Update 5.23.1.md

Add release-notes/5.23.1.md

Merge pull request #16743 from mlutfy/dev547c

dev/core#547 Fix Event financial_type_id getOptions

Merge pull request #16735 from totten/5.24-paths

dev/core#1637 - Multiple fixes for Civi/Core/Paths.php

dev/core#547 Fix Event financial_type_id getOptions

E2E_Extern_CliRunnerTest - Update to match change in `./`

Per 16735, the interpretation of `getPath('[foo]/.`)` changed
slightly - from:

* 5.22: Inconsistent/undefined tail end (may or may not have trailing `/`)
* 5.23.0: Defined to always end with `/`
* #16735: Defined to never end with `/`

CRM_Core_I18n - Fix usage of `getPath('[foo]/.')`

The interpretation of `/.` is evolving per #16735:

* When this code was first written, it was unspecified/variable whether the value `[foo]/.` would end in `/`
* During most of the testing of 5.23.beta1, this was defined to always return `/`
* During a regression fix in 5.23.1, we're flipping it back the other way so that `[foo]/.` never ends in `/`.

dev/core#1637 - Multiple fixes for Civi/Core/Paths.php

For a full write-up, see PR

dev/core#1637 - PathsTest - Redefine "[foo]/", "[foo]/0", "[foo]/."

The rationale will be discussed more via PR description.

(NFC) PathsTest - Make it easier to identify/debug test cases

When browsing the list of outputs from this test class, each of the test
cases was identified by its numerical position in the list of `$exs`.  This
makes it hard to keep track of the failures.

1. Add a symbolic name to each case (that's easier to search on)
2. Add more verbose output for failed assertions

(NFC) dev/core#1637 - Add E2E test for generation of Civi-WP backend URLs without trailing /

Revert "Revert "Make $civicrm_paths less sensitive to trailing slashes. Add tests.""

This reverts commit cbcbfd69d34c9895342e3d813a42f97d93dbb76d.

Merge pull request #16720 from agh1/5.24.0-release-notes

5.24.0 release notes: initial run

Merge pull request #16721 from seamuslee001/524_1637

dev/core#1637 Revert "Make $civicrm_paths less sensitive to trailing slashes. Add t…

Revert "Make $civicrm_paths less sensitive to trailing slashes. Add tests."

This is currently causing breakage on wordpress sites where clean urls are not enabled.

Compare the 2 urls below - the top one has an extra (breaking) slash added by this PR.

I propose a quick revert  & patch release followed by 'the right' fix at a slower pace
/wp-admin/admin.php/?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext
/wp-admin/admin.php?page=CiviCRM&q=civicrm%2Fupgrade%2Fqueue%2Fajax%2FrunNext

This reverts commit 232fdd3dfac4ea09749d7f178818139514e48e38.

5.24.0 release notes: added boilerplate

5.24.0 release notes: raw from script

Merge pull request #16707 from demeritcowboy/smartgroup-warning

dev/core#1636 - In system status check avoid E_NOTICEs for smart groups that don't have form_values[0]

skip smart groups that don't have a form_values[0]

[REF] Remove  variable from preboot locations to fix compataiblity with backdrop

Set version to 5.24.beta1

Merge pull request #16677 from eileenmcnaughton/mem

[REF] Change function signature to support moving this off the form layer

Merge pull request #16666 from colemanw/smarter

APIv4-based smart groups

Merge pull request #16649 from alifrumin/copyState3

dev/core#1605 State/province not copied on shared address

[REF] Change function signature to support moving this off the form layer

This is really a business logic function & belongs on the BAO. This change simply removes the
access to  such that it can be moved by making it an input parameter.

Next step would be o make it static & move  it to CRM_Member_BAO_Membership

Evenually this should not be called by the form as the CRM_Member_BAO_Membership
::create function will call it

Fix "no such field" failure in SmartGroup upgrade

Calling api get with no return fields specified is always risky during upgrades because upgrades can add/drop columns.

APIv4 Explorer - add docs for save & execute buttons

APIv4-based smart groups

APIv4 - Add SavedSearch entity

Add columns for api_entity & api_params to civicrm_saved_search

Merge pull request #16684 from civicrm/5.23

5.23

Merge pull request #16686 from demeritcowboy/install-seqcreditnotes

Fix parameter format for upgrade call to install/enable sequentialcreditnotes

Merge pull request #16687 from demeritcowboy/letter-h

[NFC] Add missing letter "h" in upgrade script for task description

Merge pull request #16685 from mlutfy/fixSettingsYesNo

Fix SettingTrait YesNo translation

Merge pull request #16680 from magnolia61/Contribution_Invoice_Enhancements

Style & layout clean up

Merge pull request #16683 from demeritcowboy/disable-xoauth2

dev/mail#59 - Don't release 5.23 with XOAUTH2 option enabled since it isn't ready yet

fix parameter format

add missing letter h

Merge pull request #16020 from mattwire/setentityid

Add setEntityId() to entityForm

Fix SettingTrait YesNo translation

Style & layout clean up

Merge pull request #16682 from colemanw/remove

Remove deprecated function CRM_Contact_BAO_GroupContactCache::remove

Update unit test now that the xoauth2 option is disabled for the time being

Merge pull request #16681 from alifrumin/rn5.23secondpass

rn-5.23: adding late changes to the 5.23 release notes [nfc]

add test

xoauth2 isn't ready yet

Remove deprecated function CRM_Contact_BAO_GroupContactCache::remove

adding late changes to the 5.23 release notes

Merge pull request #16675 from christianwach/lab-1628

Fix display of administrator permissions in WordPress Multisite

Fix display of administrator permissions in WordPress Multisite

Merge pull request #16676 from christianwach/lab-1629

Fix synchronisation of Users to Contacts in WordPress Multisite

Merge pull request #16679 from pradpnayak/priceSetSchema

added sudo constant for sms api type

added sudo constant for sms api type

Fix synchronisation of Users to Contacts in WordPress Multisite

Merge pull request #16672 from MegaphoneJon/core-1627

Activity Details report: fix filtering by is null/is not null

Activity Details report: fix filtering by is null/is not null

Merge pull request #16667 from eileenmcnaughton/acl

[REF] Only call getACLs when contact_id is present, remove handling

Merge pull request #16566 from eileenmcnaughton/setting_fix

Remove helper function now that contribution settings is not weirdly stored