Merge pull request #20530 from eileenmcnaughton/val

Fix v3 api payment test to validate on postAssert

Merge pull request #20541 from eileenmcnaughton/alpha_trigg

Sort trigger data before processing

Sort trigger data before processing

At some point these stopped being consistently alpha sorted - which doesn't matter
if you are just letting Civi run the trigger updates but if you output it
and diff it this inconsistency is a problem

Subset of https://github.com/civicrm/civicrm-core/pull/20472
in the hope of getting this merged

https://github.com/civicrm/civicrm-core/pull/20471 also grooms this output for diffing
albeit only in an edge case

Merge pull request #20539 from seamuslee001/group_custom_search_fix

[php8-compat] Fix Include/Exclude Group Custom Search Tests by better…

Merge pull request #20537 from colemanw/searchKitLinkFix

SearchKit - Fix link target selection

Merge pull request #20538 from seamuslee001/fix_resource_test

[php8-compat] Fix Resources Test issue due to undefined array key add…

[php8-compat] Fix Include/Exclude Group Custom Search Tests by better handling empty arrays on exclusions in the custom search

[php8-compat] Fix Resources Test issue due to undefined array key addCacheCode

Merge pull request #20535 from MikeyMJCO/patch-12

Switch table mapping to square bracket array syntax.

SearchKit - Fix link target selection and move link templating out of controller

Merge pull request #20534 from demeritcowboy/dupedao

Dedupe DAO is out of date

Switch table mapping to square bracket array syntax.

Merge pull request #20525 from seamuslee001/fix_date_tests

[php8-compat][REF] Fix Date unit tests in php8 by passing in 00 inste…

fix hash

Merge pull request #20528 from colemanw/exportFix2

dev/report#67 Fix standalone export classes

Merge pull request #20526 from seamuslee001/required_after_optional_more2

[php8-compact][REF] Fix another couple of places where by there are r…

Merge pull request #20527 from seamuslee001/payment_token_fix

[php8-compact][REF] Fix api_v3_PaymentTokenTest to work on php8

Merge pull request #20524 from seamuslee001/fix_case_custom_tests

[php8-compat][REF] Fix Case and Custom data tests by adding guards in…

Fix v3 api payment test to validate on postAssert

This fixes the test that was failing validation & moves the validation to class level

Fix standalone export classes

This fixes an 'access denied' error on standalone export forms,
caused by renaming the php classes.

Fixes dev/report#67

[php8-compact][REF] Fix api_v3_PaymentTokenTest to work on php8

[php8-compact][REF] Fix another couple of places where by there are required variables in php function declaration after optional ones

[php8-compat][REF] Fix Date unit tests in php8 by passing in 00 instead of null for hours,minutes and seconds

[php8-compat][REF] Fix Case and Custom data tests by adding guards into templates

Merge pull request #20357 from eileenmcnaughton/tax_add

Fix for tax rates being mangled on contribution update

Merge pull request #20495 from eileenmcnaughton/lines

Use line items to look up memberships

Merge pull request #20521 from eileenmcnaughton/validate

Test fix up for AdditionalPaymentTest

Fix the ContributionPageTest to set up valid memberships

Fix for https://github.com/civicrm/civicrm-core/pull/20495
- resolve invalid test config

Use line items to look up memberships

This still checks the membership payment but creates some noise if there are validity issues

Test fix up for AdditionalPaymentTest

This addresses a poor set up issue where the membership + contribution was being
set up incorrectly &  hence the line items were wrong, along with the ability
to validate the financials. It was blocking https://github.com/civicrm/civicrm-core/pull/20495
along with the efforts to get financial validation on all tests

Merge pull request #20507 from colemanw/apiSmartGroups

SearchKit - Add API filter for contacts in groups and smart groups

Merge pull request #20520 from seamuslee001/php8_more_template_guard

[php8-compat] Fix api_v3_PaymentTest failures by putting in more guar…

Merge pull request #20499 from JMAConsulting/add_permission_fi_api4

dev/core#2486 - Use read-only permissions for FinancialItem API

Merge pull request #20513 from demeritcowboy/regen-better

dev/core#1549 - Malleate civicrm_generated so that long lines are split

[php8-compat] Fix api_v3_PaymentTest failures by putting in more guards into message templates

Merge pull request #20516 from seamuslee001/php8_more_test_fixes

[php8-compat][REF] Fix some more test failures in php8

Merge pull request #20518 from seamuslee001/php8_contribution_page

[php8-compat][REF] Fix api_v3_contributionpagetest on php8

APIv4 - Include child groups in contact group filter

SearchKit - Add API filter for contacts in groups and smart groups

Adds 'type' property to API getFields to distinguish regular fields
from custom fields, extra fields and filters.

Implements `Contact.groups` as a filter, which internally adds a temp-table
and incorporates it into the query.

[php8-compat][REF] Fix api_v3_contributionpagetest on php8

Merge pull request #20517 from seamuslee001/php8_more_template_test_fixes

[php8-compat][REF] Fix more php8 test failures caused by template issues

[php8-compat][REF] Fix more php8 test failures caused by template issues

[php8-compat][REF] Fix some more test failures in php8

Merge pull request #20512 from seamuslee001/php8_contribution_api_tests

[php8-compat] Update smarty templates and some php files to get the a…

Merge pull request #20515 from seamuslee001/php8_required_optional

[php8-compat] Fix some more examples of where required parameters are…

[php8-compat] Update smarty templates and some php files to get the api_v3_contribution testclass to pass on php8

[php8-compat] Fix some more examples of where required parameters are after optional parameters in fucntion declaration

split long lines in civicrm_generated

