Merge pull request #25463 from demeritcowboy/profile-null

php8 - don't pass null to htmlspecialchars on profile admin page

Merge pull request #25462 from demeritcowboy/number

dev/core#4097 - Add deprecation notice as per comments at #25448

Merge pull request #25453 from eileenmcnaughton/notice_no_display

Smarty notice fix - check array key exists before querying

don't pass null to htmlspecialchars

Merge pull request #25323 from herbdool/core-4074

(dev/core#4074) Make CRM_Core_BAO_CMSUser CMS agnostic

add deprecation notice

Merge pull request #25448 from redcuillin/stringfix_formatLocaleNumeric

dev/core#4097 Fix empty string $amount error

Merge pull request #25452 from eileenmcnaughton/notice

Report tpl enotices fix

Merge pull request #25459 from eileenmcnaughton/is_test

Fix some more smarty assignments to assign even when FALSE

Fix some more smarty assignments to assign even when FALSE

Another array_key_exists check for notice purposes

Merge pull request #25451 from eileenmcnaughton/num

Do not attempt to format custom value of empty string

Smarty notice fix - check array key exists before querying

Report tpl enotice fix

Do not attempt to format custom value of empty string

Merge pull request #25450 from braders/tidyup-detail-test

[REF][PHP8.2] Tidy up of CRM_Report_Form_Member_DetailTest

Merge pull request #25447 from braders/properties-hooktest

[REF][PHP8.2] Declare missing property on CRM_Utils_HookTest

[REF][PHP8.2] Tidy up of CRM_Report_Form_Member_DetailTest

[REF][PHP8.2] Declare missing property on CRM_Utils_HookTest

Merge pull request #25449 from braders/dynamic-properties-participant-test

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

Merge pull request #25426 from seamuslee001/php82_payment_insturment_contribution

[REF][PHP8.2] Fix Dynamic property payment instrument id on back offi…

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

Fix empty string $amount error

Merge pull request #25440 from dontub/fix/Case-CustomDataView

Fix display of custom data in cases

dev/core#4074 make CRM_Core_BAO_CMSUser CMS agnostic

Merge pull request #25419 from mlutfy/l10nExt

dev/translation#76 Allow extension gettext mo files to live in the I18N resource dir

Merge pull request #25371 from herbdool/backdrop-77

(dev/backdrop#77) Fix fatal error with password validation

Merge pull request #25409 from colemanw/entityAutocomplete

APIv4 - Support autocompletes of the Entity entity

Merge pull request #25401 from braders/declare-system-properties

[REF][PHP8.2] Declare system property in tests

Merge pull request #25439 from totten/master-test-tx

(NFC) Speed up random tests by 55-60%

Merge pull request #25405 from braders/dynamic-property-infotest

[REF][PHP8.2] Declare file property in CRM_Extension_InfoTest

Merge pull request #25443 from braders/dynamic-properties-ajaxtest

[REF][PHP8.2] Declare properties in CRM_Activity_Page_AJAXTest

Merge pull request #25442 from braders/dynamic-property-casetest

[REF][PHP8.2] Refactor CRM_Case_BAO_CaseTest to use const

[REF][PHP8.2] Declare properties in CRM_Activity_Page_AJAXTest

[REF][PHP8.2] Refactor CRM_Case_BAO_CaseTest to use const

Fix display of custom data in cases

(NFC) Speed up more random tests, part 4 (58s=>29s)

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

[REF][PHP8.2] Fix Dynamic property payment instrument id on back office contribution form

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

Allow extension gettext mo files to live in the I18N resource dir

CiviCRM core already supports having the civicrm.mo files in a custom
directory, defined by the CIVICRM_L10N_BASEDIR constant. With this patch,
it will also be possible to have the 'mo' files from extensions in that
directory.

It will be assumed that if an extension is called "foo" then the mo file
will be foo.mo and in the same directory as civicrm.mo.

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

merge form errors into existing errors instead

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

APIv4 - Support autocompletes of the Entity entity

Allows browsing API entities with the APIv4 autocomplete select

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

[REF][PHP8.2] Declare file property in CRM_Extension_InfoTest

use civi statics

[REF][PHP8.2] Declare system property in tests

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