Remove always-true-if

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

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

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…

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

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 #23332 from darrick/dupeQuery

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

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

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

Overview
--------

Switch `@since` from "modified" version to "introduced" version.

Comment
-------

The [spec for `@since`](https://docs.phpdoc.org/guide/references/phpdoc/tags/since.html)
is more open ("when an element was introduced _or_ modified"), but this matches similar
usages in the same repo.

Merge pull request #23409 from demeritcowboy/psrlog

dev/drupal#176 - Allow wider range of psr-log versions to be installed

Merge pull request #23437 from colemanw/noTransform

SearchKit - Don't transform case for fieldsets on admin screen (more translation-friendly)

Merge pull request #23438 from colemanw/sortInactiveDashlets

Dashboard - Sort available dashlets alphabetically, add search box

Merge pull request #23435 from mlutfy/removeOldL10nDir

Translation: remove support for CiviCRM pre-4.5 l10n file locations

Dashboard - Efficiency tweak

Use static binding for non-changing values

Dashboard - Sort inactive dashlets by label, add search box.

This makes dashlets easier to find if there is a large number.

SearchKit - Don't transform case for fieldsets on admin screen (more translation-friendly)

Merge pull request #23432 from colemanw/api4TestCleanup

[REF] APIv4 - Simplify entity creation in test suite

SearchKit - Default to simple pager when creating new table display

Before: Pager default included "row count" and "adjustible page size".
After: Defaults to simple pager without the extras.

The extra stuff isn't always useful and can break the layout on small screens or dashlets,
so I think a better default is to exclude rather than include them.

Translation: remove support for CiviCRM pre-4.5 l10n file locations

SearchKit - Allow functions in the GROUP BY clause

Fixes dev/core#2485

Merge pull request #23433 from colemanw/dashletHover

Dashboard - Show full title when hovering an inactive dashlet

APIv4 - Simplify entity creation in test suite

Removes the old v3 creation and hardcoded sample data in favor of
new `createTestRecord` and `saveTestRecords` methods which automatically
handle cleanup during tearDown.

Dashboard - Show full title when hovering an inactive dashlet

[REF] [Import] Parser cleanup - remove now unused

Merge pull request #23428 from eileenmcnaughton/import_field_keys

[REF] [Import] Remove unused field get action (death to copy & paste)

Merge pull request #23421 from eileenmcnaughton/import_obs

[Import] Stop passing unused parameter

Merge pull request #23425 from eileenmcnaughton/import_member

[Ref] [Import][Member] Remove unused params

[REF] [Import] Remove unused field get action

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

Merge pull request #23426 from civicrm/5.50

5.50 to master

Merge pull request #23423 from totten/5.50-mixin-since

(NFC) mixin/**.php - Add @since tags

[Ref] [Import][Member] Remove unused params

Merge pull request #23422 from totten/5.50-mgd-ver

ManagedEntities - Update version number for mgd-php

Merge pull request #23416 from eileenmcnaughton/import_create_job

[REF] [Import] Create user-job records for Memership imports

tools/mixin/bin/mixer - Display the `@since` data

(NFC) mixin - Add `@since` tags

ManagedEntities - Update version number for mgd-php

This is a follow-up/correction for #22959 (circa 5.50.alpha1; specifically
fdc67a75b2f20aea9f0961683bd8a59c78755d5f). The prior revision had updated the
behavior of `mgd-php`, so it should have also updated the version-number.

Stop passing unused parameter

Allow AJAX paths to work properly with a custom basepage (as can be used on WordPress frontend)

allow more log

Merge pull request #23418 from eileenmcnaughton/import_valid3

[Import] Use table rather than csvs for no-match & unparsed

Merge pull request #23317 from seamuslee001/php81_trim_codegen

[REF] Fix PHP8.1 issue with passing NULL to trim()

Use table rather than csvs for no-match & unparsed

Add test for NO_MATCH flow

[REF] Fix PHP8.1 issue with passing NULL to trim()

Merge pull request #23385 from colemanw/defaultToIndividual

SearchKit - Default to search for individuals

Add test to ensure emails are validated & remove duplicate validation

[Ref][Import] Start adding testing to validation, simplify

[Import][Ref] Calculate relationship type in parser

This is being calculated
here
https://github.com/civicrm/civicrm-core/blob/c169525f72b1f956d58cec1e4a15b0f57de3eabf/CRM/Contact/Import/ImportJob.php#L132-L136

And this change moves the calculation into the
parser class so that, along with the other arrays
it can be removed as an input parameter

Merge pull request #23401 from eileenmcnaughton/import_data_source

(REF) Importer - Remove unused parameters. Simplify signature.

[REF] [Import] Create user-job records for Memership imports

Merge pull request #23414 from civicrm/5.50

5.50 to master

Merge pull request #23412 from colemanw/inputTypes

Afform - Fix RC regression of missing input types

Merge pull request #23411 from civicrm/5.50

5.50

Merge pull request #23410 from colemanw/api4ConformanceTestBase

[TEST ONLY] APIv4 - Remove unused extras from ConformanceTest

Merge pull request #23382 from eileenmcnaughton/import_simp

[REF] Import - make condition clearer

Merge pull request #23290 from eileenmcnaughton/import_cont

[REF] [Import] Contribution import - clarify return codes