Merge pull request #20331 from mattwire/suppresslegacywarnings

Allow legacy warnings to be fully suppressed in PropertyBag

Merge pull request #20509 from seamuslee001/fix_beautifier_notice

[php8-compat] Fix php beautifier notice by conditionally assinging dy…

Merge pull request #20508 from seamuslee001/product_oddness

[php8-compat] Fix issue in APIv3 Where by because product has a colum…

Merge pull request #20502 from seamuslee001/spaceship

[php8-compat] Fix issue with returning bool from uasort by using the …

Merge pull request #20500 from seamuslee001/php8_zip_test

[php8-compat][NFC] Fix using ZipArchive::open on an empty file

Merge pull request #20504 from JMAConsulting/add_aclrole_api4_entity

ACLEntityRole BAO tidy fixes

Merge pull request #20503 from seamuslee001/upgrade_fixes

[php8-compat] fix Upgrade call back issues by making functions static…

[php8-compat] Fix issue with returning bool from uasort by using the spaceship operator

Merge pull request #20498 from seamuslee001/fix_authx_drupal89

[REF] Fix Authx tests on Druapl 8/9 by ensuring that we only return a…

[php8-compat] Fix php beautifier notice by conditionally assinging dynamic foreign key to the template

[php8-compat] Fix issue in APIv3 Where by because product has a column called options the testCreateSingleValueAlter triggers a cannot access offset of type string on string in php8

[php8-compat] fix Upgrade call back issues by making functions static and also fixing an issue with an array key not existing when checking obsolete extensions

Merge pull request #20506 from colemanw/deleteActivityPreCreationSubscriber

APIv4 - Delete undocumented deprecated activityType lookup

Merge pull request #20501 from JMAConsulting/api4_batch_spec

Add APIv4 Batch.create spec

ACLEntityRole tidy fixes

Add APIv4 Batch.create spec

APIv4 - Delete undocumented deprecated pseudoconstant lookup for Activity type

[php8-compat][NFC] Fix using ZipArchive::open on an empty file

Add permissions for financial_item entity

Merge pull request #20496 from seamuslee001/php8_array_key_actionscheduletest

[php8-compat][NFC] Fix issue where by we are tryiing to access array keys …

[php8-compat] Fix issue where by we are tryiing to access array keys that haven't been created yet

Add in code comment

Merge pull request #20497 from seamuslee001/more_required_after_optional

[REF][php8-compat] Fix more instances of where there is a required pa…

Merge pull request #20488 from eileenmcnaughton/cust_strict

Clarify types on `hook_custom` and `hook_customPre`

[REF] Fix Authx tests on Druapl 8/9 by ensuring that we only return an id for the user id if it is greater than 0

[REF][php8-compat] Fix more instances of where there is a required parameter for a function after an optional one and fix an issue where by a NULL function property is treated as not exisiting in php8

Merge pull request #20479 from demeritcowboy/userload

dev/core#2636 - Authx - Undefined function in drupal 9

Merge pull request #20375 from JMAConsulting/core-65

Prevent adding duplicate dashlet if present with same name and label

Merge pull request #20470 from colemanw/entityGet

APIv4 Entity.get refactor to be more efficient

Merge pull request #20492 from colemanw/dedupeRename

Cleanup references to old dedupe class names

Fix for tax rates being mangled on contribution update

Update sample code

Handle tax_amount as empty string

m

Merge pull request #20474 from JMAConsulting/add_aclrole_api4_entity

dev/core#2486 Add ACLEntityRole APIv4 Entity

Add AclRole Api4 Entity

Cleanup references to old dedupe class names

Merge pull request #20466 from JMAConsulting/add_deduperule_api4

Add DedupeRule, DedupeRuleGroup and DedupeException API4 entity

Merge pull request #20491 from seamuslee001/array_cache_undefined

[php8-compat] Fix undefined property on Array Cache class in wordpres…

[php8-compat] Fix undefined property on Array Cache class in wordpress on PHP8

test failure fix and add API4 files

Merge pull request #20490 from seamuslee001/required_after_optional_fix

[REF][php8-compat] Further fixes where there is a required paramater …

Merge pull request #20478 from totten/master-translation-table-only

dev/translation#67 - Define "Translation" table. Add during installation/upgrade.

Merge pull request #20489 from colemanw/getFieldsFix

APIv4 - Cleanup getFields, add @internal flag for non-public field attributes

[REF][php8-compat] Further fixes where there is a required paramater after an optional paramater

Merge pull request #20487 from eileenmcnaughton/cust_value

Remove some unused variables

APIv4 - Cleanup getFields, add @internal flag for non-public field attributes

Merge pull request #20485 from seamuslee001/fix_not_always_defined_constant

[php8-compat][REF] Fix php8 error on undefined constant CIVICRM_DISAB…

Clarify types on custom hooks

The custom & customPre hooks are not called elsewhere in gituniverse
so we can add type hints and fix casting & comments to make it
clearer what the variables are

(NFC) CRM_Utils_Hook::translateFields - Tidy up comments

The previous comments reflected a developmental iteration.

Remove some unused variables

Merge pull request #20486 from civicrm/5.38

5.38

[php8-compat][REF] Fix php8 error on undefined constant CIVICRM_DISABLE_DEFAULT_MENU

Merge pull request #20483 from seamuslee001/hash_equals_string

[php8-compat][NFC] Ensure that the 2nd parameter of hash_equals is a …

Merge pull request #20484 from agh1/5.38.0-releasenotes-final

5.38.0 release notes: added late changes

Merge pull request #20480 from colemanw/afformCustomFields

Afform - Fix custom field handling and add tests