Merge pull request #20410 from eileenmcnaughton/ppp

[REF] Remove now-unused params

[REF] Remove now-unused params

Merge pull request #20407 from eileenmcnaughton/ppp

[REF] Paypal ipn - cleanup references to completion

[REF] Paypal ipn - cleanup references to completion

This extracts a function to check if the contribution is completed.

I also rationalised the validation - it was using a combo of recur and first to
validate but on thinking it through I realised all it was saying was
'if we are finalising a pending contribution the amount must match'

I think that's fine even for recur with a change in amount - that seems
to me to be something that happens down the track but we still expect
the very first one to come in with the value it originally
had - if that is NOT true then we probably should just remove the check

Merge pull request #20402 from eileenmcnaughton/pledge_tax

[NFC] Cleanup in test class

Merge pull request #20409 from eileenmcnaughton/dep

[REF] Replace deprecated code call

Merge pull request #20406 from mattwire/authnetmd5

Remove remains of md5 check from authorizenet

Merge pull request #20405 from mattwire/eventcarttest

Update eventcart test to only test payment form

[REF] Replace deprecated code call

[NFC] Cleanup in test class

Merge pull request #20404 from eileenmcnaughton/pledge_tax2

Fix contribution page pledge tests to create valid contributions

Remove remains of md5 check from authorizenet

Merge pull request #20360 from eileenmcnaughton/ppp

[REF] Cleanup validation in PaypalProIPN single function

Update eventcart test to only test payment form

Fix contribution page pledge tests to create valid contributions

This addresses an issue where they were using a contribution page with a price
set but te submit params were not valid for that price set

Merge pull request #20393 from eileenmcnaughton/ref_line

[REF] Minor simplification

Merge pull request #20390 from eileenmcnaughton/tax2

Tax fixes in unit test

Tax fixes in unit test

When this->isValidateFinancialsOnPostAssert is true the
test class checks that line items and payments are valid.

I'm trying to enable this for this class. However, there are some issues
that I have found fixes for (and at least 1 I'm still working on)
- some tests try to set tax_amount when it is not enabled
which is invalid - removed
- one test tries to use chaining in a way that
we know is not going to do a job of creating the entities
as it adds the payment before the line items. I switched
this to create a pending payment which doesn't alter the
thing under test & brings it closer to the
recommended flow
- one test is deliberately invalid - I marked it as
not eligible for the validation
- the price set id was not being passed to the Confirm->submit
function (accessed by tests, mostly via the ContributionPage.submit
api) - I added functionality to retrieve it

Merge pull request #20400 from MegaphoneJon/check-signature

fix arguments sent to hook_civicrm_check

fix arguments sent to hook_civicrm_check

Merge pull request #20383 from colemanw/afformUsageFixes

Afform - Improve Gui, prefill & submit APIs

Afform - Improve Gui, prefill & submit APIs

Standardizes prefill & submit APIs to use the same logic for validating contacts.
Displays correct form URL in GUI depending on is_frontend setting.
Shows warning about url arguments to prevent unintentional permissions escalation.

[REF] Minor simplification

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

[REF] Cleanup validation in PaypalProIPN single function

This fixes interaction with the contribution object to instead use the function
and switches the standardised error handling (throw an exception & let
it be caught).

The contribution object used to be passed to completeOrder but no longer
so there is no reason to set total_amount & invoice_id on it
(they will be used from input)

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