Merge pull request #22181 from eileenmcnaughton/smarty8

[Smarty variables] Remove another isset - user dashboard

Merge pull request #22216 from demeritcowboy/th-br

dev/core#2982 - Remove literal `<br/>` tag in header for Repeat Contributions CiviReport

Merge pull request #22177 from eileenmcnaughton/smarty5

Smarty variables]  Remove isset from relationship tab

remove br tag in header

Merge pull request #22215 from totten/master-mixin-xdebug

(NFC) tools/mixin/bin/test-all - Work-around segfault in CI

tools/mixin/bin/test-all - Work-around segfault in CI

Merge pull request #22211 from totten/master-544-fwd

Merge forward 5.44 => master

Merge remote-tracking branch 'origin/5.44' into master-544-fwd

Merge pull request #22209 from demeritcowboy/reldate

[NFC] Fix test failures for testRelativeDateRanges

Merge pull request #22210 from MegaphoneJon/optional-before-required

PHP 8 compat - no optional param before required

PHP 8 compat - no optional param before required

fix strict equality issue

Merge pull request #22206 from totten/5.44-bump0

5.44.0 - Bump version. Tweak notes.

Set version to 5.44.0

release-notes/5.44.0.md

Merge pull request #22182 from MegaphoneJon/core-2974

fixes core#2974, overrides break membership update status when removed

Merge pull request #22202 from agh1/5.44.0-releasenotes-final

5.44.0 release notes: added late changes

Merge pull request #22204 from totten/master-mixin-testall

tools/mixin/bin/test-all - Fix guard for EXT_DIR

tools/mixin/bin/test-all - Fix test for EXT_DIR

There are two slightly different dirs - the example-extension and the extension-container (ie child-dir and parent-dir).
We want to assert that the parent-dir is well-formed, but then use the child-dir.

Merge pull request #22176 from eileenmcnaughton/smarty4

[Smarty variables] [CiviCase] remove isset from case dashboard

Merge pull request #22198 from totten/master-mix-dec-first

civix#175 - Add support for mixins. Switch core extensions to mixin/setting-php

5.44.0 release notes: added late changes

Merge pull request #22173 from colemanw/domainStuff

APIv4 - Add fixes & tests for domain-specific managed entities

Remove isset from relationship tab

civicrm/contact/view?reset=1&cid=2&relatedChild=relationship

fixes core#2974, overrides break membership update status when removed

Merge pull request #21751 from mlutfy/fixCurrencies

dev/financial#184 Fix currency name for Ghana and Belarus

[Smarty variables] remove isset from case dashboard

Merge pull request #22185 from eileenmcnaughton/smarty11

[Smarty variables] - remove isset

dev/financial#184 Fix currency name for Ghana and Belarus

Merge pull request #22179 from eileenmcnaughton/smarty6

[Smarty variables]  Fix contribution tab to work with escape by default

Merge pull request #22196 from eileenmcnaughton/smarty21

[Smarty variables] The last isset....

mixin/setting-php - Remove unused boilerplate

mixin/setting-php - Convert live settings from `hook_alterSettingsFolder` to `<mixin>`

mixin/setting-php - Import

mixin/polyfill.php - Import. Update comments.

tools/mixin - Import. Also, update to run within civicrm-core, and add JUnit output.

tests/extensions/shimmy/ - Import. Also, update comments for LifecycleTest.

distmaker - Include `mixin/*` files

civix#175 - Add support for mixins. Use MixinScanner/MixinLoader and boot-cache.

Overview
--------

(NOTE: For this description, I reference the term "API" in the general sense of a programmatic interface -- such as
a hook or file-naming convention. It is not specifically about CRUD/DB APIs.)

The `civix` code-generator provides support for additional coding-conventions -- ones which are more amenable to
code-generation.  For example, it autoloads files from `xml/Menu/*.xml` and `**/*.mgd.php`.  The technique for
implementing this traditionally relies on generating a lot of boilerplate.

