Merge pull request #21817 from civicrm/5.43

5.43 to master

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 #21814 from civicrm/5.43

5.43 to master

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

Merge pull request #21810 from braders/feature/missing-translations

Make hardcoded strings translatable.

Merge pull request #21800 from civicrm/5.43

5.43

Make hardcoded strings translatable.

Wraps previously hardcoded strings in {ts} in Smarty .tpl files.

(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

Merge pull request #21799 from eileenmcnaughton/refs

Do not find references for empty values

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 #21794 from eileenmcnaughton/sms

Remove another call to getTokenDetails

Merge pull request #21798 from demeritcowboy/sid-notice

$sid doesn't get set on upgrade runFinish

Do not find references for empty values.

Merge pull request #21620 from colemanw/purifier2

Use HTMLPurifier from composer instead of packages

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

5.43.0 release notes initial run

sid doesn't get set

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

Remove another call to getTokenDetails

The goal is to make the tokenValues hook uncalled outside the
token processor & hence redundant. We were trying to do that in 5.43
but this call is not using the results anyway so I guess
putting in master is immaterial to that goal

Merge pull request #21792 from colemanw/afformLoadFix

Afform - fix loading custom fields for search displays

Afform - fix loading custom fields for search displays

Merge pull request #21791 from braders/fix-aria-atomic-bad-quotes

(NFC) Fix quote style for valid HTML - notifications.tpl

Fix quote style for valid HTML - notifications.tpl

Replaces aria-atomic=”true” with aria-atomic="true".

Previously the attribute was wrapped in Unicode "Right Double Quotation Mark" (U+201D) symbols,
instead of the standard "Quotation Mark" (U+0022) supported by HTML.

This meant the HTML was not valid, and may have led browsers to parse the aria-atomic inconsistently.

Set version to 5.44.alpha1

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`.

(NFC) MailingTest - Swap quotes. Switch assertFoos(). Declare return types.

Merge pull request #21717 from jmcclelland/notedatefield

ensure blank note date in UI defaults to current date.

add test

Reference token metadata internally

This references the token metadata rather than the metadata retrieved from the api
call. Since the former is cached it should save a look up -
albeit not when prefetch is happening, at this stage

Merge pull request #21660 from eileenmcnaughton/dep

[REF] Move getCustomFieldTokens to the tokens class, annotate more deprecations

Merge pull request #21669 from MegaphoneJon/event-64

fixes event#64: Send event confirmation to submitted email address

Merge pull request #21704 from eileenmcnaughton/date_regression

Master only regression - fix empty token date

ensure blank note date in UI defaults to current date.

See https://lab.civicrm.org/dev/core/-/issues/2884

Move getCustomFieldTokens to the tokens class

This deprecates (quietly) another function on CRM_Utils_Token. It's a pretty small function -
the sort you'd just copy & paste if you wanted to use outside of core but
I didn't add noise. The only core place that calls this is the tokenTrait
- which is 'used' by activity tokens - but only for the handling of _N_
which I think was actually not supposed to have been included in the merge.

Anyway - I'm leaving that question for later because I'm not sure if the token
trait is used outside of core or how our final interface will look

Merge pull request #21765 from colemanw/arrayShapes

[NFC] Afform - Add array shape annotations to FormDataModel

Afform - Add array shape annotations to FormDataModel

Merge pull request #21759 from colemanw/afformApiFixes

Afform - Fix chainSelect to work with anonymous users

Merge pull request #21705 from eileenmcnaughton/prefix

dev/core#2650 switch contact.individual_prefix tokens to contact.prefix_id:label (former still works)

Merge pull request #21745 from eileenmcnaughton/e_build

Support participant & membership tokens in send-email tasks

Afform - Add title to GUI editor page

Afform Admin add link icon && template cleanup

Merge pull request #21758 from eileenmcnaughton/contactv4

Remove function duplicated in error