[NFC] Minor code cleanup

Declare exceptions, use single quotes & strict comparison, do not declare unused params

Merge pull request #16552 from eileenmcnaughton/part_form

[REF] Fix static call to non-static function.

Merge pull request #16551 from totten/master-ext-tags

info.xml - Allow extensions to define a list of tags

Merge pull request #16554 from demeritcowboy/add-tag-ewarning

dev/core#1593 and dev/core#1536 - Remove unused code causing E_WARNING

Merge pull request #16525 from eileenmcnaughton/validate

Allow any casting done in Type::validate to bubble up to Request::retrieveValue

remove unused code causing E_WARNING

Merge pull request #16468 from mattwire/tokens_activitytokenprocessortrait

REF Refactor ActivityTokens to use a trait that can be shared with other entities

[REF] Fix static call to non-static function.

This non-static function was being called statically. On grepping I found it is only called from this
one place so  I moved it to the form

Merge pull request #16515 from mattwire/phpnotice_noteprivacy

Fix a PHP notice for users with limited permissions when loading cont…

Merge pull request #16533 from eileenmcnaughton/ex_angular

Do not fatally fail on angular pages if an extension is missing

info.xml - Allow extensions to define a list of tags

Overview
--------

This adopts a convention by which extensions may use `info.xml` to register tags, e.g.

```xml
<extension key="org.civicrm.foo" type="module">
  <tags>
    <tag>civicontribute</tag>
    <tag>payment-processor</tag>
    <tag>install-mandatory</tag>
  </tags>
</extension>
```

Before
------

To add *any* metadata to `info.xml` files, you must update the parser and providing
filtering/indexing.

After
-----

If you need to add very simple metadata (tags), you can do that without
extra parsing code.  The tag information can be accessed in the following
ways:

* Using the extension mapper
    ```php
    $mapper = CRM_Extension_System::singleton()->getMapper();
    $allTags = $mapper->getAllTags();
    $mandatoryExts = $mapper->getKeysByTag('install-mandatory');
    ```
* The `Extension.get` API will return the list of tags for each ext

Merge pull request #16548 from civicrm/5.23

5.23

Merge pull request #16547 from seamuslee001/5.23

[NFC] Ensure that annon permissions are correctly assigned when enabl…

Merge pull request #16549 from seamuslee001/style_fix_property_bag

[NFC] Fix style error

[NFC] Fix style error

Merge pull request #16532 from eileenmcnaughton/5.23

dev/core#1588 Fix regression where empty string is passed to PropertyBag

[NFC] Ensure that annon permissions are correctly assigned when enabling civicrm webtest module in backdrop

Merge pull request #16544 from seamuslee001/dev_core_1594

dev/core#1594 [TEST] Fix running of unit tests within an extension

Merge pull request #16543 from jmcclelland/issue1589

dev/core#1589 avoid error when merging record with null create date

Merge pull request #15998 from demeritcowboy/case-audit-skip-broken

dev/core#1366 Bypass nonfunctioning case audit interactive screen and go straight to report

dev/core#1594 [TEST] Fix running of unit tests within an extension

Merge pull request #16539 from eileenmcnaughton/text

Remove reference to mysql 5.0 & 5.1

Merge pull request #16542 from JGaunt/patch-3

Update CONTRIBUTORS.txt

Merge pull request #16540 from eileenmcnaughton/part_pend

[REF] Remove never used property

avoid error when merging record with null create date

https://lab.civicrm.org/dev/core/issues/1589

Update CONTRIBUTORS.txt

Added in mine and my colleagues name.

Merge pull request #16527 from alexymik/added_by

Change "Added By" to "Added by"

Remove never used property

I grepped for _roleId & found it in only these 2 places - removing

Merge pull request #15390 from kewljuice/patch-4

Check email when creating a user in drupal 8

Remove reference to mysql 5.0 & 5.1

Subset of https://github.com/civicrm/civicrm-core/pull/16084

Merge pull request #16516 from colemanw/request

APIv4 - merge ActionUtil with Request::create

Merge pull request #16523 from yashodha/header

add column for report

Merge pull request #16537 from civicrm/5.23

5.23

Merge pull request #15824 from mfb/email-search-speed-boost

Speed boost for civicrm/ajax/checkemail

Merge pull request #16467 from ixiam/dev-1110

fix contribution summary report's statistics when grouping and having

Merge pull request #16451 from mattwire/settingreadonly

Settings Fix setting readonly attribute

Merge pull request #16535 from eileenmcnaughton/523rel

dev/core#1592 fix regression on relation active period

Merge pull request #15330 from mattwire/paymentprocessor_testmodelivemode

Fix issues with retrieving supportsTestMode/supportsLiveMode for payment processors

