Merge pull request #22222 from civicrm/5.45

5.45

Merge pull request #22221 from seamuslee001/nfc_fix_system_test_mysql8

[NFC] Update System Uft8mb4 check to handle for the fact that MySQL8 …

Merge pull request #22220 from seamuslee001/drupal9_get_version

[REF] Add in getVersion override for Drupal 8/9 to support cv testing…

Merge pull request #22201 from colemanw/exportPseudoconstantFix

APIv4 Export - Fix logic for exporting pseudoconstant syntax

[NFC] Update System Uft8mb4 check to handle for the fact that MySQL8 outputs utf8mb3 when the charset has been set to utf8 as utf8mb3 is the underlyling charset for utf8

[REF] Add in getVersion override for Drupal 8/9 to support cv testing and also cv vars:show picking up the right CMS version

Set version to 5.46.alpha1

Set version to 5.45.beta1

Merge pull request #22199 from totten/master-mix-dec-all

civix#175 - Switch core extensions to mixins (ang-php, menu-xml, theme-php, case-xml, mgd-php)

Merge pull request #22219 from demeritcowboy/selenium-webtests

[NFC] Remove Selenium WebTest code from CiviMailUtils

Merge pull request #22218 from colemanw/fixCanAggregate

SearchKit - Fix undefined variable for searches of Afforms etc.

remove selenium webtest code

mixin/case-xml - Enable CiviCase during testing

Merge pull request #22189 from eileenmcnaughton/smarty14

[Smarty variables] Consistently assign isAdmin at php layer

Merge pull request #22205 from totten/master-hook-conform

APIv4 ConformanceTest - Demonstrate entity APIs which fail to emit `hook_civicrm_post(delete)`

(REF) MixinLoader - Convert static to local property

(REF) Mixins - Promote `MixinLoader` to tracked service-object

(REF) Mixins - Move `applyMixins()` logic into `MixinLoader`

(REF) Mixins - Move more aspects of scanning from MixinLoader to MixinScanner

(REF) Mixins - Split apart class-loader and mixin-loader

ext/**.civix.php - Remove _civix_glob(). Not used anymore.

ext/**.civix.php - Remove _civix_find_files(). Not used anymore.

mixin/mgd-php - Remove unused boilerplate

mixin/mgd-php - Convert live `*.mgd.php` from `hook_managed` to `<mixin>`

mixin/mgd-php - Import

mixin/ang-php - Convert live Angular modules from `hook_angularModules` to `<mixin>`

mixin/menu-xml - Remove unused boilerplate

mixin/menu-xml - Convert live menu-items from `hook_xmlMenu` to `<mixin>`

mixin/menu-xml - Import

mixin/case-xml - Remove unused boilerplate

mixin/case-xml - Import

mixin/theme-php - Remove unused boilerplate

mixin/theme-php - Import

mixin/ang-php - Remove unused boilerplate

mixin/ang-php - Convert live Angular modules from `hook_angularModules` to `<mixin>`

mixin/ang-php - Import

Merge pull request #22207 from colemanw/baoDeleteHooks

[REF] - Switch BAOs to use standard delete function which calls hooks

Merge pull request #22191 from eileenmcnaughton/smarty16

Smarty variables]  Remove isset from a.net

Merge pull request #22190 from eileenmcnaughton/smarty15

[Smarty variables] remove isset from Mail accounts

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

SearchKit - Fix undefined variable for searches of Afforms and other non-DAO entities

APIv4 Export - Fix logic for exporting pseudoconstant syntax

FinancialType - Use standard delete function which calls hooks

FinancialAccount - Use standard delete function which calls hooks

PaymentProcessorType - Use standard delete function which calls hooks

PaymentProcessor - Use standard delete function which calls hooks

LocationType - Use standard delete function which calls hooks

MessageTemplates - Use standard delete function which calls hooks

MembershipStatus - Use standard delete function which calls hooks

MembershipType - Use standard delete function which calls hooks

RelationshipType - Use standard delete function which calls hooks

WordReplacement - Use generic writeRecords/deleteRecords which call hooks

OptionGroup - Use standard delete function which calls hooks

ContactType - Use standard delete function which calls hooks

CaseType - Use standard delete function which calls hooks

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

ConformanceTest::checkDeletionAllowed - Coverage for `hook_pre`. Fix `Contact`.

1. Expand assertions used for `hook_post` to check `hook_pre`.

2. Recognize the quirks in how `Contact` records are treated in the hooks.

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

ConformanceTest - Demonstrate entity APIs with non-conformant deletion

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

Remove isset from a.net

[Smarty variables] remove isset from Mail accounts

civicrm/admin/mailSettings?reset=1

[Smarty variables] Consistently assign isAdmin at php layer

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