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 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

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

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

5.38.0 release notes: added late changes

Merge pull request #20458 from eileenmcnaughton/group2

Extract code that populates temp table for an individual group

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

Merge pull request #20481 from colemanw/getFieldsFix

APIv4 - Fix getFields to respect default_value from getFields

dev/translation#67 - Add titles to XML+DAO

APIv4 - Fix getFields to respect default_value from getFields

GetFields had an odd way of setting defaults for field metadata;
this reuses the default_value property instead for more internal consistency.

Afform - Fix custom field handling and add tests

This ensures custom fields are handled properly by Afform,
including multi-record custom field groups & their autogenerated blocks,
and contact reference fields.

Merge pull request #20475 from eileenmcnaughton/trigg

Remove pass-by-ref as param not altered

APIv4 - Refactor Entity.get and deprecate the includeCustom param

This makes Entity.get more efficient - it no longer does a file scan when getting one or more entities by name.
Also deprecates the includeCustom param which was redundant with the where clause.
This is the first APIv4 param to be deprecated - added handling to emit a warning when using a deprecated param
and hide deprecated params in the APIv4 explorer.

deprecated function

Merge pull request #20477 from seamuslee001/core_ext_phpunit8

[php8-compat][phpunit8][NFC] Fix PHPUnit Warnings and fix a php error on f…

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

This creates an entity, `Translation` (`civicrm_translation`), to represent a single translated database value. Loosely speaking,
any field in the database can be designated as translatable -- and then it will be permitted to store values like:

```sql
INSERT INTO civicrm_translation (entity_table, entity_id, entity_field, language, string)
VALUES ('civicrm_event', 100, 'title', 'fr_FR', 'La nouvelle chaine')
```

This is based on a `civi-data-translate` strings table, but with some changes:

* Entity names are usually singular, but `String` is conflicted. I previously used hybrid
  String/Strings (depending on context), but we negotiated `Translation` on tcon.
* The language only needs 5 characters (NN_nn).
* Consolidated `bool is_active` and `bool is_default` into one `int status_id`.
* Added indexing
* Mark dynamic foreign key

This commit includes the BAO with some of the backing-methods required for
API exposure.  However, the API won't really work until we have the
validation-values event, so the API has been kicked to a subsequent PR.

The list of translatable entities/fields will be signficant because it will
determine when/how to redirect data in API calls.  This patch does not
commit to specific translatable fields - but it does provide a hook to
determine them.

When the API PR becomes unblocked, it will include test-coverage that hits the API, BAO, and hook.

Add DedupeRule, DedupeRuleGroup and DedupeException APIv4 entities

[php8-compat][phpunit8] Fix PHPUnit Warnings and fix a php error on function declaration syntax in core extension tests

Merge pull request #20473 from seamuslee001/php8_deprecated_required_follow_optional

[php8-compat] Fix deprecation error where by required function parame…

Remove pass-by-ref as param not altered

[php8-compat] Fix deprecation error where by required function parameter follows an optional function parameter

Prevent adding duplicate dashlet if present with same name and label

Merge pull request #20469 from civicrm/5.38

5.38

Merge pull request #20465 from MikeyMJCO/patch-11

Bump DOMPDF to `~1.0.0`

Merge pull request #20467 from colemanw/getSearchTasksPermissions

SearchKit - Update `hook_civicrm_searchKitTasks` signature

Merge pull request #20460 from eileenmcnaughton/logg

[Ref] [tests only] replace direct calls to enable logging with calls to the setting

Merge pull request #20468 from alifrumin/rn5.38

[NFC] 5.38 Release notes First pass

[NFC] Release notes 5.38 first pass

[Ref] replace direct calls to enable logging with calls to the setting

SearchKit - Update hook_civicrm_searchKitTasks signature to include checkPermissions and userId

Merge pull request #20459 from eileenmcnaughton/trigg

  [Ref] Remove function parameter only used from test

Update composer.lock

Merge pull request #20457 from colemanw/id_field

APIv4 - Return id_field as part of Entity.get

Merge pull request #20464 from JMAConsulting/add_financial_metadata

dev/core#2486 Add pseudoconstant callback for LineItem and Financial item entity

Merge pull request #20463 from mattwire/transactdopayment

Don't pass doPayment by reference in Contribution.transact API

Merge pull request #20433 from seamuslee001/financial_item_v4

dev/core#2486 Add in FinancialItem APIv4 Entity

Bump DOMPDF to `~1.0.0`

Having reviewed the changelogs and actual changes here https://github.com/dompdf/dompdf/releases and following a run-test on this I can't find any broken functionality as a result of this upgrade.

There are numerous fixes and improvements in this and the next planned version 1.0.3 will further improve SVG handling.

Add pseudoconstant callback for LineItem and Financial item entity

Don't pass doPayment by reference in Contribution.transact API

Fix trigger rebuild to avoid deprecated function

[Ref] Remove function parameter only used from test

The test passes a function parameter to rebuild triggers but the only other
call to this function in the civi-verse doesn't. We shouldn't make this function
more complex for just the test.

Note the function also uses the force param which I didn't move to the
test call - it's possible when the tests all run together I'll need
to force cache clearing but this seems like a dumb way

Extract code that populates temp table for an individual group

This is part of breaking the individual group actions (this might
actually be the only one that needs to be done on an individual group)
into single actions. Then the ones that could be on one
or more groups can all be consistent.

Note this is only called from one place and is protected (& covered by tests)
so there is no practical change

Merge pull request #20456 from eileenmcnaughton/group2

[Ref] remove never-passed param

APIv4 - Return id_field as part of Entity.get

All entities have a unique identifier field, usually named 'id'
but some entities the field is named something else.
e.g. Afform uses 'name' as the identifier.

This returns the name of the field as part of Entity.get,
and it's also available directly from each API class e.g. Contact::getInfo().

Merge pull request #20455 from eileenmcnaughton/group

Make functions protected

[Ref] remove never-passed param

Removes a parameter that is never passed in

Merge pull request #20448 from eileenmcnaughton/report

Mark test class as having invalid financials

Make functions protected

I did a universe search & determined these functions could be protected
as they are not called from outside this class

Merge pull request #20454 from colemanw/organizeOnClause

SearchKit - Organize ON clause field selectors with joined entity first

Merge pull request #20427 from totten/master-scanner

BAOs, Tests, etal - Support `HookInterface` and `EventSubscriberInterface` for auto-registration

Merge pull request #20431 from colemanw/garlandTabFix

AfformGui - Fix tabs when cms theme adds extra margin

Merge pull request #20440 from colemanw/apiCalculatedField

[REF] APIv4 refactoring to support calculated fields

SearchKit - Organize ON clause field selectors with joined entity first

Merge pull request #20434 from colemanw/groupContact

SearchKit - Add static groups and organize main entity selector

APIv4 - Ensure getFields always returns table_name and column_name

Merge pull request #20447 from magnolia61/missing_image_fatal_error

dev/core#2369 - Change missing image fatal error to 404

change missing image fatal error to 404

EventPrinter - More aggressive type-hinting

For Symfony-style event listeners, encourage use of `void` return

When using Symfony-style listeners, all inputs and outputs for the event go
through the event object.

Note that `hook_*()` notation does allow return values, For these functions,
it is *normal* to return void, but some existing hooks rely on returning
array-data.  It could be misleading if we made it appear that all `hook_*()`
examples have to return void.

HookStyleListener - Convey hook results. Refine type-hints.

Note: This is uncommon and discouraged for new hooks, but some hooks require returning values, e.g

```php
function hook_foo() {
  return ['my-data'];
}
```

This should fix compatibility with those.