Fix broken tests highlighted by but not caused by pull 23928

Initial go at cleaning up repeattransaction

Merge pull request #23921 from civicrm/5.51

5.51 to master

Merge pull request #23920 from MegaphoneJon/url-validation-regression

fixes #3706 URL validation regression

fixes #3706 URL validation regression

Merge pull request #23914 from seamuslee001/file_substr_null

[REF][PHP8.1] Ensure that NULL is not passed to substr in CRM_Utils_F…

Merge pull request #23916 from totten/master-wfmsg-data-boot

WorkflowMessageTest - Fix weird inconsistency in resolving `@dataProvider`

WorkflowMessageTest - Fix weird inconsistency in resolving `@dataProvider`

We see a failure scenario where:

* The first execution of WorkflowMessageTest runs fine.
* The second execution of WorkflowMessageTest fails to boot (while evaluating `@dataProvider`)
* If you reset the DB, then it works again - but only once.

[REF][PHP8.1] Ensure that NULL is not passed to substr in CRM_Utils_File::addTrailingSlash

Merge pull request #23899 from MegaphoneJon/civigrant-afform

Grants appear in afform

Merge pull request #23871 from agileware/CIVICRM-2004

Personal Campaign Pages (PCP) incorrectly displays "Don't list my contribution in the honour roll"

Merge pull request #23909 from civicrm/5.51

5.51 to master

Merge pull request #23893 from eileenmcnaughton/user_two

dev/core#3514 Add interface to allow extensions that deal with imports to cope with 5.51

Merge pull request #23907 from eileenmcnaughton/up

5.51 to master

Merge branch '5.51' of github.com:civicrm/civicrm-core into up

Fix Parser classes to use ClassScanner

Merge pull request #23888 from eileenmcnaughton/user_import

Convert the type on the UserJob entity to be a string

Import Summary - Update to match final schema ('UserJob.job_type')

eg When running "Import Activities", the page-title for the summary is incorrect.

Merge pull request #23903 from eileenmcnaughton/551

Allow tests to pick up hook too

Merge pull request #23902 from civicrm/5.51

5.51 to master

Merge pull request #23889 from eileenmcnaughton/offline_cur

Add second currency example

Merge pull request #23901 from totten/nfc-install-docblock

(NFC) Expand docblocks for hook_install and hook_postInstall

Allow tests to pick up hook too

Merge pull request #23900 from totten/5.51-sticky-classloader