Merge pull request #16534 from eileenmcnaughton/523m

dev/core#1551 regression fix

Merge pull request #16526 from eileenmcnaughton/valid_calls

Fix calls to Request::retrieve

dev/core#1592 fix regression on relation active period

Per https://lab.civicrm.org/dev/core/issues/1592 the custom query  for this field is being
ignored now it's using datepicker with relevant format changes. This  causes the
custom function for the field to  be called

dev/core#1551 regression fix

Now that mailing_id is defined as field metadata the form main metadata fn is trying to handle it, However
the form is casting it to the wrong array format -https://github.com/civicrm/civicrm-core/blob/86f20cc17dde028977130f84feaf3b1d760ca57d/CRM/Contact/Form/Search.php#L649

We could fix  this & rip it out of Mailing_BAO_Query::where but there is still a gap  in  that
there is no pseudoconstant recorded for the field so the qill doesn't work & there is a gap in
pre-parsing for the form 'IN'

Hence this seems like a good approach to remove the regression in the immediate term

Do not fatally fail on angular pages if an extension is missing

If an extension is missing trying to create a new mailing will totally fail - on QF pages we just get a
not-very-clear message

dev/core#1588 Fix regression where empty string is passed to SettingsBag

We have a scenario where the checkbox is presented but is optional. This is then in 'params' which is passed through to
PropertyBag. The value is equal to '' so it fails to validate when we use it to set the value on the PropertyBag.

I don't think we lose anything meaningful by not setting an empty string and it avoids this error so we should
merge & release as a regression fix IMHO. If we want to revist then we should do that in master

https://lab.civicrm.org/dev/core/issues/1588

Merge pull request #16530 from civicrm/5.23

5.23

Merge pull request #16521 from seamuslee001/dev_core_1489_2

dev/core#1489 Include hook_civicrm_managed into the list of upgrade f…

Merge pull request #16477 from demeritcowboy/unit-test-env

[TEST] dev/core#1572 Unit test environment not working on windows after recent CodeGen updates

Fix calls to Request::retrieve

Do not pass in values that are set by default. Use retrieveValue  rather than passing inn the NULL object

Change "Added By" to "Added by"

Allow any casting done in Type::validate to bubble up to Request::retrieveValue

The validate function returns
- NULL if the passed in value is not valid
- THe passed in value if valid or
 - The passed in value cast to an integer if the value is valid and the type is Int or Integer

By always using the returned value we allow it to be cast, where appropriate

APIv3 - Handle input errors

add column for report

Merge pull request #16522 from totten/master-simplecache-patchurl

dev/core#1562 - composer.json - Fix E2E tests run on D8 build (via "patches") #16427

Merge pull request #16512 from yashodha/header

Relationship report - add sort order for end date

