Merge pull request #21608 from eileenmcnaughton/email

[REF] Return the sendEmail function to it's owner

Merge pull request #21469 from mattwire/updatesubscriptionform

Fix display of subscription frequency on 'Update Subscription' form

EventChecker - Activate during unit-tests

EventChecker - Add base-class and loader

Merge pull request #21594 from agileware/CIVICRM-1844

Remove all references to safe_mode, DEPRECATED as of PHP 5.3.0 and REMOVED as of PHP 5.4.0

Merge pull request #21601 from eileenmcnaughton/542

dev/core#2861 Saved search loading fix

Afform - fix loading embedded blocks

dev/core#2861 Saved search loading fix

Merge pull request #21597 from seamuslee001/flexmailer_unicode

CRM-19236 - Fix Flexmailer extension to support unicode urls

Extend token date handling to 'most' date fields

Note I hit an error locally in tests on
variant of this - so throwing against jenkins as my first stop

Merge pull request #21602 from eileenmcnaughton/broke

Remove broken code

Add upgrade to token style

Return the sendEmail function to it's owner

This sendEmail function is only called from one place in core and it is not 'generally useful' having
an awful parameter set. This PR moves it back to the class that 'owns' it - which will
allow us to undo all the work of building up that parameter set
and make it possible to support tokens for other entities than those already mangled in.

I would normally add a noisy deprecation notice once a function becomes unused in
core but since that has been done to the pdf task this release I've left this
deprecation a bit quieter for now.

Note that I cleaned up the tokens handled here before deprecating so we
could get rid of those calls fully

Under the OO structure it becomes easier to add the missing token options
- membership & participant - but the business of 'one email per person, &
just grab the tokens from the last entity' is messing with
my head a bit. That's the next bit....

Merge pull request #21565 from eileenmcnaughton/email

dev/core#2851 Fix send email task contribution tokens to the processor

Afform - Accept default field values from the url

On a submission-type form, the args will be prefixed with their entity names,
e.g. civicrm/my-afform#?Individual.first_name=Bob

On a search form the args don't need a prefix and just use the name of the field

 Add upgrade to replace legacy contribution tokens from message templates

Under legacy token replacement these 4 work
contribution.contribution_status
contribution.contribution_status_id:label
contribution.campaign
contribution.campaign_id:label

But only the label ones work with the processor. This upgrades the
    old ones our of the message templates table

Remove broken code

I don't even know what this is supposed to do - but since the function
has been missing for our entire git history it must
be unreachable.

https://github.com/civicrm/civicrm-core/blob/4d5c2eb510a6be0017085d1f8dae09495e28e74b/CRM/Contact/BAO/SearchCustom.php

dev/core#2851 Fix send email task contribution tokens to the processor

Merge pull request #21598 from agileware/CIVICRM-1846

Find Activities Search, default search option for Activity Text is set to "both" in the code, but should actually be set to 6 which is the value for "both"

CIVICRM-1846 Find Activities Search, default search option for Activity Text is set to "both" in the code, but should actually be set to 6 which is the value for "both"

[REF] CRM-19236 Fix Flexmailer extension to support unicode urls

CIVICRM-1563 Find Activities Search, the default search option of "With" criteria excludes Activities by default and confuses end users when 0 results are returned

CIVICRM-1844 Remove all references to safe_mode, DEPRECATED as of PHP 5.3.0 and REMOVED as of PHP 5.4.0

Merge pull request #21588 from eileenmcnaughton/cancel

Add test to cancel subscription form

Add test to cancel subscription form

Merge pull request #21585 from eileenmcnaughton/compat

Fix test

Merge pull request #21538 from mattwire/subscriptioncleanup

Clean up UpdateBilling/UpdateSubscription/ContributionRecur to use getters

Fix test

Fixes test logic I accidentally voided

Merge pull request #21457 from colemanw/searchDisplayAfformDelete

Fixes dev/core#2824 - Handle related Afforms when deleting Search Displays

Merge pull request #21577 from jmcclelland/display-backend-recur-options

ensure recur options are present on backend cc contribution form.

