Merge pull request #21668 from eileenmcnaughton/submit

[REF] Use getSubmittedValue rather than passed values

Merge pull request #21682 from eileenmcnaughton/notice

Another enotice fix

Merge pull request #21683 from eileenmcnaughton/notice2

And another one

Merge pull request #21684 from eileenmcnaughton/notice3

Another one bites the dust

Merge pull request #21686 from eileenmcnaughton/test_name

Rename test clas to reflect tested class name

Merge pull request #21687 from eileenmcnaughton/case1

[REF] Extract getRedirectUrl method

Merge pull request #21685 from eileenmcnaughton/email

Remove unused paramter

[REF] Extract getRedirectUrl method

Rename test clas to reflect tested class name

Remove unused paramter

Merge pull request #21671 from eileenmcnaughton/wf_name

Remove workflow_id reference

Another one bites the dust

And another one

Another enotice fix

Merge pull request #21678 from demeritcowboy/test-fileoncase

[NFC] Unit test for file-on-case

test for file on case

Merge pull request #21680 from eileenmcnaughton/gogo

Remove now-silly function

ang/crmMailing/Templates.js - Dropdown shouldn't show system workflow templates

These are now identified by `workflow_name` rather than `workflow_id`.

MessageTemplate - Fix "revert" for msgtpl's that set `workflow_name` sans `workflow_id`

Merge pull request #21679 from Detsieber/patch-2

Update SelfSvcUpdate.php

Remove now-silly function

Merge pull request #21677 from eileenmcnaughton/move

Return another function to the email trait

Update SelfSvcUpdate.php

Changing this text is necessary for internationalization, because otherwise we get an ambigous translation in some languages

Return another function to the email trait

Another function that is only used from one place & is unnecessarily complicated
because of the input & output paramters

Merge pull request #21648 from eileenmcnaughton/email_task

[REF] Copy another email trait function back to the trait

Remove workflow_id reference

Merge pull request #21472 from mattwire/contributionrecurmodifieddate

ContributionRecur modified_date should update to current_timestamp on update

Merge pull request #21403 from GValFr35/fix-session-reference-5.40