(dev/core#1562) composer.json - Fix E2E tests run on D8 build (w/"patches" and URLs)

dev/core#1489 Include hook_civicrm_managed into the list of upgrade freindly hooks

Remove patch files that are now loaded from URL

The `composer.json` was updated to get these patch files via URL instead of
via local/relative path.

Future maintainers who wish to revise these patches will need to post new
revisions somewhere with a durable URL *before* they can update
`civicrm-core`.

Keeping the files in here wrongly implies that you can revise the files
in-situ - their presence is misleading.

Merge pull request #16519 from civicrm/5.23

5.23

Merge pull request #16520 from agh1/5.23.0-release-notes

5.23.0 release notes

Merge pull request #16517 from agh1/5.23.0-release-notes

5.23.0 release notes: initial run

Merge pull request #16518 from greenpeace-cee/fix-xoauth-imap

dev/mail#59 - Fix use of useXOAUTH2 constant instead of var

dev/mail#59 - Fix use of useXOAUTH2 constant instead of var

This fixes an issue where a missing $ before the useXOAUTH2 variable
prevents IMAP authentication with XOAUTH2 from working.

5.23.0 release notes: added boilerplate

Merge pull request #16513 from eileenmcnaughton/credit_note_setting

Move  settings definition on contribution settings form to metadata.

APIv4 - merge ActionUtil with Request::create

The Request::create function was broken for v4, and ActionUtil had been serving the same purpose but in a non-broken way.

5.23.0 release notes: raw from script

Fix a PHP notice for users with limited permissions when loading contact summary

Move credit_note_definition on contribution settings form to metadata.

This moves the code that allows settings to be added to forms by metadata from the Generic setting form to the setting trait.

This means many more forms will support injecting settings by metadata. Specifically we switch across the Contribution preferences form,
making credit_note_prefix a metadata-added field. This is preliminary to moving that to an extension & allows us to inject it
in by simply defining the metadata in a hook

add sort order for end date

Merge pull request #16511 from colemanw/kernel

API Kernel - cleanup deprecated fn & unused param

Merge pull request #15859 from jitendrapurohit/dev-mem-17

dev/mem#17 - Deleting memberships does not delete its related line item

API Kernel - cleanup deprecated fn & unused param

Merge pull request #16510 from colemanw/ws

crmSelect2 - fix whitespace in tooltip

crmSelect2 - fix whitespace in tooltip

Merge pull request #16508 from colemanw/escape

Common.js - remove duplicate function

Merge pull request #16507 from colemanw/title

dev/core#1587 Select2 - Use description as title for each option

Common.js - remove duplicate function

This function did the same thing as the one in lo-dash

Select2 - Use description as title for each option

Merge pull request #16504 from acrollet/master

dev/financial#117 - Add link metadata for payment edit link

Merge pull request #16499 from pradpnayak/caseAuditReport

Added conditional check so that it can be altered by hook

Merge pull request #16505 from eileenmcnaughton/part_pay_dep

Add deprecation notices on PartialAmount params

Merge pull request #16497 from totten/master-pear-mail-delg

(REF) Move CIVICRM_MAIL_LOG logic from patch-files to wrapper-class

composer.json - Patch URLs should be more concrete, less symbolic

The previous commit made used patch URLs in the format:

`https://raw.githubusercontent.com/civicrm/civicrm-core/master/{FILEPATH}`

This is symbolic in that the `master` does not refer to a specific piece of content; it refers
subjectively to the current developmental code. This creates a few problems:

* When `civicrm-core` is tagged or branched, the `composer.json` still points
  to `master`.
* If you need to update the patch-file, you send in a PR with a different
  patch URL, right? But any builds based on this PR would still use the
  canonical `master` rather than the proposed revision.

Resolution: Don't point to patch-files symbolicly.  Instead, address them
specifically.  This revision uses past Github commits (because that's easy),
but it doesn't have to be in Github per se.  It should be somewhere that's
reliable and unlikely to change (preferrably with addressing by content
checksum).  In the Drupal community, you see a lot of examples pointing to
file-attachments in the issue-tracker.

LAB-1576 - Use URLs for patches, to avoid local path issues with composer-patches

Trying Github raw URLs this time ...

CRM_Utils_Mail::send - Adjust hack to work with filtered mailer

(REF) CRM_Utils_Mail - Split LoggingMailer into FilteredPearMailer and Logger

1. Captures more context (i.e. the original driver and params)
2. Changes various property names to avoid potential for conflict with delegated properties
3. Adds `addFilter($id, $func)` so that one can move more filters in here
4. Consolidates the various references ot CIVICRM_MAIL_LOG* into one file.

Merge pull request #16360 from artfulrobot/case-activities-setting

Add CiviCase option for showing case activities in normal views

Merge pull request #15410 from herbdool/cloud-21

dev/cloud-native#21 Don't cache the full path of extensions so they don't break with dynamic paths

Add deprecation notices on PartialAmount params

We've removed all the places that pass these in so these 2 places handling them are the only remaining references.

All calling code now uses Payment.create to add the payments - yay. Removing these fully will clean up the code
notably

CRM_Admin_Form_Setting_Smtp - Use the same mailer as CRM_Utils_Mail

When civicrm-core used patches to change the behavior of `Mail_mail` (etal),
this was fine.  The preceding commit removes the patches and uses a wrapper
instead - so it's more important to get the same mailer as the
regular CRM_Utils_Mail logic.

Add CiviCase option for showing case activities in normal views

Different aproach to tackling #16353

Fix array-to-string notice; Improve activity action links

remove upgrader - as per Pradeep's suggestion

update version added

tweak way description is gotten

update activity tests to check civicaseShowCaseActivities setting

Fix another array-to-string conversion error

Remove File On Case from activity View; add Manage Case

fix case activity links when using caseShowCaseActivities

remove ctx parameter for case activity links

various

US-Englishized Case settings texts.

fix permission name

Add link metadata for payment edit link

Merge pull request #16502 from yashodha/header

remove unnecessary file

Put back the original line

Putted back the original line as suggested by @demeritcowboy

Merge pull request #16487 from mlutfy/core1019

dev/core#1019 Calculate.tpl: fix the Total Amount currency formatting.

Merge pull request #16442 from eileenmcnaughton/pree

Fix backoffice participant partial payments to be stdised & not miscalculate net_amount

remove unnecessary file

Merge pull request #16500 from eileenmcnaughton/fatal

Remove fatal  from  form

Remove fatal  from  form

Merge pull request #16475 from JMAConsulting/core_1577

dev/core#1577 : Custom Group Types not filterable