[REF] Slightly more logical order of code

Just moves the lock check to the same area

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

Merge pull request #20318 from eileenmcnaughton/ppp

[REF] Add getter for contributionID

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…

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

[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

Merge pull request #20336 from colemanw/searchKitUseDefaultPagerSize

SearchKit - Use default_pager_size setting

Merge pull request #20337 from totten/master-field-dao

(REF) Regen CRM/Core/DAO/CustomField.php

(REF) Regen CRM/Core/DAO/CustomField.php

Before: The committed DAO doesn't match the generated DAO.

After: The commited DAO matches the generated DAO.

Technical Details: This DAO file grew slightly stale following 0388de0d6435161344c9483066765e7a9ca65dfe.

Merge pull request #20333 from agileware/CIVICRM-1732

Fix case sensitive parameter for call to getCustomFieldTokens

SearchKit - Use default_pager_size setting

Merge pull request #20332 from colemanw/links_callback

Core_DAO - fire links_callback for all entities

Merge pull request #20285 from colemanw/cke5Fix

Fix CKEditor5 link editing in modal dialog

Merge pull request #20210 from eileenmcnaughton/cancel

Fix old-ish regression on paypal standard self-service cancel links

Merge pull request #20243 from eileenmcnaughton/group2

[Ref] remove redundant call to clearGroupContactCache

Merge pull request #20326 from mattwire/payjunctiondopayment

Convert payJunction to doPayment

Merge pull request #20325 from mattwire/firstdatadopayment

Convert FirstData to doPayment

Merge pull request #20324 from mattwire/elavondopayment

Convert elavon to doPayment

Merge pull request #20327 from mattwire/realexdopayment

Convert Realex to doPayment

Merge pull request #20328 from mattwire/payflowprodopayment

Convert payflowpro to doPayment

Core_DAO - fire links_callback for all entities even if they don't explicitly declare links

CRM_Core_DAO has a fallback function for entities with no foreign key fields.
Previously, that function did not call createReferenceColumns or fire links_callback

Merge pull request #20323 from mattwire/ewaydopayment

Convert eway to doPayment

Merge pull request #20329 from highfalutin/patch-6

Correct copy-paste error in CustomField schema

Fix form params for testEventPaymentForms

Merge pull request #20322 from mattwire/authnetdopayment

Convert AuthorizeNet to doPayment()

Fix test to support invalidate change

[Ref] remove redundant call to clearGroupContactCache

This code does a clear on the cache and then calls the api which will eventually
call load(). Load ALSO clears the cache so this seems redundant.

However, there are a couple of places where add() is called from where it might
serve to force the rebuild due to changes to group make up. In general we have switched
to invalidating and letting the rebuild-on-demand do it's thing to allow
things like crons or lack of need for the group to reduce to browser demand.
So, I think that by switching to invalidating rather than rebuilding
we can remove this potentially locky line from GroupContactCache::add()

Merge pull request #20309 from jmcclelland/profile-county

get county fields populated via api call

Merge pull request #20303 from totten/master-extver-xml

dev/release#17 - Synchronize version#s for core and core-extensions

Merge pull request #20269 from colemanw/bulkSave

APIv4 - Save custom fields in bulk

Correct copy-paste error in CustomField schema

Convert payflowpro to doPayment

Convert Realex to doPayment

Convert payJunction to doPayment

Convert FirstData to doPayment

Convert elavon to doPayment

Convert eway to doPayment

Convert authorizenet to doPayment

Merge pull request #20273 from seamuslee001/report_default_pager

[REF] Make use of recently added default pager size setting in Reports

Merge pull request #20313 from eileenmcnaughton/mem_move2

[REF] Duplicate & deprecate processMembership

Merge pull request #20320 from eileenmcnaughton/managed

Add unit test for #20144

Merge pull request #19840 from MikeyMJCO/patch-7

Explicitly throw an error/status bounce on Edit/New if no types.