REF - Remove unneeded return from private fn, add comments

Merge pull request #23937 from demeritcowboy/case-activity-search-filter

dev/core#3709 - Make the activity search filter on manage case less unwieldy

Merge pull request #23962 from eileenmcnaughton/pledge

Remove another instance of pledges using contribution option group

Merge pull request #23963 from eileenmcnaughton/tpl

Smarty notice fix

Merge pull request #23924 from colemanw/afformPopup

Angular - Support popping up an afform or other ang module via ajax modal

Smarty notice fix

Merge pull request #23961 from totten/master-mgdphp-caseacttype

(NFC) mgd-php@1 - Add example+assertions for new case-type

Merge pull request #23960 from seamuslee001/fix_static_calling_trait

[REF][PHP8.1] Fix deprecations where by calling trait static function…

Merge pull request #23959 from totten/master-casexml-acttype

(NFC) case-xml@1 - Add example+assertions of new activity-type

Remove another instance of pledges using contribution option group

Merge pull request #23084 from pradpnayak/paypalipnchangemaount

Subscription Amount change not respected in Paypal

[REF][PHP8.1] Fix deprecations where by calling trait static functions when not from a class that implements the trait is deprecated

Merge pull request #23958 from civicrm/5.51

5.51

mgd-php@1 - Add example+assertions for new case-type (but without a real `definition`)

case-xml@1 - Add example+assertions of new activity-type

Merge pull request #23956 from agh1/5.51.0-releasenotes-final

5.51.0 release notes final changes

Merge pull request #23933 from eileenmcnaughton/require

Import fix - if there are no required fields validateRequiredFields should 'pass'

Merge pull request #23934 from eileenmcnaughton/url_tokens

dev/core#3713 Use front-end urls for event url tokens

Merge pull request #23953 from eileenmcnaughton/imp_job

dev/core#3719 fix inconistent handling of job_type:label

Afform - Close popup dialog after submitting

Afform - Use urlHash embedded in popup dialog

crmUi - Fix crmPageTitle directive to work inside a modal

Angular - Support popping up an afform or other ang module via ajax modal

Fixes the AngularLoader to work with modal dialogs via ajax.
Dedupes the already-loaded modules to only load what's needed.

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

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

5.51.0 release notes: added late changes

Add release-notes/5.50.4.md

Add release-notes/5.50.3.md

Fix typo in version of Guzzle

Add release-notes/5.50.2.md

Add release-notes/5.50.1.md

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

dev/core#3719 fix inconistent handling of job_type:label

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

Make the activity search filter on manage case less unwieldy

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

dev/core#3713 Use front-end urls for event url tokens

Import fix - if there are no required fields validateRequiredFields should 'pass'

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