Finish conversion of static function to non-static

The processSecondaryContribution function is no longer static but the class still passes itself
to the function as the form parameter. This updates calls to form to calls to this.

Also for clarity the function is made private

I did do a quick universe search to confirm no calls to it - even thought they
would have been clearly unsupported

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 #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

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#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

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....

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

Merge pull request #22982 from braders/filter_sanitize_string-deprecation

Remove use of deprecated FILTER_SANITIZE_STRING

Merge pull request #22988 from eileenmcnaughton/prof

Enotice fix (grumpy smarty)

Merge pull request #22989 from eileenmcnaughton/prof2

Enotice fixes

Merge pull request #22990 from eileenmcnaughton/init

Grumpy smarty - Never escape 'initHideBoxes'

SampleData - Fix missing option value names

NFC - cleanup whitespace in sample_custom_data.mysql

Merge pull request #22985 from demeritcowboy/postupgrademessage-2

dev/core#3119 - Post-upgrade messages no longer being displayed

Merge pull request #22998 from seamuslee001/update_guzzlehttp_18

[REF] Update Guzzlehttp/psr7 version to be 1.8.5

[REF] Update Guzzlehttp/psr7 version to be 1.8.5

Fix deleting afforms when deleting saved search

A typo in the code was preventing afforms from being deleted with a corresponding saved search,
and may have led to afforms being deleted incorrectly!

Merge pull request #22991 from eileenmcnaughton/adv_select

Do not escape hierarchical select

CRM_Upgrade_Form::doFinish() - Narrower call to session-reset

The main call(s) to `rebuildMenuAndCaches()` do clear out most caches.  It's
_just_ the session that has been skipped.

Post-upgrade messages no longer being displayed (part 2)

This line is only hit if the upgrader activates CiviGrant.

Merge pull request #21514 from mattwire/1624_2319_casedashboard

Fix issues with case sorting/searching from case dashboard

Merge pull request #22987 from mattwire/notedateform

Make note date required when creating note

Merge pull request #22993 from mattwire/is_templateundefined

Fix undefined index by making sure that is_template is always defined

Fix undefined index by making sure that is_template is always defined

Do not escape hierarchical select

We already have 'cutouts' in the escape function to not escape assigned html.
It isn't catching the hierarchical select elements - so I have marked a bunch
to not be escaped. I could have handled in 'escape' but I think tpl handling
is actually correct - the cutouts in the escape function was to get it to
a more manageable level

Never escape 'initHideBoxes'

InitHideBoxes is js - fugly as this is we shouldn't escape it - there is
a world in which we would re-write it out of existence but not this one....

Enotice fixes

civicrm/admin/uf/group/add?action=add&reset=1

Enorice fix (grumpy smarty)

Make note date required when creating note

Merge pull request #22975 from colemanw/afformLayout2

Afform - Easier layout creation with predefined container styles and built-in title

Afform - Predefined container styles

Adds a predefined "Panel Pane" style, for creating dashboards.

Afform - add collapsible title as directive

Before: A fieldset `<legend>` was treated as its own element. This was more flexible but more complex.
After: Augenerated `<legend> for fieldsets or `<h4>` for other containers based on new `af-title` directive.

This allows central control of titles for e.g. collapsible styles.
Fixes dev/core#3110

CrmUI - Improve crmUiEditable to support placeholders, and use its own css

Now this widget supports having a `placeholder`.
Also gave it its own css instead of piggybacking off crm-editable classes,
as that was a bit messy and Shoreditch was applying some extra stuff which
then had to be undone.

Afform - Fix dragging and dropping search fields and containers, saving blocks

Afform - Better descriptive labels for text element

SearchKit - Better default label for new displays

Merge pull request #22902 from mattwire/contributionconfirm

Cleanup variable assigns in ContributionPage confirm

Merge pull request #22984 from civicrm/5.48

5.48

postupgrade messages no longer being displayed

Merge pull request #22963 from colemanw/fixAfformBlocks

Afform - Fix saving and editing entity blocks