Remove unused functions from payJunction

Merge pull request #20380 from seamuslee001/api_v3_civi_phpunit8_warnings

[NFC] Fix PHPUnit8 Deprecation Warnings in api_v3 and Civi Test Suites

[NFC] Fix PHPUnit8 Deprecation Warnings in api_v3 and Civi Test Suites

Merge pull request #20373 from eileenmcnaughton/gc

[REF] extract function to get locks

Merge pull request #20374 from eileenmcnaughton/gc4

[NFC] [Test] Big cleanup in test class

Merge pull request #20381 from seamuslee001/api_v4_phpunit8_warnings

[NFC] Fix PHPUnit8 Deprecation warnings in the api_v4 Test Suite

Merge pull request #20379 from seamuslee001/e2e_phpunit8_warnings

[NFC] Fix PHPUnit8 Warning notices in E2E And in some API unit tests

[NFC] [Test] Big cleanup in test class

Merge pull request #20378 from eileenmcnaughton/entry

[REF] Remove unused variable

[NFC] Fix PHPUnit8 Deprecation warnings in the api_v4 Test Suite

Further fixes

Fix PHPUnit8 Warning notices in E2E And in some API unit tests

Merge pull request #20377 from seamuslee001/crm_phpunit8_warnings

[NFC] Fix PHPUnit8 deprecation warnings in the CRM_AllTests suite

[NFC] Fix PHPUnit8 deprecation warnings in the CRM_AllTests suite

[REF] Remove unused field parameters

[REF] extract function to get locks

This is part of trying to make the logic more re-usable and make the
way of dealing with a single group more aligned with dealing with
multiple groups in this class

Merge pull request #20305 from seamuslee001/afformEntityRefSubmit

Afform Entity Reference Submit

Afform - Simplify postprocess event, cleanup api code & improve test coverage

- Simplifies getEntityWeights function using topological sorting library
- Consolodates postprocess event listners to just processGenericEntity
- Adds missing getters/setters to api classes
- Improves the AfformSubmitEvent interface to make it easeier to use
- Adds to tests

Merge pull request #20372 from eileenmcnaughton/tax

[NFC] Further cleanup in test class

[NFC] Further cleanup in test class

Merge pull request #20369 from seamuslee001/more_voids

[NFC] Add in more voids to unit tests to support PHPUnit8

Merge pull request #20371 from eileenmcnaughton/tax

[NFC] [Test] Preliminary cleanup in test class

Merge pull request #20353 from seamuslee001/fix_distmaker_sync_core_ext

Fix Distmaker building by ensuring we are rsyncing from the right folder

Merge pull request #20217 from eileenmcnaughton/contop

[Test] Add test cover for contribution tokens in pdf letter

Merge pull request #20368 from eileenmcnaughton/cc

[REF] Slightly more logical order of code

[NFC] [Test] Preliminary cleanup in test class

[NFC] Add in more voids to unit tests to support PHPUnit8

Merge pull request #20367 from colemanw/deprecateBulkSave

Deprecate function CRM_Core_BAO_CustomField::bulkSave

[REF] Slightly more logical order of code

Just moves the lock check to the same area

Deprecate function CRM_Core_BAO_CustomField::bulkSave

Merge pull request #20366 from demeritcowboy/more-oo

[REF] Useradd contact task - Use OO instead of hardcoding "if Wordpress"

Merge pull request #20359 from eileenmcnaughton/in

Superficial tidy up in test class

Merge pull request #20363 from eileenmcnaughton/group

Cleanup - declare return types, remove unused var

Merge pull request #20196 from eileenmcnaughton/import

[REF] Cleanup on import rows error

Merge pull request #20362 from eileenmcnaughton/save

Update bulkSave test to use the api

Merge pull request #20349 from colemanw/searchKitFixMenuPerm

SearchKit - Fix menu permission

Merge pull request #20365 from seamuslee001/voidtearDownAfterClass

[NFC] Add in voids to various setUpBeforeClass and tearDownAfterClass…

Merge pull request #20348 from colemanw/searchKitTweaks

SearchKit and Afform UI improvements

Afform Gui - Set height on dropdowns so CMS theme doesn't interfere

make more oo-ey

SearchKit & Afform - Style click-sort column headers

SearchKit - Prevent overwriting SavedSearch.modified_date

When saving a search, the modified_date was being overwritten with the old value.

SearchKit - Click-sort column headers on list page

Afform - Standardize labels with Title Case and no colon

Afform - Click-sort column headers on list page

Afform - Add 'Placement' colum to admin screen

SearchKit - Standardize labels with Title Case and no colon

The trailing colon is a pain for translators.

SearchKit - When editing a display, show a checkbox to toggle limit

Previously, limit was a number input which could be disabled by entering the number '0'.
That's less intuitive than a checkbox.

SearchKit - Sync menu & navigation permissions to use 'administer CiviCRM data'

Fixes dev/core#2619
https://lab.civicrm.org/dev/core/-/issues/2619

