Merge pull request #21627 from colemanw/searchKitShoreditch

SearchKit - Style buttons for compatability with Shoreditch theme

Merge pull request #21630 from eileenmcnaughton/tte

Fix test to use postProcess

Merge pull request #21609 from colemanw/afformFix

Afform - fix loading embedded blocks

Merge pull request #21629 from eileenmcnaughton/test_cont2

Test cleanup -convert to use postProcess

Merge pull request #21625 from demeritcowboy/settings-dep

Setting::getItem - use deprecatedWarning instead of just logging

Fix test to use postProcess

Test cleanup -convert to use postProcess

noisier

SearchKit - Style buttons for compatability with Shoreditch theme

Merge pull request #21622 from eileenmcnaughton/test_cont

Test cleanup

Merge pull request #21570 from eileenmcnaughton/leg_contact

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

Merge pull request #21600 from colemanw/searchKitEquations

SearchKit - Add UI for arithmetic equations

SearchKit - Add UI for arithmetic equations

APIv4 - Restructure function params, add labels

Merge pull request #21605 from eileenmcnaughton/gen_up

 Add upgrade to replace legacy contribution tokens from message templates

Merge pull request #21586 from colemanw/stableRandom

SearchKit - A random stable genius

Test cleanup

This switches us to a less hacky way to interact with the form in some of the tests.

Our helpers now support us interacting with the forms in a way that better mimics
the real flow & allows us to call 'postProcess'

Merge pull request #21617 from demeritcowboy/convert-date

Remove unused function - convertDateFieldsToMySQL

Merge pull request #21618 from eileenmcnaughton/email

[Ref] Function signature cleanup

[Ref] Function signature cleanup

Merge pull request #21612 from civicrm/5.42

5.42

unused function

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

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

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

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

(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