dev/core#2814 Fix tokenCompatSubscriber to conditionally evaluate legacy hook tokens

Merge pull request #21463 from eileenmcnaughton/mem

[Ref] Remove unnecessary pass-by-reference

Merge pull request #21458 from eileenmcnaughton/badge_of_honor

dev/core#2834 Preliminary test on badge

Merge pull request #21042 from eileenmcnaughton/uf

[REF] Use OO when determining what to suggest for settings.php prefixes for drupal/backdrop views, instead of scattered "if cms =="

[Ref] Remove unnecessary pass-by-reference

Merge pull request #21459 from eileenmcnaughton/mem

[Ref] Remove no-longer-used variables

Merge pull request #21461 from eileenmcnaughton/render_array

Ensure TokenSmary::render returns an array

[NFC] Cleanup on test class

Merge pull request #21436 from colemanw/afformWarning

Afform - gracefully handle missing html files

[Ref] Remove no-longer-used variables

Ensure TokenSmary::render returns an array

If there are no messages it should return an empty array

Views db cleanup (alternative)

I took a look at https://github.com/civicrm/civicrm-core/pull/20682 and it seemed clear that
in it's current state it would fix one CMS (d9) & break another (d7). This is
an alternate that will hopefully be non-breaky

dev/core#2834 Preliminary test on badge

Merge pull request #21452 from eileenmcnaughton/mem_test

dev/core#2832 Improvements to test on membership letter task

Merge pull request #21460 from civicrm/5.42

5.42 to master

Merge pull request #21444 from colemanw/fixSearchDisplayDelete

SearchKit - Fix deleting search displays

Merge pull request #21456 from demeritcowboy/activity-export542

dev/core#2835 - Activity export broken in 5.41

activity export fail

Merge pull request #21438 from colemanw/fixApi4GetDefaults

APIv4 - Default select clause to exclude "Extra" fields

Merge pull request #21448 from eileenmcnaughton/old_param

[REF] Remove unused parameter

Merge pull request #21451 from eileenmcnaughton/mem

[REF] Remove unused variable

Merge pull request #21450 from eileenmcnaughton/compat_less

[REF] Extract code to load the contact in tokenCompat

APIv4 - Default select clause to exclude "Extra" fields

Recently a "type" property was added to getFieldSpec to allow "Extra"
calculated fields to be declared. This updates api Get to *not*
select those fields by default, as their calculation can be expensive.

Merge pull request #21439 from mlutfy/authxReadme

Authx: README: add note about CMS permissions

Merge pull request #21453 from eileenmcnaughton/member_tokens

dev/core#2832 Add test for legacy membership tokens, add support for preferred tokens

Enhance pdf letter test for mememberships

dev/core#2832 Add test for legacy membership tokens, add support for preferred tokens

There is only one place that calls this code. With cover of the legacy tokens
I can ensure it is the same as the processor tokens. I've fixed it
to support the preferred unambiguous tokens because that means that
we don't have to deal with reports of them not working while calls
to it are phased out of extensions. Working through that
also allowed me to remove some code now

dev/core#2832 rename pdf letter test

[REF] Remove unused variable

This parameter is no longer used

[REF] Extract code to load the contact in tokenCompat

Merge pull request #21401 from eileenmcnaughton/mgd2

dev/core#2823 Restructure determination of required actions.

[REF] Remove unused parameter

There are no longer any cases in git universe that pass this parameter

Merge pull request #20807 from davidjosephhayes/wordpress-profile-signon-2

Wordpress Profile User Creation Sign On (v2)

Move loadDeclartions to reconcile

I'm getting a loop

Make managed entities functions protected

The only affected universe place I found was
https://github.com/veda-consulting-company/uk.co.vedaconsulting.gdpr/pull/294

Since putting the 'information gathering & storing' function on
the constructor is tricky due to the singleton
caching this means we are always working with a consistent entry point

dev/core#2823 Restructure determination of required actions.

The expected actions are now calculated at the start of reconcile.
and retrievable by function.

The code that calls the action-actions
is still pretty cludgey - but the intent is that
we would stop passing the dao object into those
action-action functions in the near future.

This PR punts any decisions about how outputs might look
(eg. making the array retrievable from outside of the function.

Note that there is pretty comprehensive
test cover in CRM_Core_ManagedEntitiesTest

Merge pull request #21429 from eileenmcnaughton/tok_hook2

Consolidate building of contact token list

Merge pull request #21383 from eileenmcnaughton/case_tokens

dev/core#2817 Add CaseToken processor

Merge pull request #21441 from demeritcowboy/dashboard-fail2

dev/core#2829 - "Trying to access array offset on value of type null" on dashboard

SearchKit - Fix deleting search displays

Merge pull request #21443 from demeritcowboy/test-fix-2

Alternate to https://github.com/civicrm/civicrm-core/pull/21442

Merge pull request #21428 from totten/master-tp-filter

TokenProcessor - Allow basic filter/modifier expressions

Merge pull request #21440 from eileenmcnaughton/mgd_test

dev/core#2823 fix test to be more like real function usage

contact with email might not exist

dashboard has no bridge

dev/core#2823 fix test to be more like real function usage

Merge pull request #21437 from demeritcowboy/dashboard-fail

dev/core#2829 - unit test for nonempty dashboard

Authx: README: add note about CMS permissions

test for nonempty dashboard

Afform - gracefully handle missing html files

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

TokenProcessor - Allow basic filter/modifier expressions

Before
------

`Hello {foo.bar}`

After
-----

`Hello {foo.bar|upper}` and `Hello {foo.bar|lower}`

Technical Details
-----------------

This only supports tokens that supply values through `civi.token.eval`.  At the time of
this commit, most `{contact.*}` tokens still use `TokenCompatSubscriber::onRender()` to
hack-in their values and won't work until they switch over.

The regex which recognizes the filter is pretty tight (`\w+`).  This can be relaxed
somewhat by a subsequent change, but I'd say such a change has a burden to demonstrate
safety/interoparbility when running in Token-Smarty format.  (e.g.  demonstrate that
matching of open/close symbols works correctly).

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

Consolidate building of contact token list

Extend test cover on token fields

Extends test cover to check advertised tokens for contacts and to check
those advertised in hooks are included

Add upgrade message

dev/core#2817 Add CaseToken processor

Note this isn't quite passing yet - it highlights that the pre-existing case tokens do
different stuff with date fields. Thinking

Fix booleans to use :label format

Add token processor for Contribution Recur

This adds the token processor using our preferred outputs + 100% test cover.

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

(REF) TokenProcessor - Use preg_replace_callback() instead of strtr()

This will allow more complicated token expressions.

(NFC) TokenProcessor - Docblock

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