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

Merge pull request #23406 from eileenmcnaughton/import_no_match

[REF] [Import] Summary tpl - remove always-true ifs

Merge pull request #23405 from eileenmcnaughton/import_in

[REF][Import] Remove some, now unused, parameters

[REF] [Import] Summary tpl - remove always-true ifs

[REF][Import] Remove some, now unused, parameters

[REF] Import - make condition clearer

The mapping of fields basically breaks down into 'has a relationship' - in which
case fieldName is empy and does not have a relationship - in which
case field name is not empy. This makes that explicit -
specifically referring to relatedContactKey rather than
relying on the person reading the code
understanding that the presence of fieldName implies
whether it is a relationship

[REF] [Import] Contribution import - clarify return codes, remove handling for unused CONFLICT

Merge pull request #23296 from colemanw/afformRelationships

Afform - support relationships

Merge pull request #23395 from eileenmcnaughton/import_cleanup

[Ref][Import] Remove now-unused code

Merge pull request #23402 from colemanw/customTestBase

[REF] [TEST ONLY] APIv4 - Reorganize test classes, don't use transactions for custom field tests

Merge pull request #23400 from eileenmcnaughton/import_no_match

[Import] Reduce unused copy & paste

APIv4 - Reorganize test classes, don't use transactions for custom value tests

Transactions don't work for custom field tests because schema alterations can't be inside transactions.
This moves the "implements TransactionalInterface" declaration from the base class to the test classes
so each one can opt-in individually.

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?

[Ref][Import] Remove now-unused code

This is the first round of follow up cleanup from #23384

These properties are no longer being used
so don't need to be passed in or wrangled

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

Afform - Support creating & updating relationships

Fixes dev/core#3117

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