Merge pull request #21675 from colemanw/commentCleanup

NFC - Update code comments

Merge pull request #21689 from eileenmcnaughton/not_another

Enotice tpl fix

Merge pull request #21681 from MegaphoneJon/core-2881

fixes core#2881 - APIv3 replace only replaces the first 25 records

Merge pull request #21693 from eileenmcnaughton/object

Function cleanup following move to non-static function

Merge pull request #21702 from eileenmcnaughton/notice

enotice fix

Merge pull request #21691 from colemanw/sqlExprTitles

APIv4 - Give every SQL expression a title

enotice fix

Merge pull request #21698 from totten/master-msgadm-title

message_admin - Add titles/hints to hyperlinks

Merge pull request #21666 from eileenmcnaughton/ev

 dev/core#348 Participant tokens on scheduled reminders

Merge pull request #21694 from totten/master-msgtpl-wfnull

MessageTemplate API - Fix saving of templates with workflow_name sans workflow_id

message_admin - Add titles/hints to hyperlinks

MessageTemplate API - Fix saving of templates with workflow_name sans workflow_id

Overview
--------

This fixes a bug when saving (updating) a `MessageTemplate` record that
involves a `workflow_name` and no `workflow_id`. It is an alternative to #21674.

Steps to  reproduce
-------------------

Suppose you read a MessagTemplate with values:

```php
$values = ['workflow_name' => 'foo', 'workflow_id' => NULL, ...]
```

Then you save it back:

```php
civicrm_api4('MessageTemplate', 'update', [
  'values' => $values
]);
```

Before
------

The `update` raises an exception.

After
-----

The `update` works.

Function cleanup following move to non-static function

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

APIv4 - Give every SQL expression a title

Fixes a bug in Afform where it would crash trying to get the title of SQL Equations
on search display forms.

Enotice tpl fix

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`

fixes core#2881 - APIv3 replace only replaces the first 25 records

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

NFC - Update code comments

First docblock to use the new array shapes annotation

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

 Participant tokens on scheduled reminders (wip- upgrade script needed at minimum)

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