Merge pull request #23416 from eileenmcnaughton/import_create_job

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

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

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

Afform - Fix RC regression of missing input types

This regressed because the client-side variable was renamed as
part of 47332dba8ff22aa6f08d473a0437dc02ca53458d

Merge pull request #23407 from eileenmcnaughton/import_550

[Import] Fix 5.50 error url

APIv4 Tests - Remove unused extras from ConformanceTest

[Import] Fix 5.50 error url

[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.

[Import] Fix database initialization to remove unused parameters

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....

SearchKit - Default to search for individuals

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