Convert remaining properties to local variables / private properties

Merge pull request #26855 from colemanw/actionMappingCleanup

[REF] ActionMapping - Improve class structure

Merge pull request #26874 from eileenmcnaughton/property_test

Cleanup test class, including fix undeclared property

Cleanup test class, including fix undeclared property

Merge pull request #26871 from demeritcowboy/upg-smarty

Upgrade doesn't work

Merge pull request #26872 from eileenmcnaughton/custom

Remove never set property from previously shared code

correct precedence

Remove never set property from previously shared code

be more literal

ActionMapping - Clean up obtuse methods of fetching a MappingInterface class

Changed calls to getMappings to use getMapping if they only needed a single value.
In some cases they could be entirely removed because they weren't using the retrieved class for anything.
Deprecated the use of 'filters' param in getMappings as it was redundant with simply calling getMapping with an id.

Merge pull request #26869 from yashodha/add_tpl_class

fix duplicate id for softCredit

fix duplicate id for softCredit

ActionMapping - Improve class structure

Completes the refactoring that began with https://github.com/civicrm/civicrm-core/pull/6687
Fully migrates from the old model based on the civicrm_action_mapping table
(dropped in https://github.com/civicrm/civicrm-core/pull/26646)
by removing all references to that structure and fully embracing the object-model.
New common base-class added to make it eaiser to implement MappingInterface.

Merge pull request #26866 from eileenmcnaughton/logged

Superficial cleanup in pledge form

Merge pull request #26852 from totten/master-s3b

Standalone - Web-based installation for PHP built-in server

Merge pull request #26864 from colemanw/deleteBoilerplate

[NFC] Delete unused boilerplate comments

Superficial cleanup in pledge form

Delete unused boilerplate comments

Merge pull request #26863 from aydun/searchui_create

Create SearchUI extension

Update AdminUI & SearchUI info.xml

Move Contact Search from AdminUI to SearchUI

Create SearchUI extension

Merge pull request #26862 from yashodha/add_tpl_class

Add classes to membership results

add classes to membership results

Merge pull request #26860 from eileenmcnaughton/member

[PHP8.2] move property declaration form CRM_Member_Form_Membership to parent

Merge pull request #26842 from colemanw/isActiveToggle

AdminUI - Add toggle for is_active

Merge pull request #26857 from eileenmcnaughton/logged

[tests][php8.2] Use function rather than sometimes declared property for loggedInUserID

[tests][php8.2] Use function rather than sometimes declared property for loggedInUserID

Merge pull request #26859 from eileenmcnaughton/cust_case

[php8.2] Fix undeclared properties on case custom data form

Merge pull request #26858 from eileenmcnaughton/cust_by_type

Remove never-true-if

Merge pull request #26845 from eileenmcnaughton/subname

Remove `_subName` property

[php8.2] Fix undeclared properties on case custom data form

[PHP8.2] move property declaration form CRM_Member_Form_Membership to parent

Remove never-true-if

Merge pull request #26856 from seamuslee001/p82_logged_in_user

[NFC][PHP8.2] Fix dynamic property _loggedInUser

[NFC][PHP8.2] Fix dynamic property _loggedInUser

ActionMapping - Rename ambiguous 'getEntity' function

Before: function getEntity() actually returns the name of the table, not the entity
After: function getEntityTable() returns table name and getEntityName() returns entity name

Merge pull request #26843 from colemanw/searchKitButtons

SearchKit - fix min-width of buttons

Merge pull request #26850 from totten/master-ref-mail

(REF) CMSUser::create - Rename misleading parameter

Merge pull request #26851 from eileenmcnaughton/unshared

Follow up fix (master only regression)

Setup UI - Use separate "Database" section (on standalone)

Setup UI - Display "Administrator" options (on standalone)

Follow up fix

Standalone - Interpret the $mailParam (same way as everyone else)

setup - Fix php8 warnings when MySQL password is blank

SetupResponse - Fix php8 warning

SelectValues::permissions - Fix php8 warning

DbUtil - Report MySQL failures consistently on php7+php8

router.php - Enable the "civicrm-setup" UI

router.php - Fix for php73 compat

Standalone.civi-setup.php - Fix 'HTTP'/'HTTPS' detection on non-Apache environments

(REF) CMSUser::create - Rename misleading parameter

The second parameter is very confusing:

* The name `$mail` suggests that the parameter is an email address.
* The parameter `$mail` actually contains the name of the *field* which has the email address.

This is a weird contract. It invites misinterpretation -- and now, e.g., some of the `Standalone`
code has misused it.

To confirm that `$mail` is actually the parameter-name, note that:

1. `create()` actually looks for `$params[$mail]`
2. Callers like `CRM/Contact/Form/Task/Useradd.php` actually give a parameter name

Merge pull request #26771 from totten/master-s3

Standalone - Add support for development-friendly file-layouts

Merge pull request #26839 from colemanw/searchKitActionLinksCleanup

Search kit action links cleanup

Merge pull request #26848 from mlutfy/newEventTR

[NFC] New Event: remove empty table row

Merge pull request #26846 from eileenmcnaughton/custom

Notice fix - ensure extends_entity_column_value isset

New Event: remove empty table row

Merge pull request #26844 from eileenmcnaughton/unshared

Fold private `setGroupTree` function back into caller

Notice fix - ensure extends_entity_column_value isset

Remove subName property in favour of an internal variable

This is not used outside the function

Remove use of subName property

Merge pull request #26791 from jmcclelland/hide-view-only

hide view only custom fields on merge screen

Fold private setGroupTree function back into caller

SearchKit - fix min-width of buttons

AdminUI - Add toggle for is_active

SearchKit - Ensure task links work with relationships

SearchKit - Make loading task links more efficient

Merge pull request #26837 from eileenmcnaughton/no_pre_process

Fold `preProcess` into `run`

Merge pull request #26836 from eileenmcnaughton/unshared

Remove last reference to undefined property `_subType`

Merge pull request #26838 from seamuslee001/php82_ssid_dynamic

[REF][PHP8.2] Ref fix deprecation in PHP8.2 about dynamic property _ssID

Use internal var for groupID

Merge pull request #26835 from seamuslee001/performance_fix

Ensure that we only check permissions if we want to in getActions fol…

[REF][PHP8.2] Ref fix deprecation in PHP8.2 about dynamic property _ssID

Fold preProcess into run

there are properties that appear to be being set solely to pass variables from preProcess to
run.

preProcess is called from run and that is the only way it is accessed.

There are some forms that use the pageRun hook in the universe but this has no impact on them

SearchKit - Break getLinkInfo into 3 smaller functions

Merge pull request #26834 from eileenmcnaughton/amount

[REF] Use internal variable rather than property to pass variable

Remove last reference to undefined property ->_subType

Ensure that we only check permissions if we want to in getActions follow up to #26823

Make it more readable as per coleman

Merge pull request #26832 from eileenmcnaughton/unshared

[REF] [PHP8.2] Post unshare cleanup

Use internal variable rather than property to pass variable

Merge pull request #26833 from eileenmcnaughton/unshared2

[REF] [PHP8.2]Treat subType as a internal varible, not a form property

Do not pass in subType as the same value will be calculated

Add deprecation to otherwise deprecated chunk

Treat subType as a internal varible, not a form property

Merge pull request #26830 from seamuslee001/php82_contribute_history

[REF][PHP8.2] Fix Deprectation notice due to dynamic properties on th…

Merge pull request #26828 from seamuslee001/php82_logging_detail

[REF][PHP8.2] Fix deprecated dynamic properties in Logging Report Detail

Post unshare cleanup

This stops passing & returning unused values from the previously shared function
& removes a deprecated chunk.

This deprecation was very precautionary IMHO so 6 months is long enough

Merge pull request #26829 from eileenmcnaughton/cust_share

[Ref] [php8] Unshare shared function `CRM_Custom_Form_CustomData::setGroupTree` in order to clean-up

Merge pull request #26776 from eileenmcnaughton/result

[REF] Stop passing `result` into `getContributionParams` to get one value from it, in one code path

Merge pull request #26775 from eileenmcnaughton/cont_recur

[Ref] Minor cleanup on handling of financial type

[REF][PHP8.2] Fix Deprectation notice due to dynamic properties on the Contribute History report

[REF][PHP8.2] Fix deprecated dynamic properties in Logging Report Detail

Merge pull request #26827 from seamuslee001/php82_multirecord_linsting

[REF][PHP8.2] Fix Dynamic Property Deprecation notice in MultiRecordListing

Unshare shared function in order to clean-up

Merge pull request #26826 from eileenmcnaughton/case_summary

Clean up CaseSummary Report removing undeclared properties and minor fixes

[REF] Fix Dynamic Property Deprecation notice in MultiRecordListing

Clean up CaseSummary Report removing undeclared properties and minor fixes

Merge pull request #26823 from seamuslee001/api4_get_actions_dynamic

[REF] Ensure that getActions respects any modifications by event list…

hide view only custom fields on merge screen

And, ensure they are not merged. View only fields with values on the
duplicate that is being discarded will be discarded.

Add unit test to verify patch fixes dynmaically permissable entity actions