Merge pull request #21484 from JKingsnorth/price-field-id-css

Add unique class to price field template

remove type hint to return bool - we are no longer using the ret value.

ensure recur options are present on backend cc contribution form.

We seem to be adding an extra check to see if the given
payment processor has any credit card fields before including the
recur option on the backend contribution page. But some processors,
like Stripe, insert the fields via javascript. It should be enough
to just test for the existence of a payment processor that supports
recur to run the block.

Also, no other code is checking the return value of buildPaymentForm
so removing it.

SearchKit - Keep randomly sorted results stable across pages

SearchKit allows ORDER BY RAND() but this can cause unexpected reshuffling
when using the pager, editing-in-place or bulk-updating records.

The solution is to generate a seed on the client-side when the display initializes,
and re-use it every time results are fetched. This keeps the order stable,
only reshuffling when the browser reloads the page.

Add standardized params for billingStateProvince and billingCountry

Clean up UpdateBilling/UpdateSubscription/ContributionRecur to use getters

Merge pull request #21525 from eileenmcnaughton/cont_dep

dev/core#2851 Block use of legacy style contribution tokens

Merge pull request #21580 from demeritcowboy/tokerr

dev/core#2856 - Token "filter" can be null so filter[0] doesn't exist

Merge pull request #21568 from eileenmcnaughton/labels

dev/core#2814 fix tokenCompat to be consistent with unresolved tokens

null

Merge pull request #21579 from eileenmcnaughton/notice

Enotice fix

Remove old comment

Merge pull request #21574 from colemanw/searchPrimary

Fixes dev/core#2852 - Add is_primary as a default SearchKit filter

(REF) TokenCompatSubscriber - Simplify expression that replaces unknown tokens

Additionally, this uses the regex from `TokenProcessor` instead of the regex from `CRM_Utils_Token`.
These differ in whether they capture the '|filter' expressions.

(REF) TokenProcessor::visitTokens() - Increase visibility. Refine comments+tests.

Merge pull request #21564 from lcdservices/dev-core-2850

dev/core#2850 update_email_resetdate consider opt out on hold

Merge pull request #21572 from colemanw/afformEvents

Afform - Add getters/setters to the AfformSubmitEvent class

Enotice fix

Merge pull request #21551 from colemanw/sqlEquation

APIv4 - Add support for sql equations

Merge pull request #21569 from totten/master-datets

CRM_Utils_Date - Month and day names should match active locale

Fixes dev/core#2852 - Add is_primary as a default SearchKit filter

When joining Contacts to emails, addresses, phones and IMs this will
automatically add is_primary=1 as a default search criteria.

dev/core#2814 fix tokenCompat to be consistent with unresolved tokens

This fixes the tokenCompat subscriber to replace unresolved tokens with a
blank string in a consistent way.

Prior to this it would crash if smarty was enabled but not all tokens
were resolved & print unresolved tokens if smarty was not enabled.

The inconsistencies appear to be due to 'separate evolution' rather than '*reasons*'

Merge pull request #21557 from eileenmcnaughton/log

Fix bug where log hook is ignored on custom field create

(NFC) TokenProcessorTest - When formatting dates, month and day names should be localized

CRM_Utils_Date - Month and day names should match active locale

Merge pull request #21531 from eileenmcnaughton/now

Add {domain.now}, supporting |crmDate

Merge pull request #21576 from demeritcowboy/ufval

UFMatch - E_NOTICE when nonexistent uf_id

APIv4 - Add support for sql equations

Fix bug where log hook is ignored on custom field create

It is possible to specify that various tables should not
result in a log table via the `alterLogTables` hook.

This is useful for excluding tables of low information
value or tables like summary fields that are calculated
values. However, when altering custom fields this
hook's output was being ignored as it was going through
the specific path of
which did not have awareness of this hook.

This code path is the primary use for this function.

It's also called via createMissingLogTables - but in
that case the calculation is already done. It is not
elsewhere in git universe and the return value
is never used, so I removed it

Merge pull request #21578 from civicrm/5.42

5.42 to master

Merge pull request #21575 from agh1/5.42.0-releasenotes-initial