[NFC] Add in voids to various setUpBeforeClass and tearDownAfterClass functions

Cleanup - declare return types, remove unused var

Merge pull request #20361 from eileenmcnaughton/zero

Unit test for #20342

Merge pull request #20297 from eileenmcnaughton/roup

[REF] Extract function to build temporary table of members of group from Report_Form

Save test

Unit test for #20342

Merge pull request #20342 from bugfolder/mandatory_keys_check

Fix mandatory keys check fail for value of float 0.0 in a required key in an entity

Superficial tidy up in test class

Merge pull request #20355 from eileenmcnaughton/ppp2

[REF] Extract getContributionObject

Merge pull request #20356 from seamuslee001/phpunit8_config

[NFC] Update PHPUnit config to not cache results in phpunit8 and add …

Merge pull request #20354 from eileenmcnaughton/ppp

[REF] Remove now-unused parameter

Merge pull request #20241 from eileenmcnaughton/haystack

Add test for mixed Order data

[REF] Extract function to build temporary table of members of group from Report_Form

[NFC] Update PHPUnit config to not cache results in phpunit8 and add more voids into PayflowProTest

Extract getContributionObject

[REF] Remove now-unused parameter

Fix tearDown & setup to work under default test db state & leave it closer to default test db state

Merge pull request #20318 from eileenmcnaughton/ppp

[REF] Add getter for contributionID

[Test] Add test cover for contribution tokens in pdf letter

Merge pull request #20345 from eileenmcnaughton/mem_move

[REF] Make previous unshared function protected & Remove hard-coded params

Merge pull request #20330 from mattwire/fixpaymentformstest

Fix form params for testEventPaymentForms

Merge pull request #20352 from seamuslee001/test_credit_card_validation

[NFC] Add in unit tests of credit card validation rules

Merge pull request #20351 from seamuslee001/update_pear_finance_creditcard

[REF] Update Pear Validate Finance CreditCard to use latest tagged re…

Fix Distmaker building by ensuring we are rsyncing from the right folder

Merge pull request #20340 from eileenmcnaughton/schema2

Fix extraneous white space in generated sql

Merge pull request #20278 from aydun/name_custom_css

Give the custom css file a 'name' to make it easier to manipulate in hook_civicrm_alterBundle()

Merge pull request #20346 from eileenmcnaughton/confirm

[REF] Make previously unshared function protected

[NFC] Add in unit tests of credit card validation rules

[REF] Update Pear Validate Finance CreditCard to use latest tagged release that now includes the improved mastercard 2bin regex

Merge pull request #20319 from eileenmcnaughton/pp

Remove extraneous debug

Merge pull request #20347 from demeritcowboy/pass-compare

dev/core#2613 - Replace deprecated function in useradd task

[REF] Make previously unshared function protected

This clarifies that it is not used outside core - which we
know as it was only just moved here 5187a13c4d0e4278bb9b18f9fb327b2407766dff

switch from deprecated function

Ensure that we only replace entity reference fields and add unit test

Add in unit test of the submit action and re-work as per feedback from coleman

Rework the listeners so that they are called on the basis of each entitiy rather than just once. Also extract weight function out and make improvements following Coleman's review and add tests

WIP Entity Reference Submission fields

[REF] Remove hard-coded params

Merge pull request #20301 from eileenmcnaughton/mem_move

[REF] Remove unused parameters

Merge pull request #20344 from civicrm/5.38

5.38

Merge pull request #20274 from agileware/CIVICRM-1737

Create User Record now hides the Password field and uses the check isUserRegistrationPermitted - which prevents Administrators from setting passwords for new Users

Merge pull request #20307 from colemanw/searchTaskHook

SearchKit - Rename crmSearchActions module to crmSearchTasks & add hook

Separate checks for 0.0 and 0

Merge pull request #20343 from seamuslee001/nfc_tiered_test_timecop

[NFC] Remove timecop function references from Cache Tiered Test

CIVICRM-1737 Create User Record now hides the Password field and uses the check isUserRegistrationPermitted - which prevents Administrators from setting passwords for new Users

[NFC] Remove timecop function references from Cache Tiered Test

Fix mandatory keys check fail for value of float 0.0

Merge pull request #20341 from colemanw/greenwichLinkColor

dev/core#2403 - Change Greenwich link color from gray to blue

Merge pull request #20239 from eileenmcnaughton/act

[REF] Cleanup BAO_ActionSchedule::getlist() signature

dev/core#2403 - Change Greenwich link color from gray to blue

Fixes dev/core#2403

Fix extraneous white space in generated sql

- trailing space when comment is not declared for a db table
- spaces before arguments that don't exist
- extra lines

Schema.tpl whitespace overhaul

Note I tried regenerating civicrm.mysql with this & it looked good - jenkins can
pass judgement too...

I also tried regenerating the sql in search_kit but I hit what seems to me to be
an unreleated regression - ie search_kit generates
 ROW_FORMAT=DYNAMIC

With my version (lastest from master) of civix I get ENGINE = INNODB but
not the ROW_FORMAT