fix exception type

Merge pull request #25420 from eileenmcnaughton/snap

Fix slow queries in snapshot process

Merge pull request #25473 from eileenmcnaughton/email

dev/core#4096 Rename Primary Email to Is Primary

Merge pull request #25433 from eileenmcnaughton/sk_admin

Do not crash the whole SearchKit UI if one entity fails

Merge pull request #25475 from dontub/update-contributor-key

Add dontub to contributor-key.yml

Add dontub to contributor-key.yml

dev/core#4096 Rename Primary Email to Is Primary

This field caused us problems because people mistook it
for .... the primary email field.... Renaming per agreement
on gitlab

(NFC) CRM/Upgrade/Snapshot.php - Work-around for bug in linter

The linter for `@return` has a bug; it expects to see a `return`
and doesn't recognize the `yield` notation (which is radically new,
having been added in PHP 5.5 circa 2013).

Merge pull request #25418 from eileenmcnaughton/imap

Increase timeout on imap

Merge pull request #25468 from civicrm/5.58

5.58 to master

Merge pull request #25467 from demeritcowboy/resubscribe

Some renames missing from #24982

earlier rename missing this one

Merge pull request #25334 from eileenmcnaughton/string_exception

Ensure useful exception thrown for all Smarty errors from user strings

Add preUpgrade message

Merge pull request #25463 from demeritcowboy/profile-null

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

Ensure useful exception thrown for all Smarty errors from user strings

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)

Merge pull request #25435 from totten/5.58-upgrade-msg

Upgrade - Simplify call to action. Use URL redirect.

Merge pull request #25437 from totten/5.58-status-constraint

Status Checks - Add alert for bad foreign key

Merge pull request #25410 from larssandergreen/Fix-missing-default-values-for-required-fields-in-event-registration-setup

CiviEvent - Fix missing default values for required fields in event registration setup

Status Checks - Add alert for bad foreign key

FiveFiftySeven - Simplify call to action.

FiveFiftySeven - Point static link to adjustable redirect

Add required field to test data

Event Registration - Treat omission-by-unset and omission-by-blank-string as the same problem

Do not crash the whole SearchKit UI if one entity fails

[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

Fix slow queries in snapshot process

SELECT COUNT(*) is slow to run on a large database but SELECT MAX(id) is
quick. The snapshot check only needs to be very rough - so MAX(id) should
be accurate enough.

This does make the message a little inaccurate - but my money says no-one
will notice & it's not worth creating work for the translators to change

Increase timeout on imap

About once a day we get an error because the imap server takes longer than the 5
second timeout. This would be largely invisible to most sites as it
would cause a run of fetch_bounces to fail quietly & then the next
one would probably work. For us the impact is we get notified.

I think the impact of increasing to 15 seconds is small enough we
don't need to make it configurable and I doubt that we hit this
timeout any more than anyone else (our imap is within our
own network which is surely more reliably fast than gmail)
so the main impact is that people's fetch_bounces jobs are likely
to result in data getting into civi with less failed runs & hence
a bit quicker

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…