Merge pull request #21866 from seamuslee001/fix_pdf_test

[REF] Fix PDF Test failure on php versions 7.4 and later

[REF] Fix PDF Test failure on php versions 7.4 and later

Merge pull request #21851 from colemanw/entityTypesHookRecursionFix

Entity types hook recursion fix

Merge pull request #21827 from eileenmcnaughton/sent

dev/core#2914 Fix for incorrect sent count in message

Cleanup ContactType BAO fixme and comments; use common function

This fixes the fixme in basicTypeInfo to use getAllContactTypes instead of running its
own query, and cleans up some comments.

Fix possible recursion during Entity::get()

Use plain SQL instead of APIv4 in a function that is called by APIv4 Entity.get
to avoid an infinite loop.

dev/core#2914 Fix for incorrect sent count in message

Merge pull request #21839 from jensschuppe/api4CreateRequestEvent

Load the API kernel service prior to dispatching the `civi.api4.createRequest` event when creating an API4 request

Load the API kernel service prior to dispatching the `civi.api4.createRequest` event when creating an API4 request

Merge pull request #21837 from totten/5.43-nfc-escape

(NFC) TokenConsistencyTest - Add some examples of HTML escaping

(NFC) TokenConsistencyTest - More aggressive test cleanup

(NFC) TokenConsistencyTest - Add some examples of HTML escaping

Merge pull request #21830 from totten/5.43-regen