(dev/drupal#163) Session erroneously getting set to NULL on change

Merge pull request #21670 from eileenmcnaughton/dist

Add message admin to dist

Add message admin to dist

Merge pull request #21633 from eileenmcnaughton/notices_s

Fix header e-notices

Merge pull request #21590 from totten/master-msgtplui

dev/mail#83 - Import current revision of "Message Admin"

Merge pull request #21637 from eileenmcnaughton/cont_test

Tests -  update to use postProcess

Merge pull request #21455 from jaapjansma/issue_645

Fixed issue with sending from do-not-reply@domain address

Merge pull request #21661 from eileenmcnaughton/cleanup

Basic IDE cleanup on test class

ContributionRecur modified_date should update to current_timestamp on update

User getSubmittedValue rather than passed values

Merge pull request #21613 from eileenmcnaughton/update_from

Simplify getRecurFromAddress

Merge pull request #21665 from civicrm/5.42

5.42 to master

Merge pull request #21664 from jitendrapurohit/end_event-542

Fix missing value of End Adjustment column from Membership status page

Merge pull request #21662 from eileenmcnaughton/mem

Test fix

fix end_event_adjust key

Test fix

Basic IDE cleanup on test class

Fix header e-notices

This should solve enotices across a bunch of screens

Simplify getRecurFromAddress

This switches to use the code extracted from the other function

Merge pull request #21652 from eileenmcnaughton/cont_render_more

Remove call to the dreaded replaceMultipleContributionTokens

Merge pull request #21521 from eileenmcnaughton/member_tokens

dev/core#2832 Switch membership pdf to use token processor

Merge pull request #21658 from eileenmcnaughton/enotice

Enotice fix

Merge pull request #21659 from eileenmcnaughton/notice2

Enotice fixes

Merge pull request #21595 from agileware/CIVICRM-1563

Find Activities Search, the default search option of "With" criteria excludes Activities by default. Confuses End Users when 0 results are returned when criteria appears to be correct.

Merge pull request #21562 from jitendrapurohit/notify-user

Do not send email notification to the user if `notify` is not set in the params

Remove call to the dreaded replaceMultipleContributionTokens

Remove calls to replaceContributionTokens

Merge pull request #21657 from totten/master-wfvn

MessageTemplate - Deprecate `valueName`. Emphasize `workflow`. Consolidate converters.

Enotice fixes

Enotice fix

Merge pull request #21656 from eileenmcnaughton/role

Fix label for participant role

Merge pull request #21587 from eileenmcnaughton/badge_ev

dev/core#2834 Use event tokens for participant badges

(REF) WorkflowMessage - Emphasize the newer 'workflow' instead of 'valueName'

MessageTemplate - Allow 'workflow'/'valueName' as aliases. Consolidate alias bits.

The params given to `MessageTemplate::sendTemplate()` are evolving. But they are also
exposed via hook. Blerg.

This technically does two things:

1. It takes a couple of bits which are responsible for aliasing/transitioning specific
   params, and it moves them to a common `$sync()` function.

2. It deprecates `valueName` in favor of `workflow`, and it updates the hook test
   accordingly.

(REF) ExampleWorkflowTest - Add coverage for alterMailParams($params['valueName'])

(REF) hook_civicrm_alterMailParams.evch.php - Soften 'type' check

This changes some of the plumbing in how it interprets 'paramSpecs' - but
doesn't change anything substantive.

Before: You must specify a 'type' constraint for every param.
After: You may omit the 'type' constraint for some params.

Before: If you specify a nullable string with a regex, then null-values will fail.
After: If you specify a nullable string with a regex, then null-values will pass.

(REF) CRM_Utils_Array - Add pathSync() helper

Fix label for participant role

dev/core#2834 Use event tokens for participant badges

Overview
----------------------------------------
Convert the rest of event badge token replacement to use
the token processor

Before
----------------------------------------
Testing and conversion had been done for contact
and participant tokens within the event badges - but
for the event tokens tests had been added but
the conversion was pending solving date formatting

After
----------------------------------------
Event badges use the token processor for event tokens

At this point all token processing in core for
participant or event tokens is done in the token
processor & attention can switch to cleaning that up

Technical Details
----------------------------------------
The challenge switching over event tokens was that badges used
a custom date format and we needed to figure out how to support
that first.

This also standardises the event.event_id token to participant.event_id
and updates it in the db for scheduled reminders (badges were using
'event.id' format & reminders the event.event_id format).

There are still some tokens on the scheduled reminders to
standarsise and a loading challenge to fix - but this
concludes the badge portion of fixing up event tokens

Note that the tokenConsistencyTest is the main test cover
for event tokens and it turns out we are not quite there
on the dates based on https://github.com/civicrm/civicrm-core/pull/21584

Merge pull request #21610 from eileenmcnaughton/rend

(REF) MessageTemplate - Reverse renderTemplate()<=>sendTemplate() call

Merge pull request #21423 from demeritcowboy/dompdf-options

dev/core#2126 - Allow setting dompdf options, e.g. an external font folder

Merge pull request #21655 from kurund/patch-3

typo fixes

Merge pull request #21653 from eileenmcnaughton/ref

[REF] stop passing form into createActivities

typo fixes

[REF] stop passing form into createActivities

This function is now an OO function - so we can make it protected & stop passing form in

Merge pull request #21615 from totten/master-event-test-mail

tests/events/*.php - Enforce general compliance with hook/event signatures

MessageTemplate - Restore filtering of $params

The prior commit had the effect of dropping any filtering that happend on `$params`.

Merge pull request #21651 from eileenmcnaughton/tpl

Set default for civicrm_message_template.is_reserved

Reverse Render->send to Sent->render

We have a gazillion tests on this code ...

Merge pull request #21640 from eileenmcnaughton/edge

dev/core#2870 Fix upgrade error with rpow installed

Merge pull request #21647 from colemanw/coalesce

APIv4 - Automatically coalesce potentially null field values in equations

Merge pull request #21584 from eileenmcnaughton/new_date

Extend token date handling to 'most' date fields

Merge pull request #21261 from totten/master-test-jwt

HttpTestTrait - Allow one to easily authenticate subrequests

Set default for civicrm_message_template.is_reserved

APIv4 - Automatically coalesce potentially null field values in equations

Merge pull request #21650 from demeritcowboy/fileoncase

Fix extra apostrophe added in 21636

Merge pull request #21649 from eileenmcnaughton/sub

Test regression fix

Merge pull request #21433 from jmcclelland/import-preferred-communication-method

avoid back trace when importing preferred communication method

Merge pull request #21643 from colemanw/afformUrlArgs

Afform - ensure default value data type matches input type

Test regression fix

apos

[REF] Copy another email trait function back to the trait

The only core use for this function is via the task & the signature is cludgey
This copies it back & deprecates the no-longer-in-use code

Merge pull request #21636 from eileenmcnaughton/not

Enotice fix

Merge pull request #21632 from eileenmcnaughton/notice2

Enotice fixes

Merge pull request #21639 from eileenmcnaughton/nofat

enotice fixes in fatal error template

Merge pull request #21623 from eileenmcnaughton/email

Remove handling for empty required field

Afform - ensure default value data type matches input type

Merge pull request #21624 from eileenmcnaughton/no_good

dev/core#2629 show only contribution statuses on contribution form

Merge pull request #21641 from eileenmcnaughton/contact

Remove over-copied function

Remove over-copied function

In the splitting out of Contact_Tokens from TokenSillyAbreviationSubscriber
I copied over this function - but it was an incorrect
duplication - it is not called

Merge pull request #21626 from eileenmcnaughton/contact

[REF] Separate contact tokens from 'tokenCompatSubscriber'

dev/core#2870 Fix upgrade error with rpow installed

enotice fixes in fatal error template