Fix CRM_Utils_JS::dedupeClosures to ignore comments

This runs stripComments BEFORE dedupeClosures so that comments don't interfere with the deduping.
Also fixes stripComments to remove the line break as well as the comment line.

Merge pull request #17714 from colemanw/jsDocblocks

NFC cleanup docblocks in CRM_Utils_JS

Merge pull request #17715 from civicrm/5.27

5.27

Merge pull request #17713 from MegaphoneJon/multi-record-import-fix-527

Multi record import screen fix

NFC cleanup docblocks in CRM_Utils_JS

Merge pull request #17712 from civicrm/5.27

5.27

Merge pull request #17711 from colemanw/useStrict

CRM_Utils_JS - also dedupe 'use strict' directive when deduping closures

Multi record import screen fix

Merge pull request #17676 from eileenmcnaughton/ct

[REF] Cleanup function for retrieving contact types.

Merge pull request #17250 from eileenmcnaughton/act_search

Remove slow join from activity search

Merge pull request #17710 from kcristiano/5.27-api4-joomla-install

dev/joomla/#28 5.27 Upgrade fails

CRM_Utils_JS - also dedupe 'use strict' directive when deduping closures

dev/joomla/#28 5.27 Upgrade fails

Failure is due to 'Undefined class constant 'POST_SELECT_QUERY'  see https://lab.civicrm.org/dev/joomla/-/issues/28

Merge pull request #17708 from demeritcowboy/temp-workaround-case-sort

dev/core#1844 - Temporary workaround to avoid all PR's failing tests

Merge pull request #17677 from seamuslee001/dev_core_1833

dev/core#1833 Change default value of participant_listing_id to NULL …

Remove slow join from activity search

This removes the unindexed join from activity search. The join is used to apply component
permission filtering and to retrieve the activity_type - however, this latter is still rendered
by the pseudoconstant handling

temporary workaround to avoid all PR's failing tests

Merge pull request #17704 from colemanw/api4Count

APIv4 - Improve row_count to work with HAVING, GROUP BY, and SELECT

Merge pull request #17412 from bhahumanists/remove-summary-row

Remove summary row from charts, when rollup used

Remove summary row from charts, when rollup used

Issue: https://lab.civicrm.org/dev/report/-/issues/40

Merge pull request #17687 from mattwire/cancelSubscriptionSelfService

Add selfService flag to cancelSubscription getText so we can display different text for each case

APIv4 - Improve row_count to work with HAVING, GROUP BY, and SELECT

This changes the meaning of $result->count(), to give a total count of filtered items, ignoring limit and offset.
For DAO queries, you must select 'row_count' to trigger this behavior, as it requires a seperate query.

REF - Divorce API4SelectQuery from base class SelectQuery

These two classes had grown apart to the point they share almost nothing in common,
and the link was more confusing than helpful.

Merge pull request #17389 from seamuslee001/dev_core_1776

dev/core#1776 Ensure that all fee financial item transaction date is …

Merge pull request #16956 from jitendrapurohit/core-890

dev/core#890 - Multiple line item shown on view contribution if parti…

Add selfService flag to cancelSubscription getText so we can display different text for each case

Merge pull request #17697 from MegaphoneJon/multi-record-import-fix

Multi record import screen fix

Merge pull request #17688 from artfulrobot/artfulrobot-trxn-id-and-date

Payment.create should not set contribution date to today

Multi record import screen fix

Merge pull request #17664 from jitendrapurohit/core-1829

dev/core#1829 - Custom Date field with format=yy displays calendar ic…

Merge pull request #17689 from demeritcowboy/fintest-case-sensitive

[Test framework] dev/wordpress#61 - Fix case-sensitive spelling in unit test

Merge pull request #17691 from MegaphoneJon/core-1840

core#1840 - Can't add/change recipient on non-bulk SMS

Payment.create should not set contribution date to today (PR 17688)

Merge pull request #17690 from colemanw/psr4

Use psr-4 instead of psr-0 in Civi directory

Merge pull request #17692 from colemanw/CONCAT

APIv4 - Add more SQL functions

Add user-facing titles for sql functions

APIv4 - Add more SQL functions

core#1840 - Can't add/change recipient on non-bulk SMS

Use psr-4 instead of psr-0 in Civi directory

This is a followup to #17105 which attempted the same thing but without success.
This finally gets classnames with underscores to work correctly in the Civi directory.

Merge pull request #17686 from demeritcowboy/medium-index

dev/core#1634 - Remove database index on medium_id from civicrm_activity

fix case-sensitive spelling

remove index on medium_id

Merge pull request #17685 from colemanw/entityMeta

APIv4 - add meta-metadata to supplement #17527

Merge pull request #17558 from seamuslee001/require_mysql_565

dev/core#1681 Block sites from upgrading if on a MySQL version less t…

APIv4 - add meta-metadata to suppliment #17527

Merge pull request #17680 from seamuslee001/master

Revert "dev/core#1603 remove places where taxAmount is rounded"

Merge pull request #17621 from colemanw/sortOptions

Sort options without relying on unavailable buildOptions param

Merge pull request #17682 from colemanw/api4ExpJoin

APIv4 Explorer - UI support for join ON clause

APIv4 Explorer - Update pluralize fn to match CRM_Utils_String

APIv4 Explorer - Support selectable fields from explicit joins

Merge pull request #17681 from demeritcowboy/customfieldoption-name-label

dev/core#1817 - Editing a custom field option value from the custom field admin screen changes the name field

APIv4 Explorer - UI support for join ON clause

