[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

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

SearchKit - Consistently check for date field

Add extra functions to DataSource

This adds functions from https://github.com/civicrm/civicrm-core/pull/23292 to
the DataSource class (without them being called as yet)

- part of trying to get to the point where fixes are not dependent on each other

Merge pull request #23350 from eileenmcnaughton/rel_cast

IDE cheer - cast var explicitly to an int

Merge pull request #23310 from colemanw/api4Update

Api4 - minor fixes and updates

Merge pull request #23321 from davejenx/report-column-nodisplay

PR 22070 follow-up: reinstate respecting no_display on report column headers

Merge pull request #23348 from colemanw/getFieldsSuffixes

APIv4 - Support pseudoconstant suffixes in getFields

Remove unused variable

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

IDE cheer - cast var explicitly to an int

[REF] [Import] Trivial - simplify variable

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

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

Merge pull request #23347 from eileenmcnaughton/import_var

[REF] [Import] Trivial - simplify variable

APIv4 - Support pseudoconstant suffixes in getFields