Merge pull request #21832 from civicrm/5.43

5.43

Merge pull request #21807 from braders/feature/replace-spacer-img-elements

Remove use of spacer.gif img elements in CRM_Utils_Weight

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

5.43 to master

Merge pull request #21824 from eileenmcnaughton/getfields

Use preferred entity name (Website not website)

Use preferred entity name (Website not website)

Merge pull request #21823 from civicrm/5.43

5.43 to master

Merge pull request #21815 from demeritcowboy/case-token

dev/core#2895 - Failing test demonstrating case token not resolved

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

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

Merge pull request #21804 from eileenmcnaughton/pdf

[REF] Paramter simplification

test demonstrating failure to resolve token

Merge pull request #21822 from civicrm/5.43

5.43 to master

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

Merge pull request #21817 from civicrm/5.43

5.43 to master

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

Update to remove use of spacer.gif in ordering component

Merge pull request #21800 from civicrm/5.43

5.43

Make hardcoded strings translatable.

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

Add alt text to spacer elements

All HTML img elements should contain an alt attribute.
For decorative or spacer elements this alt text should be a blank string,
which indicates to assistive technologies that the image should not be announced.

Longer-term these spacer gifs could be replaced with empty <span> elements.
Inline-block css has been added to allow for this change in future,
however, the HTML element has not been changed for now to avoid breakage with non-default themes
(e.g. civicrm-admin-utilities WordPress plugin which replaces the default css/civicrm.css file)

(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

[REF] Paramter simplification

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