This patch introduces a new construct ("mixin") which allows boilerplate to be maintained more easily.  A mixin
inspects an extension programmatically, registering new hooks as needed.  A mixin may start out as a file in `civix`
(or even as a bespoke file in some module) - and then be migrated into `civicrm-core`. Each mixin has a name and
version, which means that (at runtime) it will only load the mixin once (ie the best-available version).

See: https://github.com/totten/civix/issues/175

Before
------

The civix templates generate a few files, such as `mymod.php` and `mymod.civix.php`.
A typical example looks like this:

```php
// mymod.php - Implement hook_civicrm_xmlMenu
require_once 'mymod.civix.php';
function mymod_civicrm_xmlMenu(&$all, $the, $params) {
  _mymod_civix_civicrm_xmlMenu($all, $the, $params);
}
```

and

```php
// mymod.civix.php - Implement hook_civicrm_xmlMenu
function _mymod_civix_civicrm_xmlMenu(&$all, $the, $params) {
  foreach (_mosaico_civix_glob(__DIR__ . '/xml/Menu/*.xml') as $file) {
    $files[] = $file;
  }
}
```

These two files are managed differently: `mymod.php` is owned by the developer, and they may add/remove/manage the
hooks in this file.  `mymod.civix.php` is owned by `civix` and must be autogenerated.

This structure allows `civix` (and any `civix`-based extension) to take advantage of new coding-convention
immediately. However, it comes with a few pain-points:

* If you want to write a patch for `_mymod_civix_civicrm_xmlMenu`, the dev-test-loop requires several steps.
* If `civix` needs to add a new `hook_civicrm_foo`, then the author must manually create the stub
  function in `mymod.php`. `civix` has documentation (`UPGRADE.md`) which keeps a long list of stubs that must
  be manually added.
* If `civix` has an update for `_mymod_civix_civicrm_xmlMenu`, then the author must regenerate `mymod.civix.php`.
* If `mymod_civix_xmlMenu` needs a change, then the author must apply it manually.
* If `civix`'s spin on `hook_civicrm_xmlMenu` becomes widespread, then the `xmlMenu` boilerplate is duplicated
  across many extensions.

After
-----

An extension may enable a mixin in `info.xml`, eg:

```xml
<mixins>
  <mixin>civix-register-files@2.0</mixin>
</mixins>
```

Civi will look for a file `mixin/civicrm-register-files@2.0.0.mixin.php` (either in the extension or core). The file follows this pattern:

```php
return function(\CRM_Extension_MixInfo $mixInfo, \CRM_Extension_BootCache $bootCache) {
  // echo "This is " . $mixInfo->longName . "!\n";
  \Civi::dispatcher()->addListener("hook_civicrm_xmlMenu", function($e) use ($mixInfo) {
    ...
  });
}
```

The mixin file is a plain PHP file that can be debugged/copied/edited verbatim, and it can register for hooks on its
own.  The code is no longer a "template", and it doesn't need to be interwoven between `mymod.php` and
`mymod.civix.php`.

It is expected that a system may have multiple copies of a mixin.  It will choose the newest compatible copy.
Hypothetically, if there were a security update or internal API change, core might ship a newer version to supplant the
old copy in any extensions.

Technical Details
-----------------

Mixins may define internal classes/interfaces/functions. However, each major-version
must have a distinct prefix (e.g. `\V2\Mymixin\FooInterface`). Minor-versions may be
provide incremental revisions over the same symbol (but it's imperative for newer
increments to provide the backward-compatibility).

MixinScanner - Make it easier to instantiate and pay with instances

Ex: cv ev '$o=new CRM_Extension_MixinScanner(); var_export($o->createLoader());'

MixinScanner - Enable scanning of `[civicrm.root]/mixin`

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

Fix contribution tab to work with escape by default

This removes some isset that affect contact contribution tab if escape on output is enabled.

This might regress some smarty e-notices. I think if tests pass that is OK at this stage
as we never eliminted them & getting to security enablable seems like a higher priority

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

[Smarty variables - remove isset

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 another isset

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

APIv4 - Add fixes & tests for domain-specific managed entities

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