Cleanup sql query

On digging into this query I found that is uses protected words like value & count
so I have fixed the query to no longer use those & re-formatted to add comments.

I suspect the actual fix might need to sit around pending community input so I think it is good to
get this tidy up merged to make it all clearer

Merge pull request #23507 from eileenmcnaughton/import_match2

[REF] Remove unreachable code

Merge pull request #23495 from eileenmcnaughton/import_unused

[REF] [Import] Remove unused parameter

Merge pull request #23493 from eileenmcnaughton/import_rel

[Import] Remove silly param

[REF] Remove unreachable code

Merge pull request #23500 from civicrm/5.50

5.50 to master

Merge pull request #23497 from totten/5.50-schedrem-bool

Restore `civicrm_action_schedule.limit_to` (5.50)

[Import] Remove silly param

Merge pull request #23494 from eileenmcnaughton/import_if

[Import] Remove `if (1) {` (whitespace intensive)

Merge pull request #23492 from totten/master-silly-method

(REF) CRM_Upgrade_Form - Remove silly method `runQuery`

FiveFortyNine - Tweak task title

FiveFortyNine - Revise message about "limit to"

1. This fixes a problem where the message shows at the opposite-of-correct time
   (ie it should display on builds that have already run 5.49.{beta,0,1}, but
   it actually displayed on builds that have not run 5.49.{beta,0,1})

2. This makes the pre+post messages match.

FiveFortyNine - Don't munge soft credit 'limit-to'

I'm not sure what it even means to limit a scheduled-reminder to a "soft_credit_type".

But it's a thing - I can use the 5.48 GUI to create it. Here's how it looks in the DB (in 5.48):

+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+
| id | title                                    | recipient        | limit_to | recipient_manual | recipient_listing | group_id |
+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+
| 11 | Limit To - Soft Credit                   | soft_credit_type |        1 | NULL             | in_memory_of      |     NULL |
+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+

The `changeBooleanColumnLimitTo()` was coercing down to `limit_to=null` because it lacked `recipient_manual` and `group_id`.

FiveFortyNine - Fix guard on `changeBooleanColumnLimitTo()`

add pre+post upgrade message

FiveFortyNine - Draft message for 5.49.2

Revert civicrm_action_schedule.limit_to boolean column to be NULL

[REF] [Import] Remove unused parameter

