Merge pull request #24064 from eileenmcnaughton/dep_err

Log a bit more deprecation info

Update CRM/Core/Error.php

Co-authored-by: demeritcowboy <demeritcowboy@hotmail.com>

Update CRM/Core/Error.php

Co-authored-by: demeritcowboy <demeritcowboy@hotmail.com>

Merge pull request #24032 from seamuslee001/remove_dependency_strftime

[REF][PHP8.1][INTL] Remove dependency on strftime function by using t…

Merge pull request #24065 from eileenmcnaughton/pdf_inv

Avoid sending blank invoices

Merge pull request #24050 from eileenmcnaughton/pledge

dev/financial#201 Fix PaypalPro IPN to refer to correct option group

Update to use INTL Date formatter instead

[REF][PHP8.1][INTL] Remove dependency on strftime function by using ts to generate day full names and abbreviations and month abbreviations

Fix array order and add unit test to lock in fix

Merge pull request #24068 from totten/master-552

Merge forward 5.52-rc => master

(dev/core#3490) Upgrader - 5.52's #24040 is obsoleted by 5.53's #24030

5.53's #24030 is the longer term fix for dev/core#3490.

5.52's #24040 was a short term mitigation and is not needed going forward.

Merge remote-tracking branch 'origin/5.52' into master-552

Avoid sending blank invoices

Alternative to https://github.com/civicrm/civicrm-core/pull/24029

Log a bit more deprecation info

Merge pull request #24053 from agh1/5.52.0-releasenotes-initial

5.52.0 release notes initial run

Merge pull request #24057 from alexymik/patch-4

Add whitespace to exception message

Add whitespace to exception message

"Entity not loadedGrant" -> "Entity not loaded: Grant"

Merge pull request #24055 from eileenmcnaughton/nfc

(NFC) HookStyleListener - Comments

Merge pull request #24052 from colemanw/fixChainDollarSignBug

APIv4 - Allow values to start with a dollar sign in chaining

(NFC) HookStyleListener - Comments

Merge pull request #24047 from kurund/export-fixes

dev/core#3751 - export error fixes for multi-select contact reference custom field

5.52.0 release notes: added boilerplate

5.51.0 release notes: fix a couple small things

5.52.0 release notes: raw from script

APIv4 - Allow values to start with a dollar sign in chaining

Fixes dev/core#3754

Fix PaypalPro IPN to refer to correct option group

Merge pull request #24036 from colemanw/v3Custom

API - Consistently save custom data for v3 & v4

Merge pull request #24048 from yashodha/reports_improvements

remove start/end date as mandatory in Demographics report

remove start/end date as mandatory in Demographics report

export error fixes for multi-select contact reference custom field

Merge pull request #24039 from totten/5.52-snap

Upgrader - Add snapshots for 5.52's data modifications

Merge pull request #24040 from totten/5.52-postup

dev/core#3490 - Upgrader - Show additional notice about running extension upgrades

Merge pull request #24010 from eileenmcnaughton/task_notice

dev/core#3737 ensure tasks are loaded (notice fix)

Merge pull request #24046 from seamuslee001/league_csv_php81

[REF][PHP8.1] Apply patches from upstream to ensure league/csv can work on php8.1

[REF][PHP8.1] Apply patches from upstream to ensure league/csv can work on php8.1

Add in patch for league CSV to fix fputcsv in php8.1

Merge pull request #24045 from seamuslee001/package_update_2_php81

[REF][PHP8.1] Update phpoffice/phpword and tecnickcom/tcpdf  packages…

[REF][PHP8.1] Update phpoffice/phpword and tecnickcom/tcpdf  packages for php8.1 compatability

Merge pull request #23875 from alexymik/patch-3

Set target="_blank" on external footer links

Set target="_blank" on external footer links

Merge pull request #24043 from totten/master-nfc-perm

(NFC) CRM_Core_Permission - More docblocks

Merge pull request #24044 from totten/master-scss

composer.json - Update scssphp and related libraries (for PHP 8.1)

composer.json - Update scssphp and related libraries (for PHP 8.1)

Overview
--------

Update scssphp/scssphp, civicrm/composer-compile-plugin, and civicrm/composer-compile-lib.

Before
-------

Large number of warnings are generated in scssphp / php81, leading to an overflow

```
Compiling additional files (For full details, use verbose "-v" mode.)
Compile: Generate CCL wrapper functions
Compile: Greenwich CSS (dist/bootstrap3.css)

Fatal error: Allowed memory size of 1610612736 bytes exhausted (tried to allocate 36864 bytes) in phar:///Users/totten/bknix/bin/composer/vendor/symfony/process/Pipes/UnixPipes.php on line 132

Check https://getcomposer.org/doc/articles/troubleshooting.md#memory-limit-errors for more info on how to handle out of memory errors.Subcommand @composer compile  returned with error code 255

In ComposerPassthru.php line 72:

  Subcommand @composer compile  returned with error code 255

install [--prefer-source] [--prefer-dist] [--prefer-install PREFER-INSTALL] [--dry-run] [--dev] [--no-suggest] [--no-dev] [--no-autoloader] [--no-progress] [--no-install] [--audit] [--audit-format AUDIT-FORMAT] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--apcu-autoloader-prefix APCU-AUTOLOADER-PREFIX] [--ignore-platform-req IGNORE-PLATFORM-REQ] [--ignore-platform-reqs] [--] [<packages>...]
```

After
-----

Compilation works

Comments
--------

Generated via: `composer update civicrm/composer-compile-lib scssphp/scssphp civicrm/composer-compile-plugin`

(NFC) CRM_Core_Permission - More docblocks

Merge pull request #24042 from seamuslee001/master

5.52

Merge branch '5.52'

Merge pull request #24038 from seamuslee001/5.52

[REF] Upgrade DOMPDF to 2.x branch

Merge pull request #23896 from colemanw/afformManaged

Afform - Switch to APIv4 for managed afform_type option group

Upgrader - Show additional notice about running extension upgrades

Merge pull request #24030 from totten/master-run-ext-upgrades

dev/core#3490 - Apply extension updates after core updates

Merge pull request #24016 from colemanw/afformTokens

Afform - Post-submit redirect tokens

FiveFiftyTwo - Add snapshots for modified data structures

whitespace fix

Fix installed-fonts cache

[REF] Upgrade DOMPDF to 2.x branch

Merge pull request #23970 from MegaphoneJon/false-not-zero

APIv4 - Fix mishandling of boolean custom values

Merge pull request #23895 from colemanw/searchKitManaged

SearchKit - Switch to APIv4 for managed entities

Afform - Switch to APIv4 for managed afform_type option group

Merge pull request #24035 from colemanw/actionLogFix

Fix ActionLog BAO to not overwrite existing data

APIv3 - Don't handle custom data already handled by BAO

Fixes dev/core#3747
It's confusing to handle saving custom data in the APIv3 layer,
because APIv4 expects the BAO to handle it.

Until every BAO consistently uses WriteRecord this will continue to be a pain point.

Fix ActionLog BAO to not overwrite existing data

Merge pull request #24027 from totten/master552-status-footer-skipsort

Status Check - Report the overall status (accurately)

Merge pull request #24031 from yashodha/reports_improvements

Fix 'Pledge Made Date' display to show date and not time

Merge pull request #24025 from demeritcowboy/batch-remove-2

dev/core#3050 - (Alternate) Fix crash removing an entry from a batch

fix 'Pledge Made Date' to show date and not time

Merge pull request #24018 from seamuslee001/php8_1_null_batch_4

[REF][PHP8.1] Batch 4 of fixing issues found in unit tests where pass…

[REF][PHP8.1] Batch 4 of fixing issues found in unit tests where passing NULL values triggers deprecations

Apply fixes as per Demeritcowboy

Merge pull request #23767 from colemanw/afformRequireAuthx

Afform - Require Authx

Update CRM/Utils/Check.php

Co-authored-by: demeritcowboy <demeritcowboy@hotmail.com>

Upgrader - Enqueue extension upgrade tasks

Upgrader - Re-assess dispatch policy at the start of every upgrade task

The practical effect should be this:

* All tasks before `doCoreFinish()` run with the `upgrade.main` policy
* All tasks after `doCoreFinish()` run with the `upgrade.finish` policy

Framing this change is a bit delicate because tasks can run in different
ways (eg AJAX queue runner; headless queue-runner; cv queue-runner).  This
should have the same consequence in all runners.

Merge pull request #24028 from yashodha/reports_improvements

remove pledge status as mandatory in Pledge report

Merge pull request #24023 from colemanw/searchKitHtml

SearchKit - Allow html in columns

remove pledge status as mandatory in Pledge report

Upgrader - Pick initial dispatch policy based on how far we've progressed in upgrade

(REF) Extract method \CRM_Upgrade_DispatchPolicy::useTemporarily()

(REF) Extensions - Extract method to build upgrade queue

I was slightly concerned that building a queue under a different there are
two risks, although I don't think either is very big:

1.  That someone references the queue by name during the building process.
    However, it wouldn't make much sense from a DX POV (given how "Upgrader"
    classes are generally written), and I can't any references to
    `QUEUE_NAME` or `ext-upgrade` in `universe`

