Merge pull request #16919 from jaapjansma/dev-1674

dev/core#1674: fix rebuilding of container on every request

(REF) CRM_Api4_Services - Cleanup extra comment/try/catch

Merge pull request #16940 from eileenmcnaughton/email4

[REF] Move properties to Trait

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

Move properties to Trait

Merge pull request #16935 from eileenmcnaughton/email_common

[REF] Start transitioning EmailCommon class to a trait

dev/core#1674 - Update container cache when folder is removed or created

Conceptually, you want to:

* (A) Keep the container cache when nothing has changed (regardless of whether that status-quo has an API4 subscriber folder).
* (B) Update the cache if Civi/Api4/Event/Subscriber/*.php has been newly created
* (C) Update the cache if Civi/Api4/Event/Subscriber/*.php has been newly removed

To test this out, I hacked `Civi\Core\Container::loadContainer()` to emit a debug message
to indicate if it is reusing the cache or updating. Then, I prepared two terminals for running alternate steps:

* In terminal 1, go back and forth with adding/removing folders/files like `Civi/Api4/Event/Subscriber`.
* In terminal 2, periodically run `cv ev 'echo "Hello\n";'` and note whether the cache is update.

In this way, we can see if a developer action (adding a file) leads to an automatic update in the cache.

I found that the previous commit fixed (A) and did (C), but it failed to
update per (B).  This commit should handle (A), (B), and (C).

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

Update style warnings

dev-1674: fix rebuilding of container on every request

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