Merge pull request #17667 from colemanw/taskIcons

Add icons to standalone contact tasks

don't change name field when updating option value

Add icons to standalone contact tasks

Merge pull request #17666 from eileenmcnaughton/entity_log

Throw exception if id not passed into discard

Revert "dev/core#1603 remove places where taxAmount is rounded"

This reverts commit 777d61015ae35bb60a08f779d9aa8cb8dc9df708.

Merge pull request #17679 from civicrm/5.27

5.27

Merge pull request #17668 from seamuslee001/intl_check

dev/translation#48 Add check for PHP-Intl Extension

Merge pull request #17673 from seamuslee001/expandaccordion_error

[REF] Ensure that our custom error handler is called when jquery vali…

Merge pull request #17678 from seamuslee001/master

[REF] Fix javascript syntax error

[REF] Fix javascript syntax error

dev/core#1833 Change default value of participant_listing_id to NULL from 0

dev/translation#48 Add check for PHP-Intl Extension

[REF] Cleanup function for retrieving contact types.

This switches to using the apiv4 as part of trying to eliminate calls to executeQuery where the DAO is passed in
like

```
       = CRM_Core_DAO::executeQuery(, [],
        FALSE, 'CRM_Contact_DAO_ContactType'
      );
```

Note that this file seems to be the only place this is done.

Also note that I added tests first & the changes to the test highlight what is changed in the PR in the output - namely
- stricter type casting (courtesy apiv4)
- same keys present for all types

Merge pull request #17674 from seamuslee001/contacttype_test_mysql8_ordering_issue

[NFC] Fix Unit test failure on MySQL 8 due to mysql ordering issue

[NFC] Fix Unit test failure on MySQL 8 due to mysql ordering issue

Merge pull request #17588 from artfulrobot/artfulrobot-property-bag-support-empty

Improve PropertyBag handling of offsetGet and custom properties; add …

Merge pull request #17644 from eileenmcnaughton/export_fix_standalonne

Export fix

Merge pull request #17672 from mattwire/onbehalfof

Fix validation of select2 fields in "On behalf of Organisation" block

Export - further cleanup export class to remove unnecessary switch

Fix issues with select2 validation when it is part of hidden 'On behalf of organisation' block

Whitespace cleanup

Merge pull request #17671 from seamuslee001/select2_error_message

[REF] Fix jquery error message display on select2 field validation

[REF] Ensure that our custom error handler is called when jquery validates and expand any collapsed accordions that are hiding required fields that haven't been filled in

Merge pull request #17670 from seamuslee001/fix_allow_clear_radio

[REF] Fix allowing users to clear values for raido custom fields when…

Merge pull request #17665 from eileenmcnaughton/ct

Fix master regression on inactive contact types

[REF] Fix jquery error message display on select2 field validation

[REF] Fix allowing users to clear values for raido custom fields when not required

Merge pull request #16488 from mattwire/quickform_requiredfields

Pass required attribute for quickform fields through to form

Fix master regression on inactive contact types

Removing the WHERE type.name = NULL here makes sense

https://github.com/civicrm/civicrm-core/commit/351e8d470a95e1a610280731706c816033fb79fa#diff-ae3cfb8afd3fe1ab8f471b3a069563cdL244

but a few lines later an extra condition is added - we need the WHERE there to prevent it being
added to the join clause.

Note that I did something a bit yuck with the static. I want to properly re-write the function
with consistent caching but after it's under testing, hence a quick fix

Merge pull request #17615 from seamuslee001/phpunit7_new

Use PHPUnit7 as the primary test runner for PHP versions >= 7.1 when using the phpunit script

dev/core#1681 Block sites from upgrading if on a MySQL version less than 5.6.5

Add in details of proposed MySQL 5.6 drop as well

Merge pull request #17642 from eileenmcnaughton/trxn

[REF] Extract getTestTrxnID

Merge pull request #17654 from eileenmcnaughton/cancel

Fix A.net to retrieve processor id from property bag

[REF] Use PHPUnit7

Fix tap printer

Merge pull request #17663 from seamuslee001/prep_phpunit7

[NFC] Fix tests in prepartion for phpunit7

Throw exception if id not passed into discard

https://lab.civicrm.org/dev/drupal/-/issues/123 - may not be related but this seems a good precaution

dev/core#1829 - Custom Date field with format=yy displays calendar icon that doesn't work

[NFC] Fix tests in prepartion for phpunit7

Merge pull request #17660 from seamuslee001/fixflexmailer

[REF] Fix Compatability issue with flexmailer caused by the PHP7.4 fix

Merge pull request #17662 from seamuslee001/master

Revert "[NFC] Fix tests in prepartion for phpunit7"

Revert "[NFC] Fix tests in prepartion for phpunit7"

This reverts commit 11a56cfe5f918a54284b7614a9489d53c9c41208.

Merge pull request #17661 from seamuslee001/prep_phpunit7

[NFC] Fix tests in preparation for phpunit7

[NFC] Fix tests in prepartion for phpunit7

[REF] Fix Compatability issue with flexmailer caused by the PHP7.4 fix

Merge pull request #17608 from seamuslee001/dev_translation_48

dev/translation#48 Implement Brick/Money to better handle money values

Merge pull request #17612 from eileenmcnaughton/format

[NFC] Remove instances where html is passed to crmMoney

dev/translation#48 Implement Brick/Money to better handle currecny values

Added in test case as per Eileen comment so we know we can handle large floats

Fix A.net to retrieve processor id from property bag