2.  That someone generates queue tasks with `weight`s that somehow clash with
    the `weight`s in core-queue. However, this also doesn't make much sense
    from a DX POV (given how "upgrader" classes are generally written).

(REF) CiviEventDispacher - Enable getDispatchPolicy()

CRM_Queue_Task - Fire internal events

Status Check - Report the overall status (accurately)

Overview
--------

The page-footer includes a summary message about the system status (eg "System
Status: Ok" or "System Status: Error").  The footer has inaccurately summarized
the status.

Before
------

Suppose you have 5 "Error" messages and 1 "OK" message. It will summarize as "System Status: Ok".

(It emphasizes the *least severe* status-message.)

Code is more complex.

After
------

Suppose you have 5 "Error" messages and 1 "OK" message. It will summarize as "System Status: Error".

(It emphasizes the *most severe* status-message.)

Code is less complex.

Comments
--------

The overall value is determined in `CRM_Utils_Check::checkAll()`.  Heretofore,
`checkAll()` sorted the messages by severity and then emphasized the *first*
visible message.

The problem is that the sort-order has flip-flopped, so "first" means different things:

* In v4.7, the messages were descending. So "first" was "most severe".
* In v5.39 (11d593edde82e7de962ec9056cfebe87975fb499), the order was ascending.
  So "first" was "least severe". (It appears accidental.)
* In v5.45 (728c1cc31d0d323f74f7b980a1e183fb499d4e9e), the order flipped back
  to descending, but *only* for the Angular (`civicrm/a/#/status`) -- not the footer.

IMHO, the flip-floppiness means that callers cannot rely on the order returned
by `checkAll()` -- they should instead do a higher-level sort (as in v5.45's
728c1cc31d0d323f74f7b980a1e183fb499d4e9e).  The `uasort()` within `checkAll()`
is therefore redundant.  We might as well skip it -- and simplify the logic.

Merge pull request #24024 from MegaphoneJon/print-summary-afform

Add afform blocks to print summary

fix crash removing an entry from a batch

add afform blocks to print summary

SearchKit - Allow html in columns

This adds a column type 'html' and defaults rich-text fields to use that column type.
Fixes dev/core#3723

Merge pull request #24021 from mattwire/activitypageajaxicons

Use fontawesome icon helper for activity/Page/AJAX

Use fontawesome icon helper for activity/Page/AJAX

Merge pull request #24013 from colemanw/afformNav

Afform - provide easy way to add navigation menu from the form

Merge pull request #24017 from demeritcowboy/polyfill-too-high

Lower minimum polyfill version since conflicts with drupal/core-recommended

Merge pull request #24015 from demeritcowboy/composer-patches

Update cweagans/composer-patches to 1.7.2 to fix delete-and-reinstall problem

lower minimum polyfill version

Afform - Post-submit redirect tokens

update to fix delete-and-reinstall problem

Merge pull request #24011 from demeritcowboy/composerlock

composer.lock hash out of date

Merge pull request #24012 from yashodha/add_Classes

add classes to the contribution view page

Afform - Insta-refresh menubar when adding a navigation item

APIv4 - Flush navigation caches after writing or deleting navigation item

Historically the expectation was that this cache gets flushed manually.
I've kept that unchanged for legacy code (v3 create/delete and direct BAO::add)
but with APIv4 and onward the cache will get flushed automatically after a write operation.

Afform - provide easy way to add navigation menu from the form

add classed to tpl

hash out of date

dev/core#3737 ensure tasks are loaded

Merge pull request #23954 from eileenmcnaughton/poly

Poly-fill-me-up