Merge pull request #22196 from eileenmcnaughton/smarty21

[Smarty variables] The last isset....

Merge pull request #22165 from eileenmcnaughton/no_escape

Smarty variables]  Prevent settings form assigned html from being escaped

Merge pull request #22197 from totten/master-xdebug3

DebugSubscriber - Fix test-suite compatibility with XDebug 3

Merge pull request #22188 from totten/master-uninstall

ManagedEntities - Fix permission error during uninstallation (regression-fix)

Merge pull request #22175 from eileenmcnaughton/smarty3

[Smarty variable][Civicase] remove isset

Merge pull request #22180 from eileenmcnaughton/smarty7

Remove issets from Activity search screen

Merge pull request #22172 from eileenmcnaughton/smarty

Avoid default escaping for blog titles

Merge pull request #22174 from eileenmcnaughton/smarty2

Remove isset checks on isForm

Merge pull request #22184 from eileenmcnaughton/smarty10

[Smarty variables] Remove isset from add new group form

Merge pull request #22164 from eileenmcnaughton/parent

Remove empty check previously commented to be meaningless

DebugSubscriber - Fix compatibility with XDebug 2/3

Merge pull request #22178 from demeritcowboy/custom-null

dev/core#2973 - All custom fields broken on edit forms

[Smarty variables] The last isset....

civicrm/report/list?reset=1

Merge pull request #22081 from colemanw/searchKitSortable

SearchKit - Add drag-sortable weight functionality

ManagedEntities - Fix permission error during uninstallation (regression-fix)

Overview
--------

Fixes a recent regression that prevents you from uninstalling extensions via
CLI.  This specifically affects extensions which use managed entities.

Steps to reproduce
------------------

```
cv en afform
cv dis afform
cv ext:uninstall afform
```

Before
-------

```
[bknix-max:~/bknix/build/dmaster/web/sites/all/modules/civicrm] cv en afform && cv dis afform && cv ext:uninstall afform
Enabling extension "org.civicrm.afform"
Disabling extension "org.civicrm.afform"
Uninstalling extension "org.civicrm.afform"
Error: API Call Failed: Array
(
    [entity] => Extension
    [action] => uninstall
    [params] => Array
        (
            [keys] => Array
                (
                    [0] => org.civicrm.afform
                )

            [debug] => 1
            [version] => 3
        )

    [result] => Array
        (
            [error_code] => unauthorized
            [entity] => Extension
            [action] => uninstall
            [trace] => #0 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php(147): Civi\API\Kernel->authorize(Object(Civi\Api4\Provider\ActionObjectProvider), Object(Civi\Api4\Generic\DAODeleteAction))
 #1 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/AbstractAction.php(234): Civi\API\Kernel->runRequest(Object(Civi\Api4\Generic\DAODeleteAction))
 #2 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/api/api.php(85): Civi\Api4\Generic\AbstractAction->execute()
 #3 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Core/ManagedEntities.php(467): civicrm_api4('OptionValue', 'delete', Array)
 #4 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Core/ManagedEntities.php(303): CRM_Core_ManagedEntities->removeStaleEntity(Object(CRM_Core_DAO_Managed))
 #5 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Core/ManagedEntities.php(134): CRM_Core_ManagedEntities->reconcileUnknownModules()
 #6 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Core/Invoke.php(409): CRM_Core_ManagedEntities->reconcile()
 #7 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/CRM/Extension/Manager.php(483): CRM_Core_Invoke::rebuildMenuAndCaches(true)
 #8 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/api/v3/Extension.php(183): CRM_Extension_Manager->uninstall(Array)
 #9 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Provider/MagicFunctionProvider.php(89): civicrm_api3_extension_uninstall(Array)
 #10 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php(149): Civi\API\Provider\MagicFunctionProvider->invoke(Array)
 #11 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php(81): Civi\API\Kernel->runRequest(Array)
 #12 /home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/api/api.php(22): Civi\API\Kernel->runSafe('Extension', 'uninstall', Array)
 #13 phar:///home/me/bknix/bin/cv/src/Command/BaseCommand.php(49): civicrm_api('Extension', 'uninstall', Array)
 #14 phar:///home/me/bknix/bin/cv/src/Command/ExtensionUninstallCommand.php(63): Civi\Cv\Command\BaseCommand->callApiSuccess(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput), 'Extension', 'uninstall', Array)
 #15 phar:///home/me/bknix/bin/cv/vendor/symfony/console/Command/Command.php(257): Civi\Cv\Command\ExtensionUninstallCommand->execute(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
 #16 phar:///home/me/bknix/bin/cv/vendor/symfony/console/Application.php(850): Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
 #17 phar:///home/me/bknix/bin/cv/vendor/symfony/console/Application.php(193): Symfony\Component\Console\Application->doRunCommand(Object(Civi\Cv\Command\ExtensionUninstallCommand), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
 #18 phar:///home/me/bknix/bin/cv/src/Application.php(46): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
 #19 phar:///home/me/bknix/bin/cv/vendor/symfony/console/Application.php(124): Civi\Cv\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
 #20 phar:///home/me/bknix/bin/cv/src/Application.php(15): Symfony\Component\Console\Application->run()
 #21 phar:///home/me/bknix/bin/cv/bin/cv(27): Civi\Cv\Application::main('phar:///Users/t...')
 #22 /home/me/bknix/bin/cv(14): require('phar:///Users/t...')
 #23 {main}
            [is_error] => 1
            [error_message] => Authorization failed
        )

)
```

