Merge pull request #20391 from eileenmcnaughton/tax_valid

Add call to validateAllContributions & fix getTotalAmount

Add call to validateAllContributions & fix getTotalAmount

I've updated 2 tests for readability and extended one to call
validateAllContributions. The call failed and it turned out to
be the check was wrong. Fixing that caused a test to fail.
I have fixed the test by fixing getTotalAmount on the internal
helper BAO_Order class to include the tax_amount for each line
in the total.

I think this would most affect tests - but it might affect the
Membership_Form due to changes made recently to that
form so I'm on the fence about going for the rc with this
one

Merge pull request #20358 from eileenmcnaughton/partial

Remove legacy references to partial_amount_to_pay

Merge pull request #20389 from eileenmcnaughton/tax

[NFC] {test} code cleanup in test class

[NFC] {test} minor cleanup

Merge pull request #20387 from eileenmcnaughton/tax

[NFC] Minor cleanup

Merge pull request #20386 from eileenmcnaughton/cont_f

[NFC] Minor code cleanup

Merge pull request #20370 from seamuslee001/phpunit8

Use PHPUnit8 in main test suites

[NFC] Minor cleanup

[NFC] Minor code cleanup

Merge pull request #20384 from mattwire/deprecateddopayment

If paymentprocessor still uses doTransferCheckout/doDirectPayment trigger deprecated function warning

If paymentprocessor still uses doTransferCheckout/doDirectPayment trigger deprecated function warning

Use PHPUnit8 for main runs

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

Remove legacy references to partial_amount_to_pay

This was an early concept of how to do partial payments that we eliminated a while back
in favour of Payment.create

It is not set anywhere in core - only checked so time to remove the checks

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