Merge pull request #23400 from eileenmcnaughton/import_no_match

[Import] Reduce unused copy & paste

Merge pull request #23399 from civicrm/5.50

5.50

[Import] Reduce unused copy & paste

Unlike the CONFLICT return code the NO_MATCH code IS returned
- but ONLY in the contact import - in all the other imports
it is there by the virtue of copy & paste

Merge pull request #23397 from totten/5.50-e2e-error

(E2E) `testErrorChrome` - Update assertions for page-chrome

Merge pull request #23398 from totten/5.50-e2e-error-bd

(E2E) testErrorChrome - Skip Backdrop and D8+ (much like D7)

Merge pull request #23393 from eileenmcnaughton/import_reg

dev/core#3447 Fix for master-only-regression

Merge pull request #23394 from eileenmcnaughton/import_550

Import 550

Merge pull request #23396 from totten/5.50-upgrade-esc

5.50.alpha1.mysql.tpl - Escape `grouping` (reserved word in MySQL 8.0)

(E2E) testErrorChrome - Revise page-chrome check for all Drupal-style UFs

(E2E) testErrorChrome - Skip Backdrop and D8+ (much like D7)

This test was added during 5.50.alpha (#23257, #22805).  It's purpose is to
assert that the page-chrome/site-wide-template is present on certain
error-messages; however, the underlying behavior was only implemented on
WordPress. It has not yet been implemented on other CMS.

Before
------

* __WordPress__: Runs testErrorChrome. Expected to pass.
* __Drupal 7__: Skip testErrorChrome (mostly) because it is expected to fail.
* __Drupal 8+, Backdrop__: Runs testErrorChrome. Fails.

After
-----

* __WordPress__: Runs testErrorChrome. Expected to pass.
* ___Drupal 7, Drupal 8+, Backdrop__: Skip testErrorChrome (mostly) because it is expected to fail.

Comment
-------

It still runs `testErrorChrome` for `permission` errors; per MM discussion, it appears that this does actually work, but we need
to tune assertion for BD+D8.

(E2E) `testErrorChrome` - Update to work with current wp-demo build

This test was added during 5.50.alpha (#23257, #22805).  It's purpose is to
assert that the page-chrome/site-wide-template is present.  It appears that
the markup from WordPress has changed slightly.  (It may be that I
originally tested against an older version of WP.) In any event, this
relaxes the test so that it also passes on a newer WP build.

5.50.alpha1.mysql.tpl - Escape `grouping` (reserved word in MySQL 8.0.1+)

Import - also export invalid as export error

I found that rows that fail validation are marked 'invalid'
rather than error & won't show on the preview screen.

I guess I could check for invalid vs error separately
- gets kinda weird though - if you leave them
in the file you are importing surely they are still errors
to see on the summary screen even though
you knew they were invalid on the Preview?

dev/core#3447 Fix for master-only-regression

Merge pull request #23386 from eileenmcnaughton/import_no_var

[REF] [Import] Stop passing unused parameters

Merge pull request #23391 from civicrm/5.50

5.50

Merge pull request #23389 from colemanw/getFieldsContactTypeFix

APIv4 - Fix fatal error on getFields when passing in contact id

APIv4 - Fix fatal error on getFields when passing in contact id

Merge pull request #23384 from eileenmcnaughton/import_active

[Import] [Ref] Iterate through the mapping of fields

Merge pull request #23388 from civicrm/5.50

5.50

Merge pull request #23387 from totten/5.50-cacert

Update expired root certs (5.50)

update expired root certs

Set version to 5.51.alpha1

Set version to 5.50.beta1

[REF] [Import] Stop passing unused param

[Import] [Ref] Iterate through the mapping of fields rather than a 'count'

The mapper holds an array of mappings for each of the active fields.
The active field count holds a count of this array - switching to
a foreach loop means we can stop calculating extra stuff.

This PR doesn't start ripping that out - for legibility reasons - but it
will take out a whole lotta code....

Merge pull request #23343 from eileenmcnaughton/import_parser_job

[REF] [Import] Stop passing submittedValues as parameters

Merge pull request #23380 from eileenmcnaughton/import_conflict

[REF] [Import] Remove another good intention - import 'conflicts'

Merge pull request #23381 from colemanw/searchKitDateFields

SearchKit - Add date pseudo-fields

Merge pull request #23371 from eileenmcnaughton/import_valid

[REF] [Import] extract validation functionality

[REF] [Import] Remove another good intention - conflict

There is all sorts of handling for 'conflict' - but a conflict return code is never
generated

[REF] [Import] Stop passing submittedValues as parameters

These parameters are all availabe via

Extract validateValues

Merge pull request #23378 from colemanw/searchKitNow

APIv4 - Add NOW() date function

Merge pull request #23379 from samuelsov/tsOnOff

Translation - distinguish between 'On' for date and 'On'/'Off'

SearchKit - Add date pseudo-fields

Merge pull request #21993 from mattwire/aclcachequery2

ACLCache: Efficiency. Fix duplicate inserts

Translation - distinguish between 'On' for date and 'On'/'Off'

Merge pull request #23369 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variables

APIv4 - Add NOW() date function

Adds a function for NOW which returns the full date+time, and distinguishes
it from CURDATE which just returns the date part.

Merge pull request #23377 from colemanw/month

APIv4 - Add MONTH sql function

Merge pull request #23375 from eileenmcnaughton/ref

[Ref] Get rid of private deprecated legacy function

Merge pull request #23282 from eileenmcnaughton/import_default_location_type

[REF] Do not treat site having a default location type as optional

Merge pull request #23292 from eileenmcnaughton/import_output_table

[Import] Output errors, duplicates csvs directly from the user job table

APIv4 - Add MONTH sql function

[Ref] Get rid of private deprecated legacy function

It now doesn't have enough code to justify it's own function

Merge pull request #23368 from eileenmcnaughton/ref

[REF] Minor code simplifcation - reduce silly lines

Remove conflict download from preview

I can't see where these would arise - but definitely
not on the preview screen - which only runs basic validation

Add output form for csv-on-the-fly

Make tablename a property of the datasource

The datasource stores the table name on the job - we don't need to pass it around.

Also - stop cleaning up the temp table at the end - we want it to
output results but will add a cleanup routine later

Move adding status fields on import table to datasource

Cleanup cleanup on old tables for form re-submission

[REF] [Import] Remove always NULL parameter from private function

Merge pull request #23374 from eileenmcnaughton/549_up

549 to master

Merge pull request #23356 from eileenmcnaughton/pp

Ensure comma not formatted into amount

Merge pull request #23366 from eileenmcnaughton/import_tpl

[REF] [Import] Remove unused tpl assign showColNames

Ensure comma not formatted into amount

Probably the function doesn't add anything as params['amount']
should be correctly formatted - however,
it shouldn't add a comma - per
https://github.com/eileenmcnaughton/nz.co.fuzion.omnipaymultiprocessor/issues/227

Merge pull request #23370 from eileenmcnaughton/import_catch

[Import] [Ref] [Minor] Simplify error handling with a try-catch block

[REF] [Import] Trivial - simplify variables

Simplify error handling with a try-catch block

Merge pull request #23367 from eileenmcnaughton/import_prop

[Ref][Import] Remove another good intention - random email counting

Merge pull request #23363 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variables

Merge pull request #23336 from colemanw/customGroupExtends

CustomFields - Improve metadata about which custom groups belong to which entities

[REF] Minor code simplifcation - reduce silly lines

Merge pull request #23364 from eileenmcnaughton/rel

Minor function simplification in private function

[Ref][Import] Remove another good intention - random email counting

[REF] [Import] Trivial - simplify variables

[REF] [Import] Remove unused tpl assign showColNames

Merge pull request #22115 from artfulrobot/artfulrobot-api4-count-methods

Implement countFetched() and countMatched() on api4 results.

Merge pull request #23276 from eileenmcnaughton/flush

[Ref] [Import] Cleanup cleanup on old tables for form re-submission

Merge pull request #23351 from lcdservices/dev-core-3439

dev/core#3439 change batch geocode query to include events

Minor function simplification in private function

The function does 2 different things depending on whether if finds a duplicate
& having it in a sub-function doesn't make the parent class clearer....

Merge pull request #23313 from herbdool/issue-3429

Issue 3249: make casetype a managed entity

Merge pull request #23361 from eileenmcnaughton/import_comment

Import - fix comment

APIv3 - Fix testCustomDataGet errors caused by legacy entity naming issues

CustomFields - Improve metadata for custom_group_extends

This allows the API to filter custom groups by entity type and other values.

CustomGroup - Add metadata about how a custom group relates to entity types

Fixes dev/core#2905
Before: Hard coded, plus a very strange use of the 'description' field to store a callback function.

After: The 'grouping' field in the OptionValue for custom extends and custom type is used.
APIv4 getfields can then retrieve the necessary options.

Merge pull request #23362 from demeritcowboy/dao

DAOs out of date

Merge pull request #23360 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variable

Merge pull request #23339 from ginkgomzd/bermuda-parishes

add missing Parishes of Bermuda (civicrm_state_province)

DAOs out of date

Merge pull request #23288 from eileenmcnaughton/import_upgrade

[Import] Upgrade saved import names for contacts

Import - fix comment

Merge pull request #23340 from eileenmcnaughton/import_preview2

[REF][Import] [Contact] Clean up preview screen

[REF] [Import] Trivial - simplify variable

Assign this->activeFields[i]->_relatedContactType to
a variable rather than keep repeating it.

Cleanup cleanup on old tables for form re-submission

Merge pull request #23342 from eileenmcnaughton/import_map

[Import] [Ref] [Contact] Extract row mapping and add test

Merge pull request #23345 from eileenmcnaughton/cypr

 Stop adding obsolete currencies to new installs

Merge pull request #23357 from mattwire/participant

NFC Minor cleanup to Participant confirm

regen;

add missing Parishes of Bermuda (civicrm_state_province)

Merge pull request #23344 from eileenmcnaughton/import_functions

[Ref] [Import] Add extra functions to DataSource

Merge pull request #23359 from colemanw/managedDocs

[NFC] Fix APIv4 docblock annotations for Managed API

NFC - Fix APIv4 docblock annotations for Managed API

Merge pull request #23349 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variable

Merge pull request #23352 from eileenmcnaughton/leg

Remove unused variable

Minor cleanup to Participant confirm

Merge pull request #23355 from colemanw/userJobIfNotExists

Fix 'Table already exists' error when running 5.50 upgrade twice

Merge pull request #23354 from colemanw/searchKitDateField

SearchKit - Consistently check for date field

Fix 'Table already exists' error when running 5.50 upgrade twice