Merge pull request #22036 from eileenmcnaughton/notice

Escape-on-output-proof enotice fixes

Merge pull request #22034 from seamuslee001/dashboard_followup_fix

[NFC] Additional Fix on UserDashBoard Tests to assign action value to…

Merge pull request #22033 from eileenmcnaughton/quick

Do not smarty encode quicksearch html

Escape-on-output-proof enotice fixes

Merge pull request #22019 from colemanw/searchKitLoading

SearchKit - Animated loading skeleton

[NFC] Additional Fix on UserDashBoard Tests to assign action value to dashboard

Merge pull request #22025 from demeritcowboy/aggregate-smarty2

dev/report#77 - Smarty warning about relationship in contribute/history civireport

Merge pull request #22024 from demeritcowboy/aggregate-weird

[NFC] dev/report#77 - Remove useless comment in contribute/history civireport

Do not smarty encode quicksearch html

Merge pull request #22029 from eileenmcnaughton/action_set

Ensure action is set when loading contactFooter

Merge pull request #21986 from colemanw/searchKitTabs

SearchKit - tabbed display for custom vs packaged searches

Merge pull request #22028 from eileenmcnaughton/issetinfo

Remove use of isset for info.tpl

Merge pull request #22027 from braders/fix-graph-empty-logic

Flip logic for determining if $section variable is set.

Ensure action is set when loading contactFooter

Remove use of isset for info.tpl

Merge pull request #22023 from demeritcowboy/php8-userdashboard

[NFC] php8 - fix test fails for CRM_Contact_Page_View_UserDashBoardTest

Merge pull request #22026 from mattwire/lockmessages

REF: Switch to debug level logging for lock debugging

Enotice fix for footer.tpl

Ensure that contactId is always present

Flip logic for determining if $section variable is set.

Previously empty() function calls were added in order to provide PHP8 compatiability,
however, the logic commited was back-to-front.

This broke the ability to display pie and bar charts for those reports which support them.

Merge pull request #22017 from lcdservices/dev-core-1529

dev/core#1529 fix event repeat functionality

Switch to debug level logging for lock debugging

smarty warning

remove useless comment

assign formTpl

Merge pull request #22000 from eileenmcnaughton/body

Ensure beginHookFormElements is assigned

Merge pull request #22021 from totten/master-ischildpath

CRM_Utils_File::isChildPath - Fix checking of non-existent paths

Merge pull request #22020 from seamuslee001/nfc_payment_test_fix

[NFC] Fix test issue about trying to do array offset on NULL in Payme…

Merge pull request #22015 from demeritcowboy/test-histreport

[NFC] dev/report#77 - contribute/history civireport seems ok to run during reporttemplatetest

CRM_Utils_File::isChildPath - Fix checking of non-existent paths

Suppose you use call `isChildPath()` with these options:

* `$parent='/nonexistent'`
* `$child='/home/me/exists.txt'`
* `$checkRealPath=TRUE`

Before
=====

Reports that `/home/me/exists.txt` is a child of `/nonexistent`

After
=====

Reports that `/home/me/exists.txt` is NOT a child of `/nonexistent`

Comments
========

I added six test cases for how `isChildPath()` behaves when using
`$checkRealPath=TRUE`.  The first five cases behave the same with or without
the patch.  The patch fixes the last case.

[NFC] Fix test issue about trying to do array offset on NULL in Payment ProcessorFormTest

report seems fine now

Merge pull request #22018 from demeritcowboy/php8-formtpl

Ensure smarty variable formTpl is set

Merge pull request #22001 from eileenmcnaughton/hook

Ensure hookContentPlacement is assigned

Merge pull request #22016 from demeritcowboy/histreport-entryfound

dev/report#77 - Unused variable in contribute/history civireport

SearchKit - Animated loading skeleton

always set formTpl

dev/core#1529 fix event repeat functionality

unused variable

Merge pull request #21991 from eileenmcnaughton/isset4

Ensure that keys are set in tabValues before hitting smarty

Merge pull request #22002 from eileenmcnaughton/group_org

Extract getGroupOrganizationUrl & ensure always assigned

Merge pull request #22008 from artfulrobot/artfulrobot-fix-mailing-click-report-url

Fix unencoded mailing url query param in Mailing report

Merge pull request #22011 from jmcclelland/surveyprevnext

don't overfill the prevnext cache when reserving for a survey.

Merge pull request #21983 from lisandro-compucorp/email_to_organizations

Handle activity creation for emails directed to organization contacts

Merge pull request #22010 from colemanw/apiExplorerFormat

APIv4 Explorer - Optionally view results in PHP format

Merge pull request #22005 from seamuslee001/update_province_philippines

[REF] Add in missing province for Philippines and update names to matc…

[REF] Add in missing province for Philippines and update names to match ISO

Ensure we don't overwrite any customisations already in the database

APIv4 Explorer - Optionally view results in PHP format

don't overfill the prevnext cache when reserving for a survey.

Fix unencoded mailing url query param in Mailing report

Handle activity creation for emails to organization contacts

Merge pull request #22003 from civicrm/5.44

5.44

Merge pull request #21960 from colemanw/afformBaseModule

Afform - Display the extension in which an Afform is packaged

Merge pull request #21997 from colemanw/fixUpgradeErrors

dev/core#2550 Fix possible upgrade failures

Extract getGroupOrganizationUrl & ensure always assigned

Ensure that keys are set in tabValues before hitting smarty

Ensure hookContentPlacement is assigned

Merge pull request #20811 from elcapo/billing-fields

Check the billing fields returned by each payment processor

Ensure beginHookFormElements is assigned

This ensures the variable is assigned & hence does not enotice with escape on output

Merge pull request #21995 from braders/title-attr-tidyup

Tidy up title attributes

Tidy up title attributes
Wrap text in {ts} as appropriate (for translation)
Remove poorly worded or superflous title attributes

Merge pull request #21998 from webmaster-cses-org-uk/patch-3

Guard against passing empty values to CRM_Core_DAO::getFieldValue()

Merge pull request #21975 from MegaphoneJon/reporting-18

Reporting #18: Fix sorting with > 10 participant roles

Merge pull request #21940 from eileenmcnaughton/debug

Enotice fixes in debug.tpl that are compatible with default escaping

Merge pull request #21979 from eileenmcnaughton/vars

Fix isForm to be always assigned, remove checks

Guard against passing empty values to CRM_Core_DAO::getFieldValue()

This change removes a potential "getFieldValue failed" error that can be thrown if an empty scalar value occurs in a multiple-contact reference custom field.

See https://lab.civicrm.org/dev/core/-/issues/2939

Replaces https://github.com/eileenmcnaughton/civicrm-core/pull/13/commits which was opened against the wrong repo.

Merge pull request #21994 from braders/more-hardcoded-strings

Make more hardcoded strings available for translation

Fix possible upgrade failures when writing to db fields which may not yet exist

Fixes dev/core#2550

Merge pull request #21933 from eileenmcnaughton/local

Remove unused variable

Make more hardcoded strings available for translation

Merge pull request #21988 from eileenmcnaughton/isset2

Remove isset from Datasource.tpl

Merge pull request #21987 from eileenmcnaughton/enotice

Enotice fixes

Merge pull request #21956 from eileenmcnaughton/escape2

dev/core#1328 Add define to optionally enable escape-on-output

Merge pull request #21990 from civicrm/5.44

5.44 to master

Merge pull request #21985 from colemanw/searchImageFix

[Unreleased regression] SearchKit - Fix display of image fields

Remove isset from Datasource.tpl

This comes up on the first import contact screen

Enotice fixes

Add define to optionally enable escape-on-output

This adjusts the smarty class such that if you have the define below
escape-on-output is enabled

This PR + fixing the use of isset in the template layer are the pre-requisites for
turning escape on output on and off. Keeping this part in an unmerged branch is
tricky as the templates need to be recompiled when you switch to a branch
which does not have this function

I envisage us starting to use this define in our dev environments fairly soon
as it's working well locally for me on
https://github.com/civicrm/civicrm-core/pull/21935

AfformAdmin - Show counts in tab header

SearchKit - tabbed display for custom vs packaged searches

Splits the SearchKit admin UI in to 2 tabs - one for custom (locally-defined) searches,
and one for packaged searches. Adds a "revert" button for packaged searches.

SearchKit - Fix display of image fields

Merge pull request #21835 from wmortada/patch-1

Run scheduled reminder job hourly

Merge pull request #21982 from civicrm/5.44

5.44

Merge pull request #21980 from eileenmcnaughton/544

dev/core#2942 regression fix

Merge pull request #21977 from colemanw/fixSearchKitPermissions

SearchKit - Fix permission to access default display and download spreadsheets

dev/core#2942 regression fix

Fix isForm to be always assigned, remove checks

Note I'm not seeing any regressions on this locally with enotice turned on - but the
worst case, if tests pass, is we get some enotices back in a scenario where we
are only impacting bleeding edge devs

Enotice fixes in debug.tpl that are compatible with default escaping

Reporting #18: Fix sorting with > 10 participant roles

Merge pull request #21912 from eileenmcnaughton/no_display

Move responsibilty for ensuring greeting parameters are set to `ensureGreetingParamsAreSet`

Merge pull request #21976 from civicrm/5.44

5.44

SearchKit - Fix permission to access default display and download spreadsheets

Intentionally setting these gatekeeper permissions fairly open as all of these actions
perform stricter permission checks internally.

Merge pull request #21974 from christianwach/lab-core-2909-2

dev/core#2909 - Repair the option value label for nb_NO language

Repair the option value label for nb_NO language

Set version to 5.45.alpha1

Set version to 5.44.beta1

Merge pull request #21972 from eileenmcnaughton/gettok

Remove call to getTokenDetails

Merge pull request #21967 from colemanw/searchRunFix

SearchKit - Fix incorrect pager count when using filters

Merge pull request #21947 from colemanw/cssRules

SearchKit - Conditional style rules for rows/cells

Merge pull request #21957 from eileenmcnaughton/lang

Use null variables rather than isset in Core_Block

transitionParticipants() - Improve $contactDetails cache maintenance

This revises a very recent commit that touches `static $contactDetails` cache.

Imagine you have two calls to `transitionParticipants()` which involve different-but-overlapping sets:

* First invocation involves Alice and Bob
* Second invocation involves Bob and Carol

The first invocation loads Alice and Bob into `$contactDetails`.  The second
invocation identifies Carol as missing. We need to load Carol. Here's the change:

* Before: Carol overwrites Bob in `$contactDetails`. Bob goes missing.
* After: Alice, Bob, and Carol all exist in `$contactDetails`.