[Import] Remove if (1) {

(REF) CRM_Upgrade_Form - Remove silly method `runQuery`

This function just wraps `CRM_Core_DAO::executeQuery`, and it's only
used once.

Merge pull request #23476 from eileenmcnaughton/import_dupe_more

[Import] Simplify checking contact type is valid

Merge pull request #23489 from colemanw/filterCustomFields

APIv4 - Filter custom fields based on supplied values

Validate subtype change

[Import] Simplify checking contact type is valid

[Import] Duplicate finding cleanup

Text user supplied dedupe rule is used during import.

[REF] [Import] Extraction of get related contacts function

Merge pull request #23484 from eileenmcnaughton/import_get_type

Cleanup gender handling in contact import + add test

APIv4 - Filter custom fields based on supplied values

This allows targeted getfields for a particular entity or type of entity

Merge pull request #23461 from eileenmcnaughton/import_woohoo

[REF] [Import] Remove now-unused code

Cleanup gender handling in contact import

Merge pull request #23470 from eileenmcnaughton/import_dataform

[REF] [Import] Stop calling Parser in MapField mode (contact import)

Merge pull request #23483 from eileenmcnaughton/import_get

Painstaking unravelling of function

Merge pull request #23475 from eileenmcnaughton/imp_fold

[Import] Add tests for contact type & contact subtype handling

Merge pull request #23452 from colemanw/exportCustomFieldWithOptions

APIv4 - Export options along with custom field

Fold array building in

Set up related Contact Key earlier

Further simplification

Simply value building

Simplify related contact assignment

Remove always-true-if

Merge pull request #23481 from alifrumin/rn5.50

[NFC] First pass at 5.50 release notes

[NFC] first stab at 5.50 release notes

Merge pull request #23480 from demeritcowboy/too-many-questions

Remove extra question mark from url

remove extra question mark

Merge pull request #23469 from mlutfy/schemaTranslation

Fix Schema System Check translation and show missing Custom Field ID

APIv4 - Export option groups along with custom fields

Fix Schema System Check translation

Merge pull request #23446 from colemanw/searchKitDefaultAgg

SearchKit - Pick better default for aggregate functions

Merge pull request #23472 from colemanw/exportMaxLentgh

APIv4 - Respect max length for name with export action

Figure out & test what happens with  ext identifier & type, subtype combos

Merge pull request #23464 from eileenmcnaughton/import_on_dup

[REF] [Import] Minor simplification

APIv4 - Respect max length for name with export action

Merge pull request #23459 from darrick/dev/core#2300

dev/core#2300 Can not use Custom Fields defined on a contact_sub_type in dedupe…

[REF] [Import] Stop calling Parser in MapField mode

Merge pull request #23463 from eileenmcnaughton/import_proc

[REF] [Import] Remove unused variable

Merge pull request #23466 from totten/master-timeout-guzzle

CRM_Utils_Check - Catch Guzzle exception, same as core exception

Merge pull request #23457 from eileenmcnaughton/import_more_part

Consolidate datasource form - event import

CRM_Utils_Check - Catch Guzzle exception, same as core exception

Overview
--------

`checkExtensions()` is supposed to show status-messages which report on
any out-of-date extensions.  This requires a list of available updates.

Alas, the Internet is a big and scary place, and sometimes the list is
unavailable.  DNS servers, HTTP servers, timeouts, yadda yadda.  If it can't
get the list of extensions, it should show a status-message about this
problem.

Before
------

Throws an exception, causing a full failure in the page-view that does
the status-check.

After
-----

Catches the exception. Reports a status-message about the failure.

Comments
--------

It appears that this was probably the behavior before (ie `checkExtension()`
already has a try/catch for this kind of problem). However, the underlying
HTTP request was changed a versions back (ie from `HttpClient` to `Guzzle`)
which means that errors can get reported in other ways.

To simulate errors, I did this:

1. Run buggy/placeholder TCP service (`nc -l 8199`)
2. Hack `CRM_Extension_Browser::__construct()` to set `$this->repoUrl` to `http://localhost:8199`
3. Run `cv api System.check`

Merge pull request #23289 from eileenmcnaughton/import_summ

[REF] [Import] Activity - clean up copy & paste

Merge pull request #23462 from eileenmcnaughton/import_ex

[Import] [Ref] Remove unused variable

Merge pull request #23460 from darrick/dev/core#492

dev/core#492 Fix Dedupe rule creation should display labels of contact types, in c…

[REF] [Import] Minor simplification

At this stage value already holds the phone_type_id, location_type_id, im_provider_id, website_type_id
if any, so we don't need to throw it away & start again.

[REF] [Import] Remove unused variable

[Import] [Ref] Remove unused variable

[REF] [Import] Remove now-unused code

Merge pull request #23455 from eileenmcnaughton/import_validate_mum

Import validate related contacts in the same way as the main contact

Fix Dedupe rule creation should display labels of contact types, in case they have been customised.

Fix Can not use Custom Fields defined on a contact_sub_type in dedupe rule

Consolidate datasource form - event import

Merge pull request #23420 from mattwire/custombasepage

Allow for custom base page in AJAX requests

Use a more unique placeholder url path for CiviCRM AJAX requests

Merge pull request #23444 from eileenmcnaughton/import_get

[REF] [Import] Minor code cleanup

Merge pull request #23453 from eileenmcnaughton/import_no_map_field

Remove sets for no-longer gotten vars

[Import] Apply the same contact validation to related contacts as the main contact

[Import][REf] Cleanup required field checking

Remove sets for no-longer gotten vars

Merge pull request #23450 from totten/master-karma-comps

 (NFC) karma - Enable all Civi components (not just CiviCase)

Merge pull request #23448 from totten/master-typo

(NFC) CRM_Utils_Hook - Typo

Merge pull request #23442 from eileenmcnaughton/import_data

[REF] [Import] Parser cleanup

(NFC) karma - Enable all Civi components (not just CiviCase)

This is a small cleanup (suggested by @colemanw) that makes the `karma.conf.js` appear more general.

It modifies the startup process for a test-runner; as long as the test-runner still gives the
same output, it should be safe.

Before
------

`karma` loads baseline Angular modules and CiviCase Angular modules.

After
-----

`karma` loads baseline Angular modules and any/all component-based Angular modules

Comment
-------

Why was it only CiviCase?  Not certain...  but I *think* it's because
(historically) `crmMailing.ang.php` did extra work to build `settings`, and
this extra work was a bit too heavy.  That work has moved (`settings` became
`settingsFactory`) and maybe makes this safer.

(NFC) CRM_Utils_Hook - Typo

karma - Allow execution with debug_mode=1. Hide jsonFormatter.

Before
------

If you enable `debug_mode=1` and run `karma`, it produces a large number of failures:

```
PhantomJS 2.1.1 (Mac OS 0.0.0): Executed 81 of 81 (55 FAILED) (2.686 secs / 2.387 secs)
```

After
-----

If you enable `debug_mode=1` and run `karma`, it passes:

```
PhantomJS 2.1.1 (Mac OS 0.0.0): Executed 81 of 81 SUCCESS (7.232 secs / 6.95 secs)
```

Comments
--------

The `jsonFormatter` plugin is a DX nicety; when browsing a page with `?angularDebug=1` and with `debug_mode=1`, it will load extra
files so that `<crm-ui-debug>` panels look pretty. But ordinarily, loading these files is extraneous.

[REF] [Import] Remove last instance of passing fieldTypes through get->set

Merge pull request #23447 from agh1/5.50.0-releasenotes-initial

5.50.0 release notes initial run

Merge pull request #23332 from darrick/dupeQuery

(REF) CRM_Dedupe_BAO_DedupeRule - Support third party fields and tables in rules

5.50.0 release notes: added boilerplate

5.50.0 release notes: raw from script

SearchKit - Pick better default for aggregate functions

Improves the metadata for SQL functions as well.

Merge pull request #23441 from colemanw/api4TestCleanup

[TEST ONLY] APIv4 - ensure test records are cleaned up in non-transaction tests

Merge pull request #23443 from civicrm/5.50

5.50

APIv4 Unit test fixes - ensure test records are cleaned up in non-transaction tests

Use the new `createTestRecord` and `saveTestRecords` in non-transaction test classes
so the created records are automatically cleaned-up.

Merge pull request #23436 from colemanw/searchKitPagerDefaults

SearchKit - Default to simple pager when creating new table display

Merge pull request #23434 from colemanw/groupByFn

SearchKit - Allow functions in the GROUP BY clause

Use submittedValues for usps

[REF] [Import] Minor code cleanup

This chunk of code effectively constructs an array 'value' which holds the location
fields from the mapping. If the array is not empty then
use it as an array, otherwise use just the value

Merge pull request #23440 from totten/5.50-since-when

(NFC) mixin/mgd-php@1 - Twiddle `@since`

Merge pull request #23427 from eileenmcnaughton/get_mapped_params

[REF] [Import] Simplify if - exit loop early

[REF] [Import] Move datasource interaction to datasource class

Merge pull request #23439 from colemanw/dashboardUpdate

Dashboard - Efficiency tweak