Cleanup customField prepareCreate function

Merge pull request #16901 from eileenmcnaughton/settings

Fix bug on handling 'is_required'

Merge pull request #16953 from jitendrapurohit/core-1685

dev/core#1685 - Search builder returns DB error on Group => Empty filter

Merge pull request #16623 from mattwire/tokenvalues

Fix values passed to tokenValues hook

Merge pull request #16989 from colemanw/seeingDouble

[REF] Cleanup custom field handling... twice

Merge pull request #16984 from colemanw/ufGroupCleanup

Cleanup custom field handling in ufGroup BAO

Merge pull request #16990 from colemanw/multi-cleanup4

[REF] Cleanup CustomField BAO to use its own isSerialized function

Merge pull request #16988 from civicrm/5.25

5.25

Merge pull request #16991 from pradpnayak/customFieldDefaults

Fix multiselect/checkbox custom field defaults on contribution online form

Merge pull request #16981 from colemanw/fixCustom

Fix multiselect/checkbox custom field defaults on registration profile

Fix multiselect/checkbox custom field defaults on contribution online form

Cleanup CustomField BAO to use its own isSerialized function rather than guesswork

Cleanup custom field handling... twice

Merge pull request #16978 from seamuslee001/5.25

Add release-notes/5.24.1

Fix values passed to tokenValues hook

Cleanup custom field handling in ufGroup BAO

Fix multiselect/checkbox custom field defaults on registration profile

The CRM_Core_BAO_UFGroup::setProfileDefaults() function expects each field to be an array,
but this function was just passing it the number 1. So it did not have enough field metadata
to correctly format each value.

Merge pull request #16980 from eileenmcnaughton/ex2

[REF] Call makeCSVTable function directly from writeRows

Merge pull request #16979 from eileenmcnaughton/cw

Use isSerialized function rather guessing from html_type

Call makeCSVTable function directly from writeRows

Merge pull request #16966 from demeritcowboy/add-nlbe

dev/translation#4 - Add nl_BE language

Merge pull request #16970 from colemanw/multi-cleanup2

[REF] CustomGroup - cleanup handling of serialized fields in old function

Use isSerialized function rather guessing from html_type

Add release-notes/5.24.1

Merge pull request #16975 from colemanw/multi-cleanup3

[REF] CustomField - Remove pointless caching and move form variable to form class

Merge pull request #16973 from eileenmcnaughton/pan

Capitalise PAN truncation

Merge pull request #16974 from civicrm/5.25

5.25

CustomField - Remove pointless caching and move form variable to form class

CustomGroup - Use isSerialized function rather than html_type

Merge pull request #16971 from totten/5.25-seqcredit

dev/financial#84 - Fix upgrade failure. Thin-out activation logic for `sequentialcreditnotes`.

Capitalise PAN truncation

From chat translation channel:

"Pan truncation" should be written "PAN truncation" for Truncation of the primary account number (credit card number). Now it's obvious to translate!

Merge pull request #16969 from seamuslee001/master

[REF] Update XML file to match updated title in DAO File

dev/financial#84 - Simplify upgrader. Fix "wp-demo" upgrade scenario.

Steps to reproduce
------------------

* Create a site based on `wp-demo` with Civi 5.21
* Make a DB snapshot
* Update code to 5.24
* In the web UI, run the DB upgrade
    * Note: It's important to use the web UI. The problem does not reproduce in CLI.

Before
------

The upgrader freezes on "Upgrade DB to 5.24.alpha1". The `CiviCRM.log` includes:

```
Apr 03 14:41:50  [info] Running task: Upgrade DB to 5.24.alpha1: SQL

Apr 03 14:41:55  [info] Running task: Install sequential creditnote extension

Apr 03 14:42:14  [info] $CRM_Queue_ErrorPolicy_reportError = Array
(
    [is_error] => 1
    [is_continue] => 0
    [exception] => Error 1: Uncaught Error: Class 'CRM_Volunteer_Permission' not found in /home/me/bknix/build/wpmaster/web/wp-content/plugins/civicrm/civicrm/tools/extensions/civivolunteer/volunteer.php:497
Stack trace:
0 /home/me/bknix/build/wpmaster/web/wp-content/plugins/civicrm/civicrm/CRM/Utils/Hook.php(286): volunteer_civicrm_permission(Array)
1 /home/me/bknix/build/wpmaster/web/wp-content/plugins/civicrm/civicrm/CRM/Utils/Hook/WordPress.php(139): CRM_Utils_Hook->runHooks(Array, 'civicrm_permiss...', 1, Array, NULL, NULL, NULL, NULL, NULL)
2 /home/me/bknix/build/wpmaster/web/wp-content/plugins/civicrm/civicrm/Civi/Core/CiviEventDispatcher.php(86): CRM_Utils_Hook_WordPress->invokeViaUF(1, Array, NULL, NULL, NULL, NULL, NULL, 'civicrm_permiss...')
3 /home/me/bknix/build/wpmaster/web/wp-content/plugins/civicrm/civicrm/vendor/symfony/event-dispatcher/EventDispatcher.php(184): Civi\Core\CiviEventDispatcher::delegateToUF(Object(Civi\Core\Event\GenericHookEvent), 'hook_civicrm_p
    [last_task_title] => Install sequential creditnote extension
)
```

After
-----

The upgrade completes. After installation, the `sequentialcreditnotes` extension is active.

CustomGroup - cleanup handling of serialized fields in old function

Use new isSeralized method rather than looking at html_type to figure out how to handle the data

[REF] Update XML file to match updated title in DAO File

Merge pull request #16965 from colemanw/isset

Code cleanup - replace overcomplicated test for null with isset

Code cleanup - replace overcomplicated test for null with isset

Merge pull request #16964 from GuillaumeSorel/patch-1

Update FinancialTrxn.php

Update FinancialTrxn.php

PAN is an acronym for Primary Account Number (credit card number) and writing it full in uppercase will make it more obvious for people to translate as I really wondered even with context what it meant.

Merge pull request #16962 from civicrm/5.25

5.25

Merge pull request #16961 from MegaphoneJon/core-1688-525

core#1688 - false positive on missing custom field check

core#1688 - false positive on missing custom field check

add nl_BE language

Merge pull request #16700 from christianwach/lab-1635

Fix WordPress support for Get UF Locale

dev/core#1685 - Search builder returns DB error on Group => Empty condition

Set version to 5.26.alpha1

Set version to 5.25.beta1

Merge pull request #16753 from agh1/consolidated-php-version

dev/core#1528 - Consolidate to single constant for minimum PHP version

CRM_Report_DAO_ReportInstance - Update checksum

Merge pull request #16948 from eileenmcnaughton/email4

[REF] move buildQuickForm function to the trait

Merge pull request #16951 from colemanw/dirtyMoney

Revert "APIv4 - Fix contribution create"

Revert "APIv4 - Fix contribution create"

This reverts commit 2f97c5641cceec44c02585c2d1d212bda1c6228e.
It should no longer be needed due to the improvement made in 243320eda23195985815ceb65ec7ee9079ae3014.

Merge pull request #16950 from eileenmcnaughton/clean_money

Remove  deprecated cleaning  of money in  the  BAO layer

Remove  deprecated cleaning  of money in  the  BAO layer

Any code that  passed dirty money  to the Contribution::create BAO will  have been seeing
deprecation notices  for  some time.  The  supported method is via the  api & v3 api does  the cleaning
whereas  v4 expects it to be pre-cleaned  - perhaps it's time to remove  this & leave it to  the
calling code

Merge pull request #16947 from colemanw/groupByFunctions

APIv4 - Support GROUP BY and aggregate functions

Merge pull request #16943 from jitendrapurohit/core-1682

dev/core#1682 - Invalid currency  on sending offline membership receipt.

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

[REF] move buildQuickForm function to the trait

Note that Contact had an extra assign. It seems likely the  extra  assign  is not required  but it
would not be a bad thing on the others

Merge pull request #16940 from eileenmcnaughton/email4

[REF] Move properties to Trait

APIv4 - Fix contribution create

APIv4 - Add SQL expression handling and aggregate functions

Adds a series of SqlExpression classes to parse and validate sql expressions used in SELECT, GROUP BY and ORDER BY clauses.
The framework can handle most sql functions, but this first commit adds support for the aggregate functions AVG, COUNT, MAX, MIN & SUM

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

Api4SelectQuery - add more metadata to apiFieldSpec

APIv4 - Add rudimentary support for groupBy

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

dev/core#1682 - Invalid currency  on sending offline membership receipt.

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