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

Merge pull request #21760 from eileenmcnaughton/pseudo

Remove now-obsolete getPseudoTokens

Afform - Fix chainSelect to work with anonymous users

Remove static caching in deprecated function that seems to break tests

Merge pull request #21646 from MegaphoneJon/core-2874

fixes core#2874 - Search Builder on State/Province with Primary and IN

Remove now-obsolete getPseudoTokens

This is now only called from legacy code. The buildContributionArray path
is still a little used :-( but is tested
from CRM_Contribute_Form_Task_PDFLetterCommonTest
and hard-coding the array now it is settled is appropriate
to that code path's life expectancy

This code is no longer used in the Tokens classes and the functions that used it
from outside of there are now 'settled into their final state & waiting to die'
so a hard-coded list will see out their lives. They are marked now as deprecated

Clean up tests to call full postProcess for tasks

Afform - Rename submitFile param from entityName to modelName

$entityName is already a private property of APIv4 and has its own getter
which conflicted with a getter for a param named $entityName.
Better to just rename it.

Remove function duplicated in error

This function lives on & is called on the

Test & align empty token handling

Add token support for participant & membership tasks

Adjust getRows result to have schema as a key

It becomes too complex to handle in email trait otherwise

Move crazy submit pre-processing out of buildForm

Simplify getSubject

Merge pull request #21706 from eileenmcnaughton/token_metadata

Tokens - Hide more sophisticated tokens. Refactor to use "$this->tokenMetadata".

Merge pull request #21755 from civicrm/5.42

5.42

Merge pull request #21754 from agh1/5.42.0-releasenotes-final

5.42.0 release notes final updates

5.42.0 release notes: added late changes

Add release-notes/5.41.2.md

Add release-notes/5.41.1.md

Merge pull request #21753 from civicrm/5.42

Merge 5.42 to master

Merge pull request #21722 from colemanw/crmIconButton

Angular UI - Fix crmIcon to only add crm-button class to non-bootstrap buttons

Merge pull request #21740 from eileenmcnaughton/ev_cache

Flush metadata cache on event edit

Merge pull request #21743 from eileenmcnaughton/regress

Fix regression from enotice fixes

Merge pull request #21747 from totten/master-preview

(dev/mail#83) message_admin - Multiple improvements to the "Preview" dialog

Merge pull request #21731 from eileenmcnaughton/linerc

Call line item pre hook after tax amount is calculated

Merge pull request #21750 from colemanw/afformHoverFix

Afform - ensure dragging classes are removed when not sorting

Merge pull request #21752 from colemanw/searchKitPermissionFix

SearchKit - Fix anonymous access to running search displays

Merge pull request #21716 from jaapjansma/issue_2885

Fix for dev/core#2885

Merge pull request #21748 from mattwire/statusmorethan25

System status checks only shows 25 messages

Merge pull request #21676 from eileenmcnaughton/mail

dev/core#2866 Ignore preferred mail format - send to both

Merge pull request #21749 from colemanw/afformChainSelect

Afform - Fix chain-select of contry and address to work on SearchKit forms

SearchKit - Fix anonymous access to running search displays

This is the same fix as used by afform_civicrm_alterApiRoutePermissions()
to remove the barrier to entry for anonymous users.

Afform - ensure dragging classes are removed when not sorting

Afform - Fix chain-select of contry and address to work on SearchKit forms

Multiple fixes to the chainSelect mechanics
- Provides visual feedback when reloading options
- Works with defalut values from url args
- Works with multi-valued fields
- Works with searchKit joins

Fix for dev/core#2885

Added test

Added test

Allow for more than 25 system status checks to be loaded

Merge pull request #21599 from eileenmcnaughton/format

(dev/core#2638) Civi::format() - Add helper for formatting

message_admin (Preview) - Fix loop-around on "<<Previous" button

Before: If you open the new "Preview" pane and press the "<<Previous"
button, it loops to invalid position `-1`.

After: Loop-around works in both directions.

message_admin (Preview) - Add more tooltips to "Preview" dialog

message_admin (Preview) - Add drill-down for example data

Merge pull request #21744 from civicrm/5.42

5.42 to master

Merge pull request #21730 from alifrumin/rn5.42

[NFC] 5.42 First Pass at Release Notes

Fix regression from enotice fixes

Merge pull request #21735 from colemanw/tplFix

Fix possibly unefined $form variables in smarty templates

Merge pull request #21738 from agileware/ISSUE-2892

#2892 Replace event type token in action schedule changes event.event_type_id to participant.event_type_id:label. But should be event.event_type_id to event.event_type_id:label