5.42.0 release notes initial run

don't crash for nonexistent uf_id

Afform - Add getters/setters to the AfformSubmitEvent class

This makes it easier for 3rd-parties to interact with the event.

5.42.0 release notes: added boilerplate

5.42.0 release notes: raw from script

Merge pull request #21554 from eileenmcnaughton/loop

[REF] Remove illusion of looping

dev/core#2850 add comment

dev/core#2814 Add support for preferred syntax for contact tokens

This maintains support for all currently advertised tokens. But it
switches the advertisement to our preferred style (matching the db field,
specifically denoting if the label is desired.) Tests for both styles are added.

This is a step towards switching to apiv4. An important difference with v4 is
that we can distinquish between 'not fetched' and 'empty' - saving us
from extra queries when it is not clear - which will in turn
allow us to switch greeting
processing over & still support the performance tweaks via
that method

Do not notify user if not set in the params

(NFC) TokenSmartyTest - Add some more examples, mixing token-filters and Smarty-filters

TokenProcessor - Filter args should require quotes (to match Smarty notation)

Merge pull request #21563 from eileenmcnaughton/ev_toke

Initial test on event tokens

Fix tokens to pass filters through

TokenProcessor - render() and addMessage() should use same regex to find tokens

TokenConsistencyTest - Fix assertion

This is a new/recent assertion. Not passing. Looks different from other/similar assertions.

TokenConsistencyTest - Use a valid timestamp

DomainTokens - {domain.now} should abide by mocked times

Add now token

Merge pull request #21566 from eileenmcnaughton/render

Remove deprecated code lines

Test cleanup

Fixes passing null to event create. Remove some tests duplicated by syntaxconformance

Merge pull request #21555 from eileenmcnaughton/money

dev/core#2493 Add support for money laundry in `getSubmittedValue`

Merge pull request #21430 from mattwire/flushcache

flushCaches should respect permitCacheFlushMode. Also flush caches which have a NULL cache_date

Remove deprecated code lines

These have been deprecated for around 8 months
, are not used in core, and are in code where
5.43 will already require extra testing so let's remove
so we can simplify

Initial test on event tokens

Note there is an issue to do with caching affecting the custom token but
it's out of scope on this test

dev/core#2850 update_email_resetdate consider opt out on hold

Merge pull request #21553 from eileenmcnaughton/email2

[Ref] Extend email trait test, process more sanely

Merge pull request #21494 from eileenmcnaughton/compat

dev/core#2814 TokenCompatSubscriber - evaluate contact tokens during civi.token.eval phase

Merge pull request #21374 from ufundo/event-custom-tokens

Fix event custom field tokens in scheduled reminders

Merge pull request #21560 from demeritcowboy/report-query-string

CiviReport - avoid error in test environments when using built-in php web server

[Ref] Extend email trait test, process more sanely

[REF] Extract getFromValues, add test

This extracts the code used to get the from values, moves the default setting
to the default and adds a test.

The goal is actually to add a test on the submit function
but it's proving to be challenging so this at least gets the
default setting tested.

Note that this code used to be on a static class, not a trait, and
some stuff is in weird places - in particular stuff that
should be in setDefaults and postProcess is happening on
preProcess. However, most
of the classes that override this
trait do not do much more than call the trait so that helps at
least

Merge pull request #21561 from civicrm/5.42

5.42 to master

Merge pull request #21559 from eileenmcnaughton/542

dev/core#2843 push new link to the end

avoid error in test environments using built-in php web server

Fix signature_html to return html

This seems to be more in line with processor expectations

Add enough hack-handling to make this PR not change outputs

dev/core#2814 TokenCompatSubscriber - evaluate contact tokens during civi.token.eval phase

m

Merge pull request #21558 from eileenmcnaughton/no_gender

Stop testing for gender_id output that was described as 'legacy' years ago

dev/core#2843 push new link to the end

As discussed in https://lab.civicrm.org/dev/core/-/issues/2843
this makes the UI change less for users & reduces confusion

Stop testing for gender_id output that was described as 'legacy' years ago

This token is not 'advertised' in the widget. I think removing this