Merge pull request #22203 from colemanw/revertContactDeletePermission

Revert "Add permission metadata to contact is_deleted field"

Merge pull request #22230 from agh1/5.45.0-releasenotes-initial

5.45.0 release notes initial run

5.45.0 release notes: added boilerplate

Contributor key: add Chris Garaffa

5.45.0 release notes: raw from script

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

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.

Revert "Add permission metadata to contact is_deleted field"

This reverts commit d2ff128ce0503dc1956f02c86035686ea016f10f.
ACLs for accessing deleted contacts are already enforced in the query,
so this field was being hidden for UI purposes not ACL purposes.
It had the unfortunate side-effect of crashing any API call using
`is_deleted` in the WHERE clause for non-permissioned users.

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`