(dev/core#2836) regen.sh - Fix incorrect reference to "flexmailer". Update data file.

Merge pull request #21828 from colemanw/api4RecursionFix

APIv4 - Fix possible recursion during Entity::get()

Merge pull request #21831 from seamuslee001/5.43

Add release-notes/5.42.1.md

Add release-notes/5.42.1.md

Merge pull request #21826 from eileenmcnaughton/cancellabel

Fix translation regression in recurring edit workflow template

Regenerate `sql/civicrm_generated.mysql`

bin/regen.sh - Fix misnamed extension

The script tries to filter the list of extensions that wind up in the sample data-set.

The filter list targets `full_name` (ie long name), but some of the values are inaccurate.

APIv4 - Fix possible recursion during Entity::get()

Fix translation regression in recurring edit workflow template

Merge pull request #21824 from eileenmcnaughton/getfields

Use preferred entity name (Website not website)

Use preferred entity name (Website not website)

Merge pull request #21821 from totten/5.43-event-actres

(REF) Tokens - Populate "Event"/"Participant" without direct access to actionSearchResult

Merge pull request #21818 from alifrumin/rn5.43

[NFC] First Pass Release notes 5.43

Merge pull request #21819 from colemanw/APIv4EntityEvent2

REF - Fix loop in APIv4 Entity.get to only load ad-hoc entities once

Merge pull request #21812 from eileenmcnaughton/gdpr_tokens

dev/core#2907 fix for gdpr clobering contact tokens

(REF) ParticipantTokens - Simplify

The previous commit passes `eventId` through the front-door of `$context`,
so we shouldn't need as much special effort to bring it in the side-door.

(REF) Tokens - Populate "Event"/"Participant" without direct access to actionSearchResult

Overview
--------

When sending a scheduled remidner for a `civicrm_participant` record, it supports
tokens for both `{participant.*}` (`$context['participantId']`) and `{event.*}`
(`$context['eventId']`). This creates a special requirement to load the `eventId`
in addition to the `participantId`.

This patch changes the way in which `eventId` is loaded.

Before
------

In `ParticipantToken`, the token-evaluation-step for `{participant.*}`
tokens has a side-effect of copying `$context['eventId']` based on reading
`actionSearchResult` (although this appears indirect).

After
-----

The action-search returns `tokenContext_eventId`, which automatically maps
to `$context['eventId']`.

This means that the `eventId` will be set before evaluation begins.

[NFC] first pass at 5.43 release notes

REF - Fix loop in APIv4 Entity.get to only load ad-hoc entities once

dev/core#2907 fix for gdpr clobering contact tokens

This is a fix for contact tokens not rendering when gdpr tokens is installed.

Gdpr declares some addtional tokens with the namespace 'contact'. I am inclined to
agree this counts as 'naughty' and so while I have fixed the clobbering by
not processing contact-metadata tokens I have not fixed the enotices.

If we think it is non-naughty I can kill those - but I've already loosened
some strictness that I think had benefits in order to facilitate this

Merge pull request #21803 from colemanw/APIv4EntityEvent

[REF] APIv4 - Dispatch event during Entity.get

Merge pull request #21801 from eileenmcnaughton/case1

dev/core#2895 handle case ids passed via url

Merge pull request #21806 from totten/5.43-as-ctx

EntityTokens - Normalize data-loading via myEntityId (eg activityId)

APIv4 - Dispatch event during Entity.get

This allows extensions to modify the list of entities,
enabling "virtual" entities not based on php files.

dev/core#2895 handle case ids passed via url

(NFC) MailingQueryEvent - Update docblock

EntityTokens - getFieldValue() does not need to support actionSearchResult

EntityTokens - Normalize data-loading via `myEntityId` (eg `activityId`)

Before
======

EntityTokens has two protocols for fetching data:

1. For scheduled-reminders, hook into the main query. Add N-ary columns to select
   any fields used by tokens.
2. For everything else, expect a parameter `$row->context['activityId']`. Collect
   the `activityId`s and fetch the needful via APIv4.

This poses a few challenges, but TLDR it's duplicate functionality.  That
means that listeners who consume the `activityId` must check multiple
places.  It means that the data-loading could return slightly different data
(ie based on SQL query vs API data-loader).

After
=====

* EntityTokens has one protocol for fetching data: it must receive `activityId`s and then
  call APIv4 for the batch.
* For scheduled-reminders, EntityTokens uses a small adapter to ensure that `activityId`s
  are available.

Merge pull request #21783 from eileenmcnaughton/money

Use money formatting for currency in templates

Merge pull request #21796 from eileenmcnaughton/543

dev/core#2904 ensure skipStatusCal not passed from the order api

Merge pull request #21784 from eileenmcnaughton/api3

dev/core#2902 revert apiv3 handling of membership

dev/core#2904 Add tests for automatic date calculation for Memberships in Order API.

dev/core#2904 ensure skipStatusCal not passed from the order api

Merge pull request #21797 from agh1/5.43.0-releasenotes-initial

5.43.0 release notes initial run

5.43.0 release notes: oops forgot the synopsis

5.43.0 release notes: added boilerplate

5.43.0 release notes: raw from script

Use money formatting for currency

Merge pull request #21792 from colemanw/afformLoadFix

Afform - fix loading custom fields for search displays

Afform - fix loading custom fields for search displays

Set version to 5.43.beta1

dev/core#2902 revert apiv3 handling of membership

This reverts changes that were making api v3 date handling change if skipStatusCalc were set.

Merge pull request #21787 from eileenmcnaughton/case_email

Remove case token listing from contact email task

Merge pull request #21788 from eileenmcnaughton/selfsvc

Fix another instance of double token rendering

Merge pull request #21786 from eileenmcnaughton/pdf_etails

Remove call to legacy getTokenDetails

Merge pull request #21785 from eileenmcnaughton/sms

Remove extraneous call to legacy token processing in sms task

Fix another instance of double token rendering

Remove case token listing from contact email task

Remove call to legacy getTokenDetails

In this case it is just being used as a v3 api call

Remove extraneous call to legacy token processing in sms task

Merge pull request #21761 from eileenmcnaughton/contact

Contact tokens - use metadata & standardised rendering

Merge pull request #21781 from eileenmcnaughton/activity_email

dev/core#2901 Fix send email regression

Merge pull request #21782 from eileenmcnaughton/notice

Enotice fix

Fix contact tokens to use apiv4, new style tokens

Merge pull request #21780 from eileenmcnaughton/api4load

Fix contact tokens class to load using apiv4

Enotice fix

dev/core#2901 Fix send email regression

Fix contact tokens class to load using apiv4

[Ref] Extract getCacheKey

Merge pull request #21778 from eileenmcnaughton/extract

[REF] Extract function for re-use

[REF] Extract function for re-use

Merge pull request #21772 from totten/master-preview-dialog

(dev/mail#83) message_admin - Cleanup "Preview" dialogs (properly)

Merge pull request #21775 from colemanw/api4CreateRequestEvent2

APIv4 - Minor code cleanup to improve efficiency in CreateApi4RequestSubscriber

Merge pull request #21773 from totten/master-preview-locale

(dev/mail#83) message_admin - When using "Preview", use the 'locale' from the editing UI

Merge pull request #21774 from totten/master-draft-save

(dev/mail#83) message_admin - Save automatically when adding/removing drafts

Merge pull request #21764 from colemanw/afformLinkIcon

Afform Admin add link icon & template cleanup

APIv4 - Minor code cleanup to improve efficiency in CreateApi4RequestSubscriber

There is no need to check if class_exists() for custom entities.

(dev/mail#83) message_admin - Save automatically when adding/removing drafts

Merge pull request #21763 from eileenmcnaughton/trait

Remove stray token code

message_admin - Properly cleanup the `onPreview` listener

Use-case:

1. Open the editor a message template (eg `#/edit?id=17&lang=fr_FR`). This renders the edit screen.
2. Click on the "Preview" icon.
3. Close the "Preview" dialog.
4. Change the URL to navigate internally to another template (eg `#/edit?id=17&lang=de_DE`). This renders the edit screen again.
5. Click on the "Preview" icon.

Before:

* It subsequently opens two copies of the "Preview" dialog (each with
  slightly different options).  Initially, you see one dialog.  When that
  close, you will see the other dialog.
* As you proceed through closing the dialogs, you may get console warnings
  because the dialogs have the same name -- and they consequently trip-up
  each other.

After:

* It only opens one copy of the "Preview" dialog.

Technical Details:

* When rendering the setup screen (`#1`/`#4`), it registers a listener which
  will handle the "Preview" clicks.  But the listener is not properly
  unregistered.  Consequently, old listeners hang around. So the click at
  step `#5` calls both the old+new listeners, creating two dialogs.

Merge pull request #21771 from colemanw/api4CreateRequestEvent

[REF] APIv4 - Use event instead of hardcoded list to resolve entityName to className

message_admin - When using "Preview", use the 'locale' from the editing UI

(NFC) Edit.js - Remove old comment

(REF) message_admin - Split the steps of loading data and rendering

This will make it easier to inject different values while editing

GenericWorkflowMessage - Document `$tokenContext[locale]`

APIv4 - Use event instead of hardcoded list to resolve entityName to className

Typically an APIv4 entity name and class name are the same,
e.g. the "Contact" entity has the class `Civi\Api4\Contact`.
However, there are a few oddballs (CiviCase, CustomValue) and other extensions
may introduce their own oddball entities; this event allows them to do so.

Merge pull request #21756 from eileenmcnaughton/tt

Fix token regression - re-add support for empty tokens

Merge pull request #21768 from demeritcowboy/remove-wombats

[NFC/Unit Test] Remove wombats and bananas from sites/default/files/civicrm/custom

(NFC) TokenConsistencyTest - Small tweaks

1. You don't need to call evaluate() twice. It should be once after filling both rows+messages.
2. Add messages to clarify assertions

Merge pull request #21522 from eileenmcnaughton/flex

dev/core#2836 install flexmailer by default on new installs [token]

Remove token code from not-adopted pattern

Merge pull request #21767 from eileenmcnaughton/meta

Reference token metadata internally fix custom tokens label bug

Merge pull request #21766 from demeritcowboy/test-case-email

[NFC] Add test for new case email url

sql/civicrm_generated.mysql - Regen

dev/core#2836 install flexmailer by default on new installs

(REF) MailingTest - Relax assertions around `{action.*}`

* Only check for http://{action.*}` on legacy mailer
* Split assertions to be more readable
* Accept `h=` or `h=fakehash`

remove wombats and bananas

MailingTest - Slightly relax assertion about failure-message

In this error scenario, both BAO and Flexmailer complain. Flexmailer is slightly more specific in that its error reports
the actual field names (`body_text`, `body_html`) rather than a phony field `body`.