(NFC) Speed up more random tests, part 3 (26s=>15s)

(NFC) Speed up more random tests, part 2

(NFC) Speed up more random tests

(NFC) ArrayTest - Speed up ~50%

Before: On local i3-12100, takes ~17s (avg over two runs)

After: On local i3-12100, take ~8s (avg over two runs)

(NFC) StringTest - Speed up ~60%

Before: On local i3-12100, takes ~34s (avg over two runs)

After: On local i3-12100, take ~14s (avg over two runs)

(NFC) JSTest - Speed up ~60%

Before: On local i3-12100, takes ~34s (avg over two runs)

After: On local i3-12100, take ~14s (avg over two runs)

(NFC) RuleTest - Speed up ~55%

Before: On local i3-12100, takes ~64s (avg over two runs)

After: On local i3-12100, take ~28s (avg over two runs)

(NFC) DateTest - Speed up ~55%

Before: On local i3-12100, takes ~90s (avg over 2 runs)

After: On local i3-12100, take ~40s (avg over 2 runs)

Merge pull request #25425 from civicrm/5.58

5.58

Merge pull request #25412 from eileenmcnaughton/api4trait

dev/core#4088 Convert api4 helper functionality to a trait & make available

Merge pull request #25421 from aydun/fix_core_4095

dev/core#4095 - update log schema of civicrm_option_group before adding values

Merge pull request #25424 from totten/5.58-json-wp

distmaker - Fix 'WordPress' entry in JSON report

Merge pull request #25423 from seamuslee001/5.58

[REF] Bump UA Parser to 0.7.33

distmaker - Fix 'WordPress' entry in JSON report

[REF] Bump UA Parser to 0.7.33

Merge pull request #25417 from eileenmcnaughton/poly

Update polyfill packages

dev/core#4095 - update log schema of civicrm_option_group before adding values

