Merge pull request #20442 from seamuslee001/fix_required_param_after_optional

[php8-compat] [REF] Fix a couple of functions triggering deprecation notices in php…

Merge pull request #20445 from seamuslee001/fix_warning_notice_code_gen

[php8-compat] Fix Warning Notices when running schema generation

Merge pull request #20444 from seamuslee001/upgrade_symfony

[php8-compat] Update Various Symfony components to latest version in …

Merge pull request #20189 from lemniscus/oauth-client

OAuth: initial support for per-contact OAuth tokens

Merge pull request #20439 from eileenmcnaughton/proftest

Fix profile test teardown

Merge pull request #20443 from seamuslee001/fix_error_php8

[php8-compat] Fix calling method_exist with paremeter that is bool not an object in…

[php8-compat] Fix Warning Notices when running schema generation

Merge pull request #20441 from seamuslee001/fix_createrandom

[php8-compat] [REF] Fix call to function CRM_Utils_String::createRandom by ensuring that …

[REF] Fix a couple of functions triggering deprecation notices in php8.0 where by an optional parameter is before required parameters in the function signature

[php8-compat] Update Various Symfony components to latest version in the 3.4.x series to fix deprecation notice around libxml entity disable

Fix calling method_exist with paremeter that is bool not an object in php8

Fix call to function CRM_Utils_String::createRandom by ensuring that the length is the first parameter not 2nd

Fix profile test teardown

This should make https://github.com/civicrm/civicrm-core/pull/20420 pass
for this class

add support for OAuthContactTokens, with tests, upgrade step, new permissions

Per discussion with @totten, re the logic that interprets "tag" and links/creates contacts accordingly: in earlier versions of this code, I had put that logic in OAuthTokenFacade. Moving it to the api4 "create" action means it will happen even when a ContactToken is created outside the OAuth flow. I decided not to put it in BAO writeRecords() because then it would run too late for the permissions checking we do in the api4 Create action.

Note: the permissions change here means that some current users of the OAuth Extension will need to enable an additional permission to allow for the acquisition of new tokens through the auth-code flow.

Merge pull request #20435 from eileenmcnaughton/gg

[REF] Extract code to transfer groups from temp table to cache

Merge pull request #20436 from eileenmcnaughton/mem_test_fix

Fix separate payment membership test to create valid financial transa…

Fix separate payment membership test to create valid financial transations

The issue is inconsistency on the test side, not the core code

[REF] Extract code to transfer groups from temp table to cache

This continues efforts to have the 'worker functions' accept an
array of groups, not just a single

Merge pull request #20403 from colemanw/contactEmailUpdate

Ensure contacts without a name are updated when primary email changes

Merge pull request #20416 from colemanw/afformTabFix

Afform GUI - Fix tab breakage in Shoreditch theme

Merge pull request #20385 from eileenmcnaughton/gc3

Fix internal group contact cache functions to be protected

Merge pull request #20421 from eileenmcnaughton/ev_confirm

[REF] minor extraction - getInfoUrl

Merge pull request #20419 from colemanw/angularMain

Use new `crm-angular-js` instead of `ng-app` to bootstrap AngularJS

Merge pull request #20430 from seamuslee001/master

5.38

Merge in 5.38

Merge pull request #20428 from seamuslee001/ckeditor_upgrade

[REF] Upgrade CKEditor to be 4.16.1

AngularLoader - cleanup resource loading to use new service

Use new `crm-angular-js` instead of `ng-app` to bootstrap AngularJS
Emit a deprecated warning from the old load() function to alert extension maintainers of the change.

Merge pull request #20102 from seamuslee001/d8_mailing_urls

[REF] Fix issue where by mailing urls were always stuck in the origin…

[REF] Upgrade CKEditor to be 4.16.1

Merge pull request #20129 from larssandergreen/apply-mailing-tracking-only-to-a-urls

dev/mailing#95 Only track <a> urls in Flexmailer for HTML emails

Merge pull request #20415 from eileenmcnaughton/tax_test

Test - fix contributionTest to validate contributions

Merge pull request #20422 from eileenmcnaughton/ppp

dev/core#2593 Fully remove loadRelatedObjects from PaypalExpress

Merge pull request #20426 from eileenmcnaughton/tolerate_tax

Test fixes

Test fixes

This improves cleanup in one class & in another makes it accept tax_amount = 0
rather than requiring it to be NULL. In general it's better for us to return
and save 0 so this makes the test tolerate those changes

Merge pull request #20423 from eileenmcnaughton/ppp1

[REF] Paypal pro - remove call to load contact record

[REF] Paypal pro - remove call to load contact record

This removes a look up since we are going to use the contribution contact id & all we are doing
is logging warning if it has changed

dev/core#2593 Fully remove loadRelatedObjects from PaypalExpress

Merge pull request #20412 from eileenmcnaughton/ppp

[REF] Stop passing unneed params to the recur function

[REF] minor extraction - getInfoUrl

Merge pull request #20417 from mattwire/payjunctionsetparam

Remove unused functions from payJunction

Merge pull request #20418 from mattwire/dummynoparams

Remove unused variable on dummy processor

Merge pull request #20306 from aydun/api_case_defaults

Add defaults for APIv4 Case create

Remove unused variable on dummy processor

Remove unused functions from payJunction

Afform GUI - Fix tab breakage in Shoreditch theme

APIv4 - Fix Case conformance test

Now that creator_id has a default value it can also be marked as required.
Updating TestCreationParameterProvider to supply a FK before the default value
fixes the problem with user_contact_id in a test with no logged-in user.

Ensure contacts without a name are updated when primary email changes

Contacts with no name use their primary email as display_name and sort_name.
This ensures that when their primary email is updated, display_name and sort_name
will be updated as well.
Adds tests for APIv3, APIv4 & Afform.
Fixes dev/core#2622

Merge pull request #20401 from vingle/master

Revised fix for menu/UI issues for Joomla 4.0

Test - fix contributionTest to validate contributions

Still skipping tests that need more work to avoid malformed
financial entities

Merge pull request #20414 from MikeyMJCO/patch-7

Describe logic behind placement of `invalidateRouteCache`

Describe logic behind placement of `invalidateRouteCache`

[REF] Stop passing unneed params to the recur function

Merge pull request #20411 from eileenmcnaughton/ppp2

[REF] replace isFirst parameter

Merge pull request #20410 from eileenmcnaughton/ppp

[REF] Remove now-unused params

[REF] replace isFirst parameter

Here we see that the 2 paths to 'recur' were calculating isFirst slightly differently.

In one case (the dominant one) we were looking to see if it was completed whereas
in the lesser path (paypal express) we were looking to see if it was not pending.

That leaves
'cancelled' & 'failed' in a limbo. I think logically we can't complete either of those
so the dominant definition is better

[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

spacing tidy

Fixes expanding News dashlet css bug in Joomla 4

removes subhead and extra padding in Joomla 4

tidying indents

Joomla 4 fixes for menubar below menu & < $breakMin

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

Fix internal group contact cache functions to be static

These are not in git universe outside of core. In theory the
tests should still work....

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