contact with email might not exist

Merge pull request #21437 from demeritcowboy/dashboard-fail

dev/core#2829 - unit test for nonempty dashboard

test for nonempty dashboard

Merge pull request #21399 from eileenmcnaughton/mgd

dev/core#2823 Extract code to load the declarations and call from the constructor

Merge pull request #21400 from eileenmcnaughton/val

dev/core#2823 Move validation into validation function

Merge pull request #21434 from demeritcowboy/upgrade-text

dev/core#2828 - Make up for message template text version update from 5.20 that never happened

Merge pull request #21211 from colemanw/onDeletePrimary

[REF] Move Email, Address, etc. is_primary handling on delete to a hook (dev/core#2757)

better late than never

Merge pull request #21424 from colemanw/apiJoinBridgeCustomFields

SearchKit - Support custom fields in bridge join entities

Merge pull request #21431 from colemanw/searchTableAlign

SearchKit - Fix alignment in table headers

Merge pull request #21198 from colemanw/fkConstraint

[REF] Mapping - Check FK on delete with SQL instead of PHP (dev/core#2757)

SearchKit - Fix alignment in table headers

Mapping - Check FK on delete with SQL instead of with PHP

Move Email, Address, etc. is_primary handling on delete to a hook

Previously this was all done in a delegated function.
This moves that logic to a hook listener.

Merge pull request #21426 from eileenmcnaughton/api4

Replace expensive query with cheaper one

Replace expensive query with cheaper one

Rather than retrieve everything with a legacy function, just get what we want....

Note I commented on how silly I think it is but no change to outcome

Merge pull request #21416 from eileenmcnaughton/dom_toke

[REF] Extract function to getDomainTokens

Merge pull request #21408 from colemanw/bumpMinUpgradeVer

Bump minimum upgradable version to 4.5.9

Merge pull request #21425 from eileenmcnaughton/error

Remove error handling for core error being returned

Merge pull request #21414 from eileenmcnaughton/greet

dev/core#2814 Remove redundant call to `replaceContactTokens`

Merge pull request #21422 from demeritcowboy/smarty-find-activities

Smarty notices on Find Activities results page

Remove error handling for core error being returned

This would be throwing an exception - also it isn't an error situation

SearchKit - Support custom fields in bridge join entities

This fixes bridge joins in APIv4 to allow selecting custom fields that belong to the bridge entity
as if they were part of the joined entity. This was already working for core fields.

smarty notices

Remove redundant call

Merge pull request #21418 from eileenmcnaughton/toke2

[REF] Remove cruft

Merge pull request #21421 from demeritcowboy/tag-table

Smarty notice on Find Activities / Find Cases

smarty notice

Merge pull request #21417 from eileenmcnaughton/cont_mode

Follow up on removing contributeMode from templates

Merge pull request #21419 from civicrm/5.42

Merge 5.42 to master

Merge pull request #21411 from colemanw/fixSearchKitAugmentSelectClause

SearchKit - Fix aggregated joins

[REF] Extract function to getDomainTokens

This caches the whole token set by domain+locale+html rather than just part.

I am thinking about where this should do - on the BAO_Domain class?
Or on a token processor class (eventually it would only be
called from a token processor class)

Test cover in 'BAO_MessageTemplateTest.testDomainTokens'

[REF] Remove cruft

As the comment says - it's weird. Also I found the related code has been removed

Follow up on removing contributeMode from templates

Per https://github.com/civicrm/civicrm-core/pull/21059
its a really long time since this was in the stock templates

Merge pull request #21059 from eileenmcnaughton/contribuion_mode

dev/core#2748 Remove unused token assigns

dev/core#2823 Extract code to load the declarations and call from the constructor

The declarations are only used in object context so it makes sense to load
them in the constructor. In addition they are ALWAYS loaded except in test usage
(it does seem a bit silly having the option to pass them in only for tests
but we can ignore that for now - I commented it)

dev/core#2823 Move validation into validation function

As the code comments suggest the handling of a module being unrecognised sould
be handled in the validate not the enable function

Merge pull request #21412 from eileenmcnaughton/protected

dev/core#2823 update function visibility to protected

Merge pull request #21174 from mattwire/unsubscribemessaging

Mailing Unsubscribe Form: Show if email is already unsubscribed

dev/core#2748 Remove unused token assigns

dev/core#2823 update function visibility to protected

This makes some internal functions 'protected'. I did a universer search for them.

Making them protected means that when I look to clean them up it is
clear that we only need to worry about how the class
interacts with them

Merge pull request #21410 from demeritcowboy/smarty-individual

Smarty warning about missing contact subtype on New Individual

Merge pull request #21409 from demeritcowboy/double-root

Remove duplicate global declaration in civicrm.settings.php template

Merge pull request #21405 from colemanw/tableStyle

SearchKit & Afform - Use Bootstrap table styles on admin screens

SearchKit - Exclude links from aggregated joins

subtype might legitimately be empty

remove duplicate declaration

SearchKit - Partial backport of 4afe3e

Bump minimum upgradable version to 4.5.9

Merge pull request #21406 from demeritcowboy/comma

[search kit] Semicolon vs comma

comma

Merge pull request #21404 from colemanw/fixSearchDeleteConfirm

Fixes dev/core#2826 SearchKit - Fix delete search confirm dialog

Afform - Use bootstrap table classes on admin table

SearchKit - Use bootstrap table classes on admin screens

SearchKit - Fix delete search confirm dialog

Fixes dev/core#2826

Merge pull request #21397 from colemanw/tableStyles

SearchKit - Support bootstrap styles in search display tables

Merge pull request #21381 from eileenmcnaughton/ent

dev/core#2817 Block submission of pdf with text including token format that is ambiguous with the processor

Merge pull request #21394 from colemanw/debounceGetResults

SearchKit - Prevent race conditions in search display loading

Merge pull request #21388 from colemanw/searchKitFilter

SearchKit - Filter search listing by creator

dev/core#2823 Move validation into validation function

As the code comments suggest the handling of a module being unrecognised sould
be handled in the validate not the enable function

Block submission with token format that is ambiguous with the processor

Merge pull request #21398 from demeritcowboy/basic-email-test

[NFC] Put back email test without tokens and enhance a little

add test without tokens

SearchKit - Support bootstrap styles in search display tables

Merge pull request #21396 from seamuslee001/fix_upgrade

Fix Upgrade step to ensure all fields are converted correctly as well

Merge pull request #21395 from eileenmcnaughton/mgd

dev/core#2823 Make protected functions non-static

Merge pull request #20995 from ixiam/dev-import-phone-extension

Import csv-Fix not importing extension in phones

Fix Upgrade step to ensure all fields are converted correctly as well

dev/core#2823 Make protected functions non-static

It's really hard to see why this class uses a mix of static & non-static functions
for internal (protected) functions. I suspect *history*. This makes
them all non-static

SearchKit - Filter search listing by author, adding support for OR in filters

SearchKit - Prevent race conditions in search display loading

- Increases debounce timeout from 100 to 800 ms
- Internally tracks the requests to ensure that later requests always take priority

Merge pull request #21186 from demeritcowboy/multiint

dev/core#2772 - Don't crash for custom fields of type int that are multi-select

Merge pull request #20822 from seamuslee001/use_product_api_form_hook

[REF] Update Product Create to use hooks and also switch the manage p…

Indicate if email is already unsubscribed.

Merge pull request #21389 from seamuslee001/relationship_cache_table_fix

[REF] Add in upgrade step to align the relationship cache table colla…

Merge pull request #21380 from eileenmcnaughton/case_opts

dev/core#2817 towards deprecating legacy `replaceCaseTokens`

Merge pull request #21386 from seamuslee001/test_sendemail

[NFC] Extend send email test to cover for contact tokens being replaced

Merge pull request #21390 from seamuslee001/function_type_hints_upgrade

[NFC] Add in type hints to template upgrade functions

[NFC] Add in type hints to template upgrade functions

[REF] Add in upgrade step to align the relationship cache table collation and charset with the other tables

Merge pull request #21382 from mattwire/collate

Don't specify character set for civicrm_relationship_cache table

Merge pull request #21385 from eileenmcnaughton/email_fix

dev/core#2814 Fix activity:sendEmail follow up

[NFC] Extend send email test to cover for contact tokens being replaced

[REF] Update Product Create to use hooks and also switch the manage product/premium page to use APIv4 Product Entity

Update to use suggestion from Coleman

Set Product permissions to be the same as contribution entity

Merge pull request #21354 from andyburnsco/patch-1

Change Do Not SMS privacy icon

Merge pull request #21379 from eileenmcnaughton/tok_mem

dev/core#2814 Fix membership pdf to use renderTemplate

Merge pull request #21288 from greenpeace-cee/add-regexp-ops

dev/core#2793 - Add REGEXP/NOT REGEXP SQL operators

dev/core#2814 Fix activity:sendEmail follow up

Merge pull request #21350 from mattwire/optoutverifyemail

Remove requirement to enter email address on Opt Out

Merge pull request #21175 from mattwire/unsubscribeverifyemail

Remove requirement to enter email address on unsubscribe

Remove requirement to verify email on opt out

Remove requirement to enter email address to unsubscribe

dev/core#2793 - Add REGEXP/NOT REGEXP SQL operators in API4

This adds the REGEXP and NOT REGEXP operators to API4 and SearchKit.

Don't specify character set for civicrm_relationship_cache table

Merge pull request #21370 from eileenmcnaughton/fp

dev/core#2715 Move 2 more functions to financial processor class

dev/core#2817 towards deprecating legacy

This adds support for the syntax

{case.status_id:label} which what we agreed as a preferred syntax
to disambiguate the demands for labels vs ids vs machine names in
various implementations.

The new (preferred) label is advertised - but the old
one still works

See https://lab.civicrm.org/dev/core/-/issues/2817 for next steps

Merge pull request #21199 from colemanw/fkDeleteBetter

[REF] Cleanup BAO::del() functions with unnecessary FK checks (dev/core#2757)

Merge pull request #21377 from eileenmcnaughton/act_send

dev/core#2814 Fix activity:sendSMS to use renderTemplate

Merge pull request #21378 from eileenmcnaughton/act_test

dev/core#2814 Fix test to use renderTemplate

Merge pull request #21376 from colemanw/searchTags

SearchKit - enable tags for Saved Searches