Merge pull request #23947 from totten/master-php81-global

dev/core#3181 - Resolver - Compatibility fix for GLOBALS in PHP 8.1

Merge pull request #23945 from eileenmcnaughton/map

Enotice fix

Merge pull request #23346 from mattwire/totaltaxamount

Fix calculation and assignment of taxAmount on contribution page confirmation

Merge pull request #23951 from civicrm/5.51

5.51 to master

Merge pull request #23949 from alifrumin/rn5.51

[NFC] 5.51 Release Notes - first pass

[NFC] 5.51 release notes

dev/core#3181 - Resolver - Compatibility fix for GLOBALS in PHP 8.1

CRM_Utils_Array - All `pathGet()`-like methods will accept empty-path

These two are equivalent:

  (A) $data['sub1']['sub2']['sub3']
  (B) CRM_Utils_Array::pathGet($data, ['sub1', 'sub2', 'sub3'])

But what if the list of subpaths is empty -- ie you're accessing the top-level item?

  (A) $data
  (B) CRM_Utils_Array::pathGet($data, [])

It worked correctly on `pathGet()`, but accessing the top-level using a
related methods (eg `pathSet()`) would generate a PHP error.

This commit ensures that related methods can be used to check/update the
top-level item (ie path===[]).

Note that there is some decent test-coverage already for the regular
scenarios (eg `testGetSetPathParts()`).

Enotice fix

Merge pull request #23942 from tschuettler/3717-sort-mapping-page

dev/core#3717 - Fix sorting by mapping type

Merge pull request #23931 from eileenmcnaughton/repeat_pledge

RepeatTransaction - separate out repeat pledge handling from completeOrder

Merge pull request #23943 from civicrm/5.51

5.51

Merge pull request #23927 from eileenmcnaughton/url

dev/core#3707 Fix wordpress issue of page not refreshing after import

Merge pull request #23935 from seamuslee001/fix_more_null

[REF][PHP8.1] Another set of fixes for passing in NULL values to php …

Merge pull request #23928 from artfulrobot/artfulrobot-cleanup-repeattransaction

Cleanup repeattransaction

Minor tidyup of api3 completetransaction; plus comments

Add docblock comments to explain $input param in CRM_Contribute_BAO::repeatTransaction

Fix sorting by mapping type

Merge pull request #23939 from civicrm/5.51

5.51

Merge pull request #23938 from seamuslee001/brick_math_update

[REF][PHP8.1] Update Brick/Math to a version that works with php8.1

Merge pull request #23904 from eileenmcnaughton/grant

dev/core#3503, dev/core#3492 Fix grant exports and relative dates

[REF][PHP8.1] Update Brick/Math to a version that works with php8.1

[REF][PHP8.1] Another set of fixes for passing in NULL values to php string or preg replace functions

Update to Dave's suggestion for default value

Fix broken tests highlighted by but not caused by pull 23928

Merge pull request #23913 from seamuslee001/symfony_upgrade

[REF][PHP8.1] Upgrade Symfony to 4.x for WP, Drupal7, Backdrop, Joomla

Merge pull request #23912 from mlutfy/reportVars

CRM_Report_Form: set _from and _where as public for the alterReportVar hook

Merge pull request #23919 from colemanw/afformIcon

Afform - Allow picking icon for tab, add CrmUiIconPicker widget

Merge pull request #23922 from seamuslee001/str_func_null_1

[REF][PHP8.1] Stop Passing NULL values into mb_strlen in DAO nad strp…

[REF][PHP8.1] Stop Passing NULL values into mb_strlen in DAO nad strpos in API validate string function

Merge pull request #23808 from agileware/CIVICRM-1861

issues/2887 Enhancements to improve the iCal, Outlook and Google Calendar integration

RepeatTransaction - separate out repeat pledge handling from complete

Merge pull request #23930 from demeritcowboy/trigger-comment

[NFC] Add code comment about triggerinfo "variables"

add comment

Merge pull request #23926 from seamuslee001/fix_default_value_php81

[REF][PHP8.1] Fix default values which triggeres deprecation notices …

[REF][PHP8.1] Fix default values which triggeres deprecation notices in php8.1

Switch to non array::value format

[REF][PHP8.1] Upgrade Symfony to 4.x for WP, Drupal7, Backdrop, Joomla

Merge pull request #23923 from seamuslee001/upgrade_mail_mime

[REF][PHP8.1] Upgrade Pear/mail_mime package to support php8.1

Initial go at cleaning up repeattransaction

dev/core#3707 Fix wordpress issue of page not refreshing after import

[REF][PHP8.1] Upgrade Pear/mail_mime package to support php8.1

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

Afform - Allow picking icon

Allows afforms placed as a contact summary tab to specify an icon.
It also switches Afforms's hook_civicrm_tabset to use the API, which ensures virtual forms get picked up.
Adds the new icon for CiviGrant as an example.

CrmUi - Add crmUiIconPicker Angular widget

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.

CIVICRM-1861 Embed timezone output for iCal
    CIVICRM-1988 Add HTML alternate description to iCAL
    CIVICRM-1987 Add to Google Calendar Link

[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

CRM_Report_Form: set _from and _where as public for the alterReportVar hook

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

dev/core#3503, dev/core#3492 Fix grant exports and relative dates

This both stem from the wrong fix for https://lab.civicrm.org/dev/core/-/issues/3069

Refer to the gitlab for some famous last words....

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