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"

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.

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

(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

Merge pull request #21530 from eileenmcnaughton/badger2

Convert event badges to use token processor

Convert event badges to use token processor

This adds the token processor class for participants and
switches the one place where we currently resolve participant
tokens to use it.

In the process I upgrade the badge tokens with a syntax change and
drop a few more that are actually
a) weird / seemingly accidentally exposed and
b) not really part of the participant entity

ie event template title & default role id

With participant tokens we have been fortunate not
to let a non-standard implementation go out
so it's only the event badges which are
a bit obscure / less used / less likely to be
blindly used. I have added token replacement
but I think it will be all very edge case

Merge pull request #21520 from eileenmcnaughton/badger

Participant tokens - remove / replace unsupportable from badges (conversion preparation)

Merge pull request #21537 from eileenmcnaughton/ev_token2

Use the processor function to get event tokens

Merge pull request #21552 from eileenmcnaughton/test_clean

Cleanup on test tear downs

dev/core#2493 Add support for money fields to

This ensures that any money fields retrieved by  are
in a machine usable format.

We've been switching to this function rather than 'passing around arrays'
but we need to ensure this function is returning clean money to
prevent regressions.

Merge pull request #21550 from demeritcowboy/template-missed

Missed message template updates from 5.39/5.40

[REF] Remove illusion of looping

This foreach actually only copes with 1 values. Making that clearer makes it more readable

Merge pull request #21490 from eileenmcnaughton/case_email

dev/core#2554 Remove code to assign contact to the template in sendEmail

Cleanup on test tear downs

Merge pull request #21509 from JMAConsulting/core_2820

dev/core#2820: Deduping produces bad results after a rule change

Merge pull request #21508 from eileenmcnaughton/date

Add support for CiviCRM date formats in crmDate

Merge pull request #21493 from eileenmcnaughton/pdf

dev/core#2839 Fix addition of _preview to filename

Merge pull request #21547 from agileware/CIVIBLD-280

View Contribution page, move the Done button to the last button, currently shown in the middle

Merge pull request #21549 from artfulrobot/artfulrobot-htmlencoding-api-output

Add test and fix for API4 failing to decode strings stored as HTML

Merge pull request #21546 from agileware/CIVIBLD-279

Replace "Print Invoice" and "Print Invoice and Credit Note" buttons with label "Download Invoice" and "Download Invoice and Credit Note"

Merge pull request #21533 from colemanw/relEdit

SearchKit - Fix in-place edit of relationships and dates

missed from 5.39/5.40

APIv4 - Decode output of write operations

CiviCRM regrettably stores most strings as HTML in the database.
This fixes the output of APIv4 write operations to decode them and return non-html

Merge pull request #21548 from eileenmcnaughton/cleanup

Test tearDown fixes

Test tearDown fixes

Fixes places where parent:tearDown not called and civicrm_file &
civicrm_entity_file not truncated

Also note - increasing use of fail() rather than allowing
exceptions to bubble up in test functions in order
to not let tearDown be such a distraction & require annotation

Merge pull request #21543 from eileenmcnaughton/notice3

Tpl notice fix

Merge pull request #21545 from eileenmcnaughton/tpl5

tpl enotice fixes on extensions page

CIVIBLD-280 View Contribution page, move the Done button to the last button, currently shown in the middle