Update polyfill packages

 - Upgrading symfony/polyfill-php80 (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-ctype (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-iconv (v1.17.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-mbstring (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-php72 (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-intl-normalizer (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-intl-idn (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-php81 (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-php73 (v1.25.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-php74 (v1.26.0 => v1.27.0): Extracting archive
  - Upgrading symfony/polyfill-php82 (v1.26.0 => v1.27.0): Extracting archive

Merge pull request #25416 from totten/master-scan-fix

CaseActivityTest - Fix quiet regressions

Merge pull request #25396 from colemanw/customMaskDrop

CustomFields - Drop unused column 'mask'

Merge pull request #25398 from demeritcowboy/extendedreport-contributionpage

[NFC] Incorrect docblock in CRM_Contribute_PseudoConstant::contributionPage()

Merge pull request #25407 from braders/dynamic-property-xmlprocessortest

[REF][PHP8.2] Declare processor property in CRM_Case_XMLProcessorTest

CaseActivityTest - Fix regression

The relevant example data was changed by 98e528a20637e880863371a1277f13a887e37eb5.

Restore execution of CaseActivityTest

Following eb92dd792c07e0b11ee1561cf00930402345e8b3, the `CaseActivityTest` started to run
only intermittently. Why?

__high-level__: `Civi\Core\ClassScanner` and `phpunit8` both do a scan over the folder `tests/phpunit/CRM/Case/WorkflowMessage`

__low-level__: `Civi\Core\ClassScanner` has caching. Depending on the state of the cache, it may or may not do a scan:

* If the cache is filled, then `ClassScanner` doesn't need to scan.
    * When `phpunit8` subsequently does a scan, it will load `CaseActivityTest.php` normally.
* If the cache is empty, then `ClassScanner` does the first scan. It is the one that actually loads `CaseActivityTest.php`.
    * Later, `phpunit8` does a scan. Due to a quirk, it doesn't realize the class exists.

The scanner in phpunit works roughly like this:

```php
$tests = [];
foreach (glob('*Test.php') as $file) {
  $before = get_declared_classes();
  require_once $file;
  $after = get_declared_classes();
  $tests = array_merge($tests, array_diff($before, $after));
}
```

So if the class was previously loaded, then phpunit doesn't see it.

Merge pull request #25415 from totten/master-scan

(dev/core#4088) ClassScanner - Move test registration

Cleanup comments, remove unnecessary public function

Use Trait directly from managed entity test

I'm not sure if the test fail was an intermittent or related but this seems cleaner

Move over testRecords property

Clarify function signature

It was doing both pass-by-ref and return to give back the value - this
settles on using return.

Merge pull request #25404 from demeritcowboy/activityinfos-static

[REF] Use civi::statics to get activityinfos in case activity view

(dev/core#4088) ClassScanner - Move unit-test registration

Before: The `ClassScanner` includes a special  rule to load some
mocks/examples from `tests/phpunit/` which are needed for some core tests.
But (reportedly) it will load even when running other test-suites.

After: The special rule has moved to the `bootstrap.php` for
core tests. It should be inert when running other test-suites.

Technical Details: I believe the reason why the special rule was originally
embedded into `ClassScanner` was that `ClassScanner` has special place in
bootstrap/system-lifecycle.  To get around this, the patch adds
`CIVICRM_FORCE_MODULES` as a way to pre-register some hook listeners.  (To
wit: `civitest` is a "force-enabled module" defined by `bootstrap.php`.
It's always-on; it can participate in special/pre-boot hooks; and it
doesn't present as a configurable extension. You might also call it
a "ghost module"...)

Merge pull request #25413 from seamuslee001/regen

[REF] Regen following #25211

Fix country id in state_provice_data as per Dave and fix the original state abbreviations as well and regen again

[REF] Regen following #25211

Merge pull request #25211 from jmacarthur98/luxembourg-provinces

luxembourg-provices added missing provices, updated outdated ISO code…

Merge pull request #25369 from totten/master-smarty-mixin

Smarty Mixin - Allow extensions to enable Smarty via mixin

Convert api4 helper functionality to a trait & make available

(NFC) smarty-v2 mixin

luxembourg-provinces added check for country ID

luxembourg-provinces removed one unnecessary comma, added one comma

luxembourg-provices added missing provices, updated outdated ISO codes, renamed one provice

Merge pull request #25397 from colemanw/customFieldTplCleanup

CustomField - Fix smarty notices on field create/edit form

[REF][PHP8.2] Declare processor property in CRM_Case_XMLProcessorTest

use civi statics

Merge pull request #25399 from braders/dynamic-property-report-test

[REF][PHP8.2] Avoid dynamic property in ReportTest

[REF][PHP8.2] Avoid dynamic property in ReportTest

Merge pull request #25229 from braders/core-2985-original-value-displayed-after-setting-blank-balue

dev/core#2985 Original value is displayed after setting custom event field blank

incorrect docblock

CustomField - Fix smarty notices on field create/edit form

The structure of the form was changed back in 2020 to stop using a hierarchical quickform select,
so references to `$form.data_type.value.0` etc are no longer valid.
This was resulting in massive smarty notices.

This removes the offending clauses which were not doing anything as the show/hide is now handled by jQuery.

Merge pull request #25394 from civicrm/5.58

5.58

CustomFields - Drop unused column 'mask'

This column doesn't appear to be used for anything.

Merge pull request #25381 from demeritcowboy/case-act-vars

Undefined array keys on case activity view

mixin/smarty-v2 - Update for compatibility with more versions of CiviCRM

Merge pull request #25380 from demeritcowboy/act-rev

Avoid 5.57 upgrade taking hours on large activity tables

Merge pull request #25393 from seamuslee001/php82_legacy_custom_searches

[REF][PHP8.2] Fix Deprecated Dynamic class properties in Legacy custo…

[REF][PHP8.2] Fix Deprecated Dynamic class properties in Legacy custom searches extension

Merge pull request #25389 from eileenmcnaughton/main_pledge

Make otherwise unused form function private

avoid upgrade taking hours

Merge pull request #25379 from totten/master-ext-cache

(dev/core#4055) ClassLoader - Use separate cache IDs for different configurations of modules

Merge pull request #25382 from demeritcowboy/case-act-vars2

Undefined `accessKey` on case activity view

Merge pull request #25374 from colemanw/fixChange

Fix dev/core#4083 - Broken autocomplete Afform widget

Merge pull request #25376 from mattwire/paypalcleanup

Paypal cancel/notify/return URLs

Merge pull request #25388 from braders/dynamic-property-provider-test

[REF][PHP8.2] Avoid dynamic property in CRM_SMS_ProviderTest

(NFC) More type-hints

(NFC) smarty-v2 - Add more explanatory comments

smarty-v2 - Avoid duplicate registrations

search_kit, civigrant - Use mixin 'smarty-v2'

shimmy - Remove old boilerplate for registering Smarty folder

Add mixin/smarty-v2

Make otherwise unused form function private

[REF][PHP8.2] Avoid dynamic property in CRM_SMS_ProviderTest

Merge pull request #25386 from civicrm/5.58

5.58

Merge pull request #25370 from pradpnayak/waitlist

Waitlist

Do not display waitlist message dev/core/-/issues/4087

Merge pull request #25188 from aydun/optgroups_meta

Enhance metadata for OptionGroups

OptionGroup - Add paths metadata

Merge pull request #25384 from totten/master-void

(NFC) Mixin Tests - Add more type-hints

(NFC) Mixin Tests - Add more typehints

Merge pull request #25378 from eileenmcnaughton/test_base

dev/core#4088 Superficial cleanup on Api4testBase

undefined array keys

undefined vars

dev/core#4088 Superficial cleanup on Api4testBase

(dev/core#4055) ClassLoader - Use separate cache IDs for different configurations of modules

Merge pull request #25354 from seamuslee001/fix_ewaysingle_tests

[REF][PHP8.2] Fix Eway Single Payment Processor extension for dynamic…

Merge pull request #25372 from braders/dynamic-properties-contribution-page-test

[REF][PHP8.2] Use const instead of dynamic property: CRM_Contribute_BAO_ContributionPageTest

Merge pull request #25373 from braders/dynamic-properties-ziptest

[REF][PHP8.2] Declare $file property in CRM_Utils_ZipTest

Merge pull request #25375 from braders/dynamic-properties-event-test

[REF][PHP8.2] Avoid dynamic properties in api_v3_EventTest

Paypal: Use standard functions for cancelUrl / notifyUrl

Paypal (express): Pass participantID to getCancelUrl() if available

Payment: getCancelUrl participant default value of NULL

Paypal: Use getBaseReturnUrl() function

Paypal: Stop passing component when it exists on the class

[REF][PHP8.2] Avoid dynamic properties in api_v3_EventTest

Fix dev/core#4083 - Broken autocomplete Afform widget

[REF][PHP8.2] Declare  property in CRM_Utils_ZipTest

[REF][PHP8.2] Use const instead of dynamic property: CRM_Contribute_BAO_ContributionPageTest

Merge pull request #25298 from eileenmcnaughton/import

Php8.2 Contact import Map Field screen - remove undefined property usage (mutliple)

Merge pull request #25318 from eileenmcnaughton/cust_load

Pass metadata values rather than convoluted array to `processCustomFields`

Merge pull request #25239 from eileenmcnaughton/cont_mode

Remove a couple of legacy uses of `contributeMode`

Merge pull request #25259 from eileenmcnaughton/cont_ids

Deprecate meaningless parameter