After
-----

Works

Comment
-------

I encountered this while working on E2E test-coverage for other changes.
The E2E test coverage had worked on a previous iteration of 5.45.alpha1 but
failed when I rebased. Consequently, this means

You can see a prior draft of the E2E test [here](https://github.com/totten/shimmy/blob/master-reorg/shimmy/tests/phpunit/E2E/Shimmy/LifecycleTest.php#L56-L77).
However, it's being reworked as a core patch.

I'd suggest accepting this without a test - because (a) it's a regression and (b) there will be coverage from the pending change.

Remove isset from add new group form

civicrm/group/add?reset=1

Remove issets from Activity search screen

This will regress enotices - but I think it's an OK trade off at this stage to try to squeeze out a few more issets
while most people can't see those notices anyway

fix custom fields

[Smarty variable][Civicase] remove isset

Affects civicrm/case?reset=1 when cases exist

Remove isset checks on isForm

We are now assigning these from Core_Page so they should be set. If we do get
an enotice back from this then as long as it passes tests it will
be the lesser evil & picked up later

Avoid default escaping for blog titles

These have escaping - but we dont want the html to be escaped because it messed with Karin's emojis :-)

Merge pull request #21232 from colemanw/APIDelete

APIv4 - Use correct BAO delete function (fixes dev/core#2757)

Merge pull request #22171 from civicrm/5.44

5.44

Merge pull request #22170 from colemanw/fixSearchKitOptions

SearchKit - Fix regression for pseudoconstant selection

APIv4 - Add useTrash option to soft-delete contacts

This sets Contact::delete to move contacts to the trash by default.

APIv4 - Use correct BAO delete function

Uses BAO::del() only if it isn't deprecated.

SearchKit - Fix regression for pseudoconstant selection

Fixes dev/report#83

SearchKit - Add drag-sortable weight functionality

Drag-sortable weights are similar to in-place edit in that it uses the API
to update records in the table. In this case it updates the "weight" column when
the user drags a row into a different position.

Merge pull request #22169 from demeritcowboy/oauth-dropdown

dev/mail#105 - Oauth dropdown missing on mail settings form

Merge pull request #22161 from eileenmcnaughton/pay_test2

Fix more tests to use full flow

Merge pull request #22143 from mattwire/statuspagehigherfirst

Display higher severity status checks first

Merge pull request #22142 from mattwire/envokextensionsnatcase

Sort list of extensions alphabetically in 'Extensions ok' check

Merge pull request #22168 from demeritcowboy/useajax

Fix broken extension page smarty assignment

Merge pull request #22167 from eileenmcnaughton/location_tpl

[Smarty Variables] Remove isset from location type form

Merge pull request #22166 from eileenmcnaughton/isset

[Smarty variables] Remove issets from scheduled job screen

[Smarty Variables] Remove isset from location type form

https://dmaster.localhost:32353/civicrm/admin/locationType?reset=1

Merge pull request #22162 from eileenmcnaughton/pay_test

Fix membership test to use full flow

oauth dropdown missing on mail settings form

fix smarty

Prevent settings form assigned html from being escapted

[Smarty variables] Remove issets from scheduled job screen

Remove isset previously commented to be meaningless

Merge pull request #22074 from eileenmcnaughton/tags

[Smarty variables] Ensure groupElementType is always set

Merge pull request #22132 from eileenmcnaughton/test_con

[Smarty variables] [contact summary]  Assign variable to determine whether to show email signature field

Merge pull request #22138 from eileenmcnaughton/cust_edit

[Smarty variables] [custom data form] Template notice cleanup - make sane

Fix more tests to use full flow

Fix membership test to use full flow

Merge pull request #22147 from eileenmcnaughton/pager

[Smarty variables] [report test]Ensure smarty variables are consistently assigned in report

Merge pull request #22152 from eileenmcnaughton/n1

[Smarty variables] [Activity form] Ensure separation,tag are assigned to the template

Merge pull request #22155 from eileenmcnaughton/n4

[Smarty variables] Specify isRepeatingEntity when including ConfirmRepeatMode.tpl

Merge pull request #22160 from colemanw/navigationPermission

APIv4 - Treat navigation permissions as array, add pseudoconstant for operator

Merge pull request #22137 from colemanw/api4SortableEntity

APIv4 - Add SortableEntity type which auto-adjusts weights

APIv4 - Treat navigation permissions as array, add pseudoconstant for operator

This gives consistency in how the fields are handled in the Navigation and Dashboard entities

Merge pull request #22121 from colemanw/bridgeEntitiesGoneWild

SearchKit - enable search by case role

Ensure groupElementType is always set

Ensure smarty variables are consistently assigned in report

Merge pull request #22158 from colemanw/api4DeleteOptionList

APIv4 - Delete unused OptionList trait

Merge pull request #22118 from eileenmcnaughton/notices

Fix message templates screen to work with escape enabled by default

Merge pull request #22154 from eileenmcnaughton/n3

[Smarty variables] Use always-assigned isTagSet to determine if is tagset

Merge pull request #22153 from eileenmcnaughton/n2

[Smarty variables] [Activity form] Ensure activityTypeFile is always assigned

Merge pull request #22157 from eileenmcnaughton/testit

[Test] Fix tests to use form helper

Merge pull request #22156 from eileenmcnaughton/cont_tests

[Test] Test cleanup to use full-form methods

APIv4 - Delete unused OptionList trait

This trait had been added for the sake of SearchKit to determine which entities should be
searchable and which were merely supplying option lists for other entities.

However, this logic broke down quickly as some option lists *should* be searchable,
so that path was abandoned in favor of the `@searchable` annotation.

The trait now does nothing and is just taking up space and adding unnecessary complexity.

APIv4 - Limit SortableEntity exports by domain

APIv4 - Add SortableEntity and ManagedEntity traits to Navigation menu entity

Excludes 'weight' from managed entity calculations for references,
adds unit tests for the interaction of managed entities and sortable entities

Fix test for compatability with auto-weights

Fix unit tests to enable components before using them

SearchKit - Use order_by from metadata to supply default sort for searchDisplays

APIv4 - Add SortableEntity type which auto-adjusts weights

This entity type will manage weight columns automatically, allowing items to be
re-ordered easily.
Simply by updating the weight of one record, others will auto-adjust to make room for it.

Fix tests to use form helper

Merge pull request #22151 from colemanw/optionValueSearch

SearchKit - Allow searches of OptionValues, LocationTypes, CustomFields

Test cleanup to use full-form methods

Uses full form flow  - via a helper

Specify isRepeatingEntity when including ConfirmRepeatMode.tpl

Use always-assigned isTagSet to determine if is tagset

Ensure activityTypeFile is always assigned

Ensure separation,tag are assigned to the template

SearchKit - Enable for CustomField, CustomGroup & LocationType

Fix message templates screen to work with escape enabled by default

SearchKit - Allow searches of OptionGroups and OptionValues

Marks the entities searchable, and adds view/edit links for OptionValues

Merge pull request #22148 from eileenmcnaughton/link

Enotice fixes - ensure a couple more variables are consistently assigned

Merge pull request #22124 from eileenmcnaughton/cust

Remove isset from custom data tpls

Enotice fixes - ensure a couple more variables are consistently assigned

Ensure smarty variables are consistently assigned in report

Remove isset from custom data tpls

Assign variable to determine whether to show email signature field

This gets us around complex 'empty checks' and allows us to avoid enotices in smarty

Ensure formal_title is assigned to the template.

This just gets past one enotice in the great juggle

Template notice cleanup - make sane

The CRM/Custom/Form/CustomData.tpl is called with formEdit set from
1 place - CRM/Contact/Form/Edit/CustomData.tpl - when this is the
case only 1 line of code is shared with the code path if
formEdit is not true/ passed in. This is silly & can be avoided

Merge pull request #22140 from eileenmcnaughton/cust_case

Assign cgCount in Case custom data form

Merge pull request #22125 from colemanw/searchKitEditing

SearchKit - Improve UX for refresh after editing

Merge pull request #21919 from braders/deprecated-jquery-xhr-usage

[REF] Remove usage of deprecated jqXHR.error in jQuery code

Merge pull request #22014 from colemanw/apiExport

APIv4 - Add export action for managed entities

Merge pull request #22139 from eileenmcnaughton/cust_opt

Ensure custom fields properties are assigned to the template

Merge pull request #22145 from civicrm/5.44

5.44

Merge pull request #22141 from eileenmcnaughton/contexty

Ensure context is always assigned

APIv4 - Add export action to managed entities

This action generates an exportable array suitable for use in a .mgd.php file.

Merge pull request #22144 from alifrumin/rn5.44

NFC first stab at 5.44 release notes

NFC release notes 5.44

Display higher severity status checks first

Sort list of extensions alphabetically in 'Extensions ok' check

Ensure context is always assigned