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

Merge pull request #20456 from eileenmcnaughton/group2

[Ref] remove never-passed param

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.

EventScanner - More aggressive type-hinting

Merge pull request #20395 from eileenmcnaughton/entry2

[REF] Set receipt_date after sending, (from batch) not in anticipation

Merge pull request #20424 from eileenmcnaughton/ppp3

dev/core#2593 Fix failure to complete contributions when membership deleted

Merge pull request #20449 from seamuslee001/phpword_upgrade

[php8-compat] Upgrade PHPWord Package to support php8

[WIP][php8-compat] Upgrade PHPWord Package to support php8

Mark test class as having invalid financials

Marks the class as unsuitable for financial validation due to use
of mysql to set up contributions without related entities

APIv4 - Refactor field options getter into a swappable callback function

This allows any fieldSpec provider to change the callback used to build field options,
or to add an ad-hoc field not in the schema with its own option getter.

Merge pull request #20446 from eileenmcnaughton/order

[NFC] Superficial code cleanup

Merge pull request #20376 from MegaphoneJon/financial-173

financial#173: Show 'Pay Now' on user dashboard for partially paid contributions

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

[NFC] Superficial code cleanup

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.

SearchKit - Organize main entity selector with primary entities first

Adds a collapsed optgroup for secondary entities so they are available
but not as prominent as the commonly-searched primary entities.

Select2 - Move collapsible feature from Api4Explorer to Common.js for reusability in SearchKit

SearchKit - Change @searchable annotation from boolean to option list

In preparation for showing different tiers of entities, make this a string
instead of a boolean property, to mark searchability as 'primary', 'secondary', or 'none'.

SearchKit - Improve descriptions for Relationship & GroupContact joins

This changes the 'bridge' metadata property to a multi-dimensional array
to hold extra info like description.

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

Show 'Pay Now' on user dashboard for partially paid contributions

SearchKit - Fix GroupContact so static groups are searchable

AfformGui - Fix tabs when cms theme adds extra margin

Themes like Drupal's 'Garland' add extra margin to `li` elements which
interferes with the tab spacing in the Afform GUI.

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.

(REF) CRM_Api4_Services - Switch to EventScanner

(REF) RecurringEntity BAO - Switch EventSubscriberInterface

This is just an example of doing declarative event-registration from a BAO.

(REF) RelationshipCache BAO - Switch to HookInterface

This is just an example of doing declarative event-registration from a BAO.

BAOs - Allow BAOs to subscribe to hooks/events via `EventSubscriberInterface` or `HookInterface`

Before: There is no pleasant way for a BAO to register a listener for an
event.  True, one can use `Civi::dispatch()`, but (lacking a good
initialization point) it's hard to register reliably.  True, you can also
add new items to `\Civi\Core\Container::createEventDispatcher()`, but
this is rather out-of-the-way.

After: A BAO class may optionally implement `EventDispatcherInterface` or
`HookInterface`. Methods will be automatically registered. The list
of listeners will be cached in the container.

CiviTestListener - Move scanning responsibility from 'CiviTestListener' to 'EventScanner'

The removes duplicate code and expands the functionality to support methods
of the form `_on_{$symfony_event}()`.

CiviTestListener - Provide a variable to identify the active test

EventScanner - Add utility class which can auto-identify listeners in an object

```
$map = EventScanner::findListeners($someObject);
$dispatcher->addListenerMap($someObject, $map);
```

Testing - Add more examples of tests which register listeners

Merge pull request #20102 from seamuslee001/d8_mailing_urls

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

dev/core#2593 Fix failure to complete contributions when membership deleted

Final cleanup on paypal pro fixes https://lab.civicrm.org/dev/core/-/issues/2593

[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