Merge pull request #16903 from eileenmcnaughton/settting_better

Allow setting metadata to use the `table` option. Update example setting `default_invoice_page`.

Merge pull request #16993 from eileenmcnaughton/emailget

Add / make fit for purpose email.getlist api call

Merge pull request #17034 from mattwire/removeparamfromcompleteorder

REF Remove redundant param from completeOrder

Merge pull request #17031 from eileenmcnaughton/email4

[REF] Update Contact email form to use the trait for EmailCommon functions

Merge pull request #17035 from civicrm/5.25

5.25

Merge pull request #17015 from colemanw/having

APIv4 - Add support for HAVING clause

Merge pull request #17033 from seamuslee001/5.25

Revert "temp workaround for dev/core#1675"

Merge pull request #17029 from eileenmcnaughton/email3

[REF] Move generic instances of listTokens to trait

contribution is available in objects so use that instead of passing separately - it does not need to be passed by reference

Merge pull request #16959 from ray-wright/patch-4

Add email link in case summary

Merge pull request #17021 from totten/5.25-language-labels

CRM_Core_I18n - Provide a better label for new/unknown locales

Revert "temp workaround for dev/core#1675"

This reverts commit fcd23b884537a19ff2acbc98ca20a43f292c4c42.

Merge pull request #17020 from eileenmcnaughton/dedupe1

Start using apiv4 in test setup

[REF] Update Contact email form to use the trait for EmailCommon functions

Move generic instances of listTokens to trait

add client email link to manage case header

Merge pull request #16899 from mlutfy/mailingReportView

Mailing Report: hide the HTML preview

Start using apiv4 in test setup

Merge pull request #17022 from eileenmcnaughton/test_updates

[NFC][Test] cleanup on customDataTrait for tests

Merge pull request #16954 from eileenmcnaughton/email4

[REF] Move generic preProcess function to the trait

APIv4 - Add support for HAVING clause

[NFC][Test] cleanup on customDataTrait for tests

Enable test on merging country, it now works

Merge pull request #17023 from civicrm/5.25

5.25  to master

Merge pull request #17004 from pradpnayak/fixhookissue

Fixed fatal error for class not found when managed hook is invoked du…

Merge pull request #17019 from eileenmcnaughton/dupe

[NFC] Minor code cleanups

CRM_Core_I18n - Provide a better label for new/unknown locales

Overview
--------

Suppose you add new/unrecognized data files in the `l10n` folder - and then
you navigate to the screen `civicrm/admin/setting/localization?reset=1`.

Before
------

The new language appears in the admin UI as a blank item.

Every page in the app displays a warning about the unrecognized locale.

After
-----

The new language appears in the admin UI with a placeholder name (based on the code).

The warnings are not displayed.

Comment
-------

We've just had an issue where a new language was added to the `l10n`
data-set, and then all automated test-suites for all versions (incl `5.25`
RC and `5.24` stable) started to throw blocker errors on unrelated PRs.
This is because the `l10n` data is generally evergreen, but each branch/tag
of `civicrm-core` has the list hard-coded in multiple spots.

This patch tries to make the symptom less painful when mixing/matching
different revisions of `civicrm-core.git` and `l10n.git`.

Merge pull request #16837 from tunbola/case-api-case-clients

dev/core#1659: Fix Case.get API returning Case Clients As Part of Related Contacts

[NFC] Minor code cleanups

Merge pull request #17018 from civicrm/5.25

5.25

Merge pull request #17014 from seamuslee001/nl_be_5_25

add nl_BE language

Add / make fit for purpose email.getlist api call

The function CRM_Contact_Page_AJAX::getContactEmail is one of our  earlier  ajax attempts & this approach has been largely
replaced with entity Reference fields. In order to switch over we need to bring Email.getlist api to parity which  means
1) searching on sortname first, if less than 10 results on emails include emails
2) appropriate respect for includeWildCardInName (this should already be in the generic getlist)
3) filter out on_hold, is_deceased, do_not_email
4) acl support (should already  be part of the api).

The trickiest of these to support is the first - because we need to avoid using a non-performant OR
My current solution is the idea of a fallback field to search if the search results are less than the limit.
in most cases this won't require a second query but when it does it should be fairly quick.

Merge pull request #17016 from eileenmcnaughton/dupe

[REF] Clarify variable & tighten use.

Merge pull request #17013 from demeritcowboy/boo2

[REF] Use bool instead of boolean

[REF] Clarify variable & tighten use.

moveContactBelongings is only called from one place which passes in all the function variables, so we don't need defaults.

The last parameter,  is retrieved from self::getAffectedCustomTables which  always returns an
array - so all the handling for it being NULL can be removed....

add nl_BE language

Merge pull request #17012 from demeritcowboy/boo

[REF] Formatting and use bool instead of boolean

formatting and bool vs boolean

standard is bool instead of boolean

Fixed fatal error for class not found when managed hook is invoked during upgrade

Merge pull request #17009 from colemanw/multi-cleanup5

[REF] CustomField code cleanup to use isSerialized method

Merge pull request #16968 from colemanw/multi-cleanup

[REF] CustomField code cleanup

CustomField code cleanup to use isSerialized method

Merge pull request #17002 from eileenmcnaughton/deduped

[NFC][Test] Minor cleanup in test class

Merge pull request #16996 from colemanw/customCleanup

[REF] Cleanup customField prepareCreate function

Merge pull request #17001 from seamuslee001/dev_core_1673

dev/core#1673 Ensure that SQL statements are not duplicated in the de…

[NFC][Test] Minor cleanup in test class

Most notably extracting duplicated code. General cleanup

Merge pull request #17000 from demeritcowboy/comment-for-extension

[NFC] Add Comment to avoid someone removing field only used in CiviCase extension

Merge pull request #16986 from colemanw/importParser

[REF] Update custom data handing in contact import

Merge pull request #16997 from colemanw/textAreaAttrs

Fix setting custom field TextArea attributes

dev/core#1673 Ensure that SQL statements are not duplicated in the developer tab of reports

comment to avoid removing

Merge pull request #16994 from yashodha/error

add filter and sorting for receipt date in contribution report

Merge pull request #16995 from ivan-compucorp/CPS-130-case-details-empty

Case Details field is empty

Fix setting custom field TextArea attributes

The code was potentially overwriting attributes for existing fields during update ops

Cleanup customField prepareCreate function

CPS-130: Fix empty Details field when creating a Case

Merge pull request #16901 from eileenmcnaughton/settings

Fix bug on handling 'is_required'

add filter and soring for receipt date in contribution report

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

Update custom data handing in contact import

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

CustomField - switch statement to use data_type instead of html_type

Remove reference to obsolete field type and simplify statement

The history of this section of code is murky but from what I can tell,
this is applying special formatting to checkboxes. So instead of a double-negative conditional
"not field types that are not checkbox" let's just say "if checkbox".

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