misc

CRM

Civi

api4

Merge pull request #22069 from eileenmcnaughton/context

Do not check is context is set, should always be

Merge pull request #22067 from eileenmcnaughton/format

[NFC] Array formatting

[NFC] Array formatting

Do not check is context is set, should always be

Merge pull request #22030 from eileenmcnaughton/isset_summary

Fix all the issets that affect ability to load contact summary page

Merge pull request #22066 from demeritcowboy/gender-display

Fix typo in smarty template "gender_display"

Merge pull request #22045 from seamuslee001/dev_mail_103

[NFC] dev/mail#103 Add in unit test to demonstate issue with API…

fix typo

Merge pull request #22065 from seamuslee001/master

[NFC] ENsure that formTpl is assigned in CRM_Custom_Form_OptionTest

[NFC] dev/mail#103 Add in unit test to demonstate issue with APIv4 and joins

Add comment to test along the lines from Coleman and fix test cleanup

Update test to extend from BaseCustomValueTest

[NFC] ENsure that formTpl is assigned in CRM_Custom_Form_OptionTest

Merge pull request #22061 from demeritcowboy/formtpl2

[NFC] php8 assign formTpl in HookTest test

Do not check if contact_type_label isset - it always is

Ensure contact values are always assigned to the template

Merge pull request #22056 from eileenmcnaughton/context

Ensure context is always set

Merge pull request #21909 from eileenmcnaughton/greeting_sanity

[REF] Preliminary cleanup in update greeting

Merge pull request #22055 from eileenmcnaughton/tagg

Ensure contactTag is always set

Merge pull request #22057 from civicrm/5.44

5.44

assign formTpl

Merge pull request #22059 from demeritcowboy/smarty-static-problem

php8 - fix for breadcrumb fails during tests

temp fix for static smarty during tests

Merge pull request #22051 from eileenmcnaughton/544

APIv4 - Add backticks around fields in join criteria

Merge pull request #22044 from eileenmcnaughton/dep

Mark a few more token functions as deprected

Merge pull request #22046 from totten/5.44-eventid

(dev/core#2947) Tokens - Auto-enable `{event.*}` if `participantId` is present

Ensure context is always set

Ensure contactTag is always set

Merge pull request #22047 from eileenmcnaughton/isset_gender

Ensure gender_display is always set

Merge pull request #22053 from civicrm/5.44

5.44

Merge pull request #22043 from seamuslee001/5.44

[REF] Skip testCutesyTokenData on php8 for the moment

Merge pull request #22048 from demeritcowboy/session-status

[NFC] php8 - set smarty null defaults for CRM_Core_SessionTest

APIv4 - Add backticks around fields in join criteria

Fixes dev/mail#103

Merge pull request #21989 from colemanw/managedEntityDelete

ManagedEntities - Track modification and auto-update

Refine alignment of old/new token lists

The prior commit means that `$context['participantId']` will activate `{event.*}` tokens - in contexts
that use `TokenProcessor`. And the `TokenProcessor` is *somewhat* matched to the older functions.

This twiddles the match-up:

* Provide continuity for the deprecated `CRM_Core_SelectValues::participantTokens()` - so it still
  only returns `{participant.*}` tokens. Borrow the technique which excludes `{domain.*}` tokens.
  This should ensure better continuity for any callers that actually use `CRM_Utils_Token`.

* Update the assertions for the `TokenProcessor` to show that the `{event.*}` tokens are presented in
  participant-related contexts.

dev/core#2947 - Auto-enable `{event.*}` if `participantId` is present

[REF] Fix Pledge Test failing on php8 by ensuring all payments have a status assigned and that customGroup is always assigned to the template and updating test to pass in some additional form values

php8 - set null defaults

Ensure gender_display is always set

Always add it, rather than checking if it is assiged in the template

[REF] Preliminary cleanup in update greeting

Merge pull request #22042 from eileenmcnaughton/isset2

Ensure userRecordUrl is always set- fixes enotice with escape-on-output

[REF] Skip testCutesyTokenData on php8 for the moment

Mark a few more token functions as deprected

These are only used by the legacy mailing code or, in some cases, used by unsubscribe where it should use
the token processor

Merge pull request #22038 from yashodha/report

(dev/core#2951) expose contribution custom fields in lybunt report

Merge pull request #22041 from demeritcowboy/aggregate-report

dev/report#77 - Make contribution filters work for contribute/history civireport

Ensure userRecordUrl is always set

Merge pull request #22040 from colemanw/expectedKeys

Fix tabs getting overwritten by double foreach loop

Merge pull request #21936 from JMAConsulting/issue_2912

dev/core#2912 : Can't save extra custom fields when inline and required

Merge pull request #22031 from eileenmcnaughton/image

Ensure imageUrl is always set

make contribution filters work

Fix tabs getting overwritten by double foreach loop

Merge pull request #22037 from seamuslee001/ref_pledge_test_php8

[REF] Fix Pledge Test failing on php8 by ensuring all payments have a…

expose contribution custom fields in lybunt report

[REF] Fix Pledge Test failing on php8 by ensuring all payments have a status assigned and that customGroup is always assigned to the template and updating test to pass in some additional form values

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

Ensure imageUrl is always set

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

ManagedEntity - Add update mode 'unmodified' and fix cleanup mode 'unused' for APIv4

Update mode 'unmodified' will only update a record if it has not been locally edited.
This new setting works only for entities opted-in to the APIv4 ManagedEntity trait, and will
emit a warning and fall back on 'always' for others.

Cleanup mode 'unmodified' now works for APIv4 managed entities, and they are cleaned up
in reverse order to ensure references are deleted before their parents.

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

APIv4 - Add getRefCount utility & include afforms in SavedSearch refCounts.

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