(dev/core#3660) CRM_Extension_ClassLoader - Defend against redundant refreshes

Merge pull request #23894 from eileenmcnaughton/off_it

Minor test cleanup

Examples - Title should differentiate EUR/CAD examples

Merge pull request #23891 from eileenmcnaughton/smarty

Reset smarty-security after an exception

(NFC) Expand docblocks for hook_install and hook_postInstall

Add second currency example

Grants appear in afform

(dev/core#3660) CRM_Extension_ClassLoader - Defend against redundant refreshes

Overview
--------

This is a follow-up to #23824 (c24dd7db7e1e91120fd7daeb7e151f856d6b78c3) which addresses a
regressive edge-case.

Steps to Reproduce
------------------

* Write an extension like `wmf-civicrm` which (a) calls `System.flush` (`rebuildMenuAndCaches()`)
  during `hook_install` -- and then (b) loads some class from the same extension.
    ```php
    function foo_civicrm_install() {
      civicrm_api3('System', 'flush', []);
      CRM_Foo_Helper::doStuff();
    }
    ```
* Try to install the extension.

Before
------

Crashes on loading the class `CRM_Foo_Helper`

After
-----

Loads the class `CRM_Foo_Helper`.

Comments
--------

(1) To see what's happening, consider `CRM_Extension_Manager_Module::onPreInstall()`.
This registers the new classloader and then fires `hook_install` which eventually
fires `rebuildMenuAndCaches()`. With c24dd7db, this resets the classloader again.
But the extension isn't fully installed yet - so it forgets about the new extension.

(2) Is it safe to have some (temporarily) sticky items?  Ish.  You might
say: "Ah, but what if we need to remove an extension?  Won't this static
variable retain stale things?" Doesn't matter.  In PHP, classloading is a
one-way-street.  (You cannot unload.) So you'll still have old classes in
memory, regardless of whether the `ClassLoader` has some old metadata
about where to find classes.

(3) I'm on the fence about whether this patch is a good idea.  Calling
`System.flush` explicitly in this context seems like an invitation to
trouble.  OTOH, it worked before #23824, so it can be called a regression.

Merge pull request #23898 from civicrm/5.51

5.51 to master

Merge pull request #23897 from agh1/5.51.0-releasenotes-initial

5.51.0 release notes initial run

Set version to 5.51.beta2. Retargeg FiveFiftyOne::updateUserJobTable to beta2.

5.51.0 release notes: added boilerplate

Convert the type on the UserJob entity to be a string

Currently we are storing a numeric ID - however if we
permit non-core classes to register types we find numeric ids
quickly become hard to manage as uninstalling an extension could
change the id. This switches to using a string type

Minor test cleanup

Merge pull request #23852 from IProSoft/master

New domain token base_url

Merge pull request #23892 from eileenmcnaughton/order

Ensure apiv3 params are converted at the line item level

New domain token base_url

Change url retrival method for base_url token

Change base url token title. Fix test error

PHPUnit TokenConsCRM_Utils_TokenConsistencyTest update

Ensure apiv3 params are converted at the line item level

We switched to using apiv4 but at least one test seems to be passing in a string
at the line item level which is a bit weird but we should handle it

Reset smarty-security after an exception

This stops tests from getting befuddled

Merge pull request #23887 from civicrm/5.51

5.51 to master

Merge pull request #23854 from totten/master-mixin-wfmsg

Scan for classes based on the PHP interface (WorkflowMessageInterface, ExampleDataInterface)

Merge pull request #23886 from totten/master-act-dao

Activity - Regenerate DAO with current description

Merge pull request #23885 from totten/master-wfmsgtest

WorkflowMessageTest - Re-enable skipped test

Activity - Regenerate DAO with current description

ExampleDataInterface - Switch to ClassScanner (part 3, swap scanner)

ExampleDataInterface - Switch to ClassScanner (part 2, use new names)

ExampleDataInterface - Switch to ClassScanner (part 1, rename *.ex.php => *.php)

WorkflowMessage - Switch to `ClassScanner` (was: bespoke `include_path` search)

mixin/scan-classes@1 - Allow extensions to opt-in to class scanning

ClassScanner - Add utility for scanning classes based on PHP interfaces

CRM_Utils_System::flushCache - More consistent behavior. Prepare for interface-php test.

* The interface-php test registers an instance of WorkflowMessageInterface.
* The list of WorkflowMessageInterface's is stored in the 'long' cache.
* When you enable/disable an extension, it should clear WorkflowMessage list (and other things).
* Before this patch, interface-php's `LifecycleTest` is inconsistent: it passes with
  `testLifecycleWithSubprocesses()` and fails with `testLifecycleWithLocalFunctions()`.
* After this patch, interface-php's `LifecycleTest` is consistent: it passes with both
  `testLifecycleWithSubprocesses()` and  `testLifecycleWithLocalFunctions()`.
* The problem - while executing `testLifecycleWithLocalFunctions()`, it evidentally uses
  `Arraycache` as the backend -- and thus skips the flushes. However, I cannot fathom why
  one would want to clear-caches for SQL+memcache+redis but keep them for Arraycache.

WorkflowMessageTest - Re-enable skipped test.

This is a follow-up to #23811 (409b1e0c30d28179f6244c1f180838a473d7582c). The prior commit
changed the way it found examples - and caused it to skip some examples.

This re-enables the example and adds an assertion to ensure that it actually
finds some examples.

Merge pull request #23879 from mattwire/wpnotify

Enable the 'notify' parameter for WordPress user creation

Merge pull request #23881 from civicrm/5.51

5.51

Merge pull request #23874 from AlainBenbassat/empty-weight-column-during-upgrade

returns weight 0 when weight column name is empty

Merge pull request #23722 from jensschuppe/fix/multipleLogfilesMultilanguage

Avoid multiple log files in multi-language environments

Merge pull request #23878 from herbdool/issue-3691

dev/core#3691: better description of activity table columns

Merge pull request #23876 from colemanw/checkRecentItemsPerms

RecentItems - Enforce permissions before adding item

Merge pull request #23602 from kainuk/core-3369

Fixes core#3369 scheduled Reminder limited by Participant Role fails if any participant has multiple roles

Enable the 'notify' parameter for WordPress user creation

Issue 3691: better description of activity table columns

5.51.0 release notes: raw from script

RecentItems - Enforce permissions before adding item

Merge pull request #23856 from eileenmcnaughton/pref

dev/core#2866 ignore preferred_mail_format, send both

returns weight 0 when weight column name is empty

Merge pull request #23873 from colemanw/uiDialogBackground

dev/core#3432 UI Dialog - Fix transparent background caused by certain other plugins

dev/core#2866 ignore preferred_mail_format, send both

Merge pull request #23872 from demeritcowboy/valuestyle

Unassigned variable in invoice message template

UI Dialog - Fix transparent background caused by certain other plugins

Fixes dev/core#3432
It was reported that certain Drupal modules that include a copy of jQuery UI
cause the background to become transparent. This should prevent that.

Merge pull request #23831 from mlutfy/purgeMenus

navigation: cleanup less frequently used New items

Unused variable. Meaningless comment.

Merge pull request #23829 from eileenmcnaughton/inv_lines

Replace more assigns with tokens in invoice, add tests

CIVICRM-2004 CiviCRM Personal Campaign Pages (PCP), incorrectly display "Don't list my contribution in the honor roll" even if the Contribution Honour Roll feature is not enabled or the user has opted out of the Honour Roll

Swap out some additional assigns for tokens, improve tests

Merge pull request #23868 from civicrm/5.51

5.51 to master

Merge pull request #23867 from demeritcowboy/currency551

dev/core#3680 - Currency incorrectly displayed for price set when default is not USD

navigation: cleanup less frequently used New items

use the actual currency if there is one not the default

Merge pull request #23866 from civicrm/5.51

5.51 to master

Merge pull request #23865 from colemanw/behalf

dev/core#3675 Fix Permissioned Relationship pseudoconstant

dev/core#3675 Fix Permissioned Relationship pseudoconstant

Fixes dev/core#3675 Permissioned Relationship no longer set up
for "On Behalf of Organization" Membership Contribution Form submissions

Merge pull request #23860 from civicrm/5.51

5.51

Merge pull request #23847 from eileenmcnaughton/sync

Attempt to make function more readable by extracting in-line function

Merge pull request #23839 from eileenmcnaughton/import_outy

dev/core#3665 import summary fixes

Merge pull request #23849 from eileenmcnaughton/import_55

dev/core#3673 do not attempt to import empty related contact

Merge pull request #23855 from mlutfy/fixAccentsIcalAgain

dev/core#1541 Fix ICalendar random invalid utf8 (PHP <7.4 compat) and tests

Merge pull request #23836 from stesi561/dev/core/#3671

Refs: dev/core#3671 Fix regression involving CiviCRM Webform + Cases.

Attempt to make function more readable by extracting in-line function

Merge pull request #23858 from seamuslee001/guzzle_658

[REF] Update Guzzle to be 6.5.8

Merge pull request #23846 from seamuslee001/fix_warning_karma

[REF] Follow on from #23749 and remove refernece to ui-utils from kar…

[REF] Update Guzzle to be 6.5.8

Merge pull request #23850 from demeritcowboy/rsspubdate

Alternate to 23826 - Events RSS feed does not output a pubDate for each Event

Merge pull request #23853 from pradpnayak/updatecasefixes

Fixed fatal error on update multiple case

dev/core#1541 Fix ICalendar random invalid utf8 (PHP <7.4 compat) and tests

Fixed fatal error on update multiple case

dev/core#3673 do not attempt to report empty related contact

Merge pull request #23834 from colemanw/deleteCustomOption

Call hooks when deleting an option value from CustomOption BAO