Merge pull request #16941 from jitendrapurohit/16845-followup

Follow up PR for #16845 - Fix form error when  variable is NULL

Merge pull request #16945 from lcdservices/dev-core-1683

fixes dev/core#1683 set profile frontend title

Merge pull request #16946 from civicrm/5.24

5.24

Merge pull request #16944 from alifrumin/5.24-rn

[NFC] adding late changes to release notes for 5.24

fixes dev/core#1683 set profile frontend title

late changes to release notes

Follow up PR for #16845 - Fix form error when  variable is NULL

Merge pull request #16935 from eileenmcnaughton/email_common

[REF] Start transitioning EmailCommon class to a trait

Merge pull request #16885 from demeritcowboy/test-getrelatedcases

[NFC/TEST] Resurrect unfinished test for getRelatedCases()

Merge pull request #16921 from pradpnayak/newHook

Add hook to alter display value or Custom field value

Merge pull request #16938 from civicrm/5.24

5.24

Merge pull request #16937 from seamuslee001/5.24

[REF] Fix handling of owner url parameter from Membership Dashboard

[REF] Start transitioning EmailCommon class to a trait

This is a really awful class - I've added a trait &  moved the calls from the various forms to that trait
so the code  on EmailCommon for buildForm is now only called once. I will follow up with more the same

[REF] Fix handling of owner url parameter from Membership Dashboard

Merge pull request #16933 from eileenmcnaughton/member_search

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

Merge pull request #16930 from eileenmcnaughton/524

dev/core#1677 Fix multisite regression on domain membership types

Merge pull request #16928 from eileenmcnaughton/email

[REF] Remove fatal, fix some code comments

Merge pull request #16932 from eileenmcnaughton/mailing_name

dev/core#1656 Remove duplicate label

Merge pull request #16929 from eileenmcnaughton/email2

[REF] extract code to getEmails

Merge pull request #16927 from eileenmcnaughton/campaign

[REF] Minor simplification - do not use variable variables

Merge pull request #16868 from twomice/contribution_detail_pledge

Add "pledge id" as column and filter in Contribution Detail report.

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

Remove duplicate label

Merge pull request #16931 from civicrm/5.24

5.24 to master

Merge pull request #16926 from demeritcowboy/quickfix-case-addtimeline

dev/core#1675 - Temporary regression fix for case add timeline

dev/core#1677 Fix multisite regression on domain membership types

Fixes a regression where membership types bleed between domains

[REF] extract code to getEmails

[REF] Remove fatal, fix some code comments

[REF] Minor simplification - do not use variable variables

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

temp workaround for dev/core#1675

Added unit test

Add hook to alter display value or Custom field value

Merge pull request #16912 from eileenmcnaughton/conf_2

[Test] Remove skip that seems no longer required

Merge pull request #16925 from pradpnayak/pseudoCon

Added pseudoconstant for acl_id

Merge pull request #16924 from demeritcowboy/brackets

dev/core#1676 - E_NOTICE because of missing brackets for order of operations

Added pseudoconstant for acl_id

Merge pull request #16907 from eileenmcnaughton/deprec

Remove deprecated function

missing brackets

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

[Test] Remove skip that seems no longer required

The removed line seems to be loosening the test for a deprecation that I didn't hit in testing today

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

Remove deprecated function

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

test for getRelatedCases

Merge pull request #16878 from colemanw/api4Sql

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