Remove-single-value-loop

The communicationType array only has one value. This removes the foreach, swapping
the values for the only value they could be. This resulted in 2 always-true
ifs being replaced too

[REF] Code cleanup on location entities for the Contact Summary screen

This builds on https://github.com/civicrm/civicrm-core/pull/22966 to improve loading of location entities.

It addresses 2 problems
1) the code is really confusing - handling the loading in 2 places
2) not all keys are loaded - resulting in enotices at the tpl layer.

Note that address still seems kinda tricky so I haven't worked through
that in this PR.

Merge pull request #23072 from eileenmcnaughton/isset

Remove more isset from offline receipt

Remove more isset from offline receipt

Merge pull request #23064 from eileenmcnaughton/isset

Remove isset from event registration receipt

Merge pull request #23066 from colemanw/relationshipCacheJoins

SearchKit - Fix joins when RelationshipCache is base search entity

Merge pull request #23068 from seamuslee001/master

5.48

Merge branch '5.48'

SearchKit - Fix joins when RelationshipCache is base search entity

Merge pull request #23058 from eileenmcnaughton/ext_notice

Do not default-escape upgrade linkes + Enotice fix on Extension screen

Merge pull request #23063 from demeritcowboy/timezone-multilingual

dev/core#3151 - Don't localize event_tz field when backing up the field during upgrade

Remove isset from event registration receipt

Remove remaining empty checks on pay_later offline receipt

Merge pull request #23060 from eileenmcnaughton/isset

Remove issets from event_offline_receipt pay_later_text

don't localize event_tz field when backing up

Merge pull request #23062 from MegaphoneJon/fix-batch-url

[NFC] Fix URL documenting Batch entity

[NFC] Fix URL documenting use of Batch entity

Merge pull request #23006 from eileenmcnaughton/task

Fix notices on Task.tpl (grumpy mode)

Remove issets from event_offline_receipt pay_later_text

Merge pull request #23017 from eileenmcnaughton/second

Remove isset(taxAmount) from membership online receipt

More notice fixes

Escape the upgrade link

Enotice fix on Extension screen

Remove isset(taxAmount) from membership online receipt

Merge pull request #23054 from ginkgomzd/payflow-no-urlencode

do not URL Encode parameters for PayFlowPro

do not URL Encode AMT (amount) parameter for PayFlowPro

Merge pull request #22532 from seamuslee001/dev_core_3034

dev/core#3034 Ensure that filename contains the file extension for PDFs

Merge pull request #23050 from demeritcowboy/upmerge

5.48

Merge pull request #23053 from demeritcowboy/broken-contact

dev/core#3142 - Contact form and summary page completely broken

contact form/summarypage completely broken

regen

upmerge relevant files from 22940

Merge pull request #23045 from braders/chartenabled-var-expected

Add chartEnabled to expectedSmartyVariables for reports

Merge pull request #23046 from braders/pricefield-php-notices

Avoid PHP notices from smarty on pricefield table

Merge pull request #22940 from demeritcowboy/revert-timezone-upgrade

dev/core#2122 - Upgrade scripts for revert event timezones

Merge pull request #23047 from civicrm/5.48

5.48

Merge pull request #23035 from colemanw/mergeContactsRedirectFix

Dedupe - Fix redirecting to contact after merge

Merge pull request #23028 from civicrm/5.48

5.48

Avoid PHP notices from smarty on pricefield table

Add chartEnabled to expectedSmartyVariables for reports

Merge pull request #23044 from braders/https-maps-links

Link to Google Maps over HTTPs

Merge pull request #23043 from braders/addtotag-class-comment

Replace incorrect class comment

dev/core#2122 - TimezoneRevertTrait - Failsafe to opt-out of TZ converter

dev/core#2122 - TimezoneRevertTrait - Cleanup modified TZ-offsets and DST skews

Link to Google Maps over HTTPs

Replace incorrect class comment

Merge pull request #22966 from eileenmcnaughton/retrieve

[REF] Deprecate BAO_Contact::retrieve

Dedupe - Fix redirecting to contact after merge

The final redirect was accidentally changed by a6f2a80ffbddfc0788199ec824b8aff46ccd1542
Fixes dev/core#3135

Merge pull request #23030 from alifrumin/releasenotes5.48

[NFC] First Pass at 5.48 release notes

[NFC]  5.48 Release Notes - First Pass

Merge pull request #22689 from colemanw/removeCiviMailVar

CiviMail - Remove deprecated variable only used by Mosaico

Merge pull request #23027 from eileenmcnaughton/548

dev/core#3134 Fix regression on unselected lineitem

Merge pull request #23003 from colemanw/fixAllRowsSelected

SearchKit - Fix "all selected" checkbox when selecting a single page …

dev/core#2122 - TimezoneRevertTrait - Downgrade should use same TZ as prior upgrade

dev/core#3134 Fix regression on unselected lineitem

Merge pull request #23021 from yashodha/tpl

add classes to the membership userdashboard page

Merge pull request #22955 from colemanw/schemaTrait

Add utils for managing schema to extension upgrader base class

Merge pull request #23024 from colemanw/fixSubmitActions

Afform - fix missing submit actions

Afform - fix missing submit actions

This regressed due to 09a3e37058f393276662301038a6437eaaa346e4 specifying the wrong type.

Merge pull request #23023 from seamuslee001/master

5.48

Merge in 5.48

Merge pull request #23012 from colemanw/fixSearchAfformFilter

SearchKit - Fix multi-valued afform filters

add classes to the tpl

dev/core#2122 - TimezoneRevertTrait - Tweak messaging

dev/core#2122 - TimezoneRevertTrait - Split out messaging across bullets

dev/core#2122 - For 5.48.beta2, use TimezoneRevertTrait and updates msg tpls

dev/core#2122 - Add TimezoneRevertTrait (shared by 5.47/5.48)

Based on upgrade-steps from demerit. Adapted to trait by totten.

Set version to 5.48.beta2

Merge pull request #22930 from demeritcowboy/revert-event-timezone

dev/core#2122 - Revert event timezone

Merge pull request #23014 from eileenmcnaughton/second

Remove grumpy-mode-incompatible isset in payment notification

Adjust tests to not expect /bin/bash balance

Merge pull request #23005 from eileenmcnaughton/noex

noexpand seems to never be assigned

Merge pull request #23010 from colemanw/stableSearchKit

SearchKit - mark stable

Merge pull request #23011 from colemanw/fixAfCollapsible

Afform - Fix collapsible style

Merge pull request #22981 from braders/remove-assert

Remove bad use of PHP assert, removed in PHP 8.0

Merge pull request #23002 from christianwach/lab-core-3130-2

Keep "EntityFile" records until after Symfony "delete" hooks fire

Remove grumpy-mode-incompatible isset in payment notification

Merge pull request #23007 from eileenmcnaughton/is_admin

E-notice fix on pdfLetter

Merge pull request #23004 from eileenmcnaughton/mem

Remove issets on pay_later_receipt

Merge pull request #23009 from seamuslee001/dev_core_3132

dev/core#3132 Ensure that contacts are not placed in null island if g…

SearchKit - Fix multi-valued afform filters

Multi-select search filters stopped working as of 007167dfb90eaaeb0c76f2c9e5b0327f0b3e22e9

Afform - Fix collapsible style

SearchKit - mark stable

dev/core#3132 Ensure that contacts are not placed in null island if geocoders return 'null' string for geocodes

Merge pull request #23001 from agh1/5.48.0-releasenotes-initial

5.48.0 release notes initial run

Remove bad use of PHP assert, removed in PHP 8.0

E-notice fix on pdfLetter

Reachec by choosing task print/merge letter from search results.

This drives the help text next to the token selector and neither
variable seem to impact on the help popup working....

Fix notices on Task.tpl (grumpy mode)

The addition of group.id to this id dates back to
https://github.com/civicrm/civicrm-core/pull/3191

- a huge PR which attempted to block collisions on the data table id per
https://issues.civicrm.org/jira/browse/CRM-14636

However, I could find no evidence that 'group' would ever be assigned here
and as a rather large PR I suspect there was a bit of pattern replacement
beyond what was confirmed....

Removing 'group' means the selector becomes 'selectedRecords-' - which is
what it is in practice anyway since group is not assigned. This is possibly
accidentally unique anyway because of the trailing '-'

noexpand seems to never be assigned

Remove issets on pay_later_receipt

These issets were added to get tests passing with the later php version
which is now used on PR CI.

I think for these templates the value is always assigned so we can remove.

I am probably going to do a bunch of PRs removing issets to
hone in on which ones are actually still needed for the CI
so that I can put the work into the required ones

Merge pull request #22904 from eileenmcnaughton/event_msg

Add workflow template for offline event

SearchKit - Fix "all selected" checkbox when selecting a single page of results

Merge pull request #23000 from colemanw/nfcSampeDataWhitespace

SampleData - Fix missing option value names

Merge pull request #22957 from colemanw/afformClearCache

Afform - reset managed entities when deleting a dashlet

Merge pull request #22999 from civicrm/5.48

5.48

Keep "EntityFile" records available until after Symfony "delete" hooks fire

Merge pull request #22997 from colemanw/fixDeletingAfformWithSavedSearch

Fix deleting afforms when deleting saved search

5.48.0 release notes: added boilerplate

5.48.0 release notes: initial run

Merge pull request #22970 from eileenmcnaughton/grump2

Enotice fix in grumpy smarty mode