Merge pull request #21955 from colemanw/managedEntity

APIv4 - Add managed entity functionality

APIv4 - Provide pseudo-fields and Revert action for managed entities

This new "ManagedEntity" trait provides 2 extra fields and a Revert action
to facilitate UIs which show the managed state and a revert button,
similar to the AfformAdmin UI.

Merge pull request #21929 from colemanw/searchKitDefaultDisplays

SearchKit - default displays and ACL-aware links

Merge pull request #21959 from civicrm/5.43

5.43

Merge pull request #21958 from seamuslee001/fix_token_rendering_campaign_legacy_php8

[REF] Fix pseduoconstant token rendering for contributions via legacy…

SearchKit - Fix action checkboxes to work with any key

Lifts the assumption that every entity has a key column named "id"

SearchKit - Default label when adding a display

[REF] Fix pseduoconstant token rendering for contributions via legacy way on php8 and include campaign_id pseudoconsntants as well

Merge pull request #21934 from eileenmcnaughton/empty

Ensure no_licence is assigned for drop.tpl

Merge pull request #21952 from eileenmcnaughton/im

Handling for intermittent Im->getfields error

Use empty not isset in drop.tpl

On experimenting a little with smarty escaping it turns out that isset is gonna be
a lot more painful than empty - we should avoid where we can. In this case it
is passed in as TRUE or not at all (from schema.tpl) so an empty check is just fine

Merge pull request #21948 from braders/hardcoded-error-text

Avoid hardcoded text in error strings; make translatable

Merge pull request #21954 from braders/avoid-hardcoded-period-type-labels

Don't hardcode Period Type in Manage Premiums template.

Don't hardcode Period Type in Manage Premiums template.

Merge pull request #21949 from braders/harcoded-setstatus-text

Avoid harcoded text in setStatus calls; make translatable

Merge pull request #21951 from eileenmcnaughton/messages

Ensure communityMessages is assigned to the template.

Ensure communityMessages is assigned to the template.

This is a minor code simplification & ensures the parameter is set rather than
handling it not being set at the template level

Handling for intermittent Im->getfields error

I'm not quite sure when this started but I've been seeing Im->getfields not found errors of late - this helps

Merge pull request #21950 from braders/misc-hardcoded-strings

Make miscellaneous strings translatable

Merge pull request #21946 from braders/dedupe-find-label-associations

Accessibility: Fix label associations for DepupeFind template.

Avoid hardcoded text in setStatus calls; make translatable

SearchKit - Expose default display to the UI

Adds a 'view' link to the searchKit admin listing, which links to the default display
Adds a 'view' dropdown to the compose search screen, with links to the default + all other displays
Updates Afform to work with default search displays

Make miscellaneous translatable

Avoid hardcoded text in error strings; make translatable

SearchKit - Only show tags filter if tags exist

Remove clutter if an org does not use tags.
Also removes an unused copy-pasted function.

SearchKit - Set links in admin UI using entity+action+join instead of path

Now that SearchDisplay.Run understands this new link syntax, the admin UI
can use it too. The old links set in existing search displays will still work,
but in order for ACLs to be checked they need to use the new syntax.

SearchKit - Check ACLs before displaying links or in-place edit

SearchKit - Autogenerate default display table for saved searches; calculate links by entity+action

This adds a SearchDisplay.getDefault APIv4 action, which autogenerates the default table display
which is used on the admin screen.

It also refactors links so they can be calculated based on a given entity+action which does not
require paths to be stored in the search display.

Accessibility: Fix label associations for DepupeFind template.

This ensures that the labels have an accessible name for assistive technologies.

Merge pull request #21931 from eileenmcnaughton/ple

Convert more tests to use full form flow

Do not pass test-blocked parameter to function that doesn't use it

Merge pull request #21938 from mattwire/groupcontactcacheunused

Remove unused function from GroupContactCache

Convert more tests to use full form flow

Remove unused function

Merge pull request #21855 from eileenmcnaughton/part

Add test for selvsvctransfer, remove use of `$contact` from template

Merge pull request #21930 from colemanw/removeCxn

Remove "Connections" from navigation menu

Merge pull request #21932 from eileenmcnaughton/quotes

[NFC] [Tests] remove code smell - unnecessary double quotes

Fixes dev/core#2910 - Remove "Connections" from navigation menu

This obscure feature is currently unused, so there's no reason to show it in the menu.
Fixes https://lab.civicrm.org/dev/core/-/issues/2910

[NFC] [Tests] remove code smell - unnecessary double quotes

This is an IDE cleanup. It touches a few files but
1) only test files and
2) only that specific change, made via an IDE is in here

So I think it should be possible to merge based on test outcome

Merge pull request #21790 from eileenmcnaughton/greetings_earthling

Switch replaceGreetingTokens over

Merge pull request #21922 from jitendrapurohit/taskmenu

dev/joomla#36 - Fix fatal error in joomla with civicrm task query param

Regenerate sql

Replace call to getTokenDetails with simple contact.get

Add test for selvsvctransfer, remove from template

APIv4 - Cache SchemaMap in Civi::cache('metadata')

When adding/removing custom groups, the schema map changes.
Civi already flushes the metadata cache when this happens,
but the APIv4 SchemaMap was being cached in memory (in a container service).
This changes it to be stored in the metadata cache like everything else.

Merge pull request #21926 from colemanw/fixCrmCoreMenu

[REF] CRM_Core_Menu - Fix syntax, undefined variables, unused code, pass-by-ref

CRM_Core_Menu - Fix syntax, undefined variables, unused code, pass-by-ref

Merge pull request #21820 from colemanw/searchKitServerSideRender

SearchKit server-side rendering

Merge pull request #21925 from eileenmcnaughton/dom_from

Simplify usage of getting domain from email

Merge pull request #21917 from braders/feature/translate-set-title-calls

Make hardcoded labels passed to setTitle translatable

Merge pull request #21928 from eileenmcnaughton/usused

Remove unused function

Remove unused function

Note we don't slow the removal down by adding deprecation notices for internal financial functions as they have
never been stable / supported for external use

Merge pull request #21914 from eileenmcnaughton/trxn

[REF] Move another function to the FinancialProcessor

SearchKit - Use server-side preprocessing for editable fields

SearchKit - server-side rendering

Moves logic for resolving field values from the client to the server-side,
which gives more control over formatting, links, and greater consistency
of spreadsheet output.

Simplify usage of getting domain from email

This adds a function that 'just gets the string' - rather than overloading
another function and (worse) getting the string back as an array that needs to be
'currented'. It also switches
the participant function to use it - as we will soon not be passing
domainValues into this function

Merge pull request #21924 from civicrm/5.43

5.43 to master

Merge pull request #21921 from eileenmcnaughton/gc

[NFC] [REF] Test class cleanup

Merge pull request #21913 from eileenmcnaughton/worddoc

dev/core#2864 fix for rc breakage in pdf-letter-to-word

dev/core#2864 fix for rc breakage in pdf-letter-to-word

Merge pull request #21853 from colemanw/allCoreTables

AllCoreTables - Allow virtual entities to share a DAO class

Merge pull request #21907 from eileenmcnaughton/tests

Fix a few more tests to use a more realistic flow

Merge pull request #21906 from eileenmcnaughton/cont_test

Fix more tests to use the full form flow

AllCoreTables - Allow virtual entities to share a DAO class

Indexes entities by brief name instead of class name,
which allows one class to be shared by multiple entities.

Fix fatal error in joomla with civicrm task query param

[NFC] [REF] Test class cleanup

The driver for this was to remove uses of the DAO->createTestObject methods which were creating
individuals with data invalid for them. However, this is a full 'margin cleanup' and
standardises the test.

I think the whole createTestObject approach is best viewed as an early attempt to get
lots of entities testable quickly but it's overwraught for creating contacts
(and generally a bit flakey)

Merge pull request #21920 from colemanw/getInfoItemCache

APIv4 - Fix performance drag caused by getInfoItem calling the API

Merge pull request #21918 from braders/feature/spelling-fixes

Miscellaneous spelling fixes

APIv4 - Fix performance drag caused by getInfoItem calling the API

This function is sometimes called thousands of times per request, e.g. for every cell
in a searchKit display table. Going through the API wrapper that many times is a
considerable (+10 sec) drag on performance.
Instead of calling Entity.get, we can just read directly from the cache where it stores its data.

Miscellaneous spelling fixes

Merge pull request #21915 from eileenmcnaughton/protext

[Ref] Make functions private to protect them

Merge pull request #21847 from eileenmcnaughton/pledge2

Standardise our 2 pledge templates to use tokens, add tests

Translate hardcoded labels passed to setTitle

[REF] Move another function to the FinancialProcessor

At this stage just moving all the financial processing functions to the same class

[Ref] Make functions private to protect them

As we move functions to this class making them private once they are unused outside the class
makes it easier to clean them up

Switch pledge_reminder over to use tokens

Update pledge acknowledgement to use token processor only

Add test for domain, contact tokens in pledge

Merge pull request #21910 from eileenmcnaughton/new_validate

Fix cleanup on test, also NFC cleanup

Merge pull request #21692 from demeritcowboy/dbssl

Fix unreplaced template vars in civicrm.settings.php when using legacy installer

Fix cleanup on test, also NFC cleanup

This test was not cleaning itself up very well in the tearDown - causing the post test validate to fail

Fix test to manage statics within the test

CiviTest - Define `CRM_Fake_DAO_*` classes

The tests are being a being a brittle because someone wants to load these non-existent classes:

* CRM_Fake_DAO_FakeFile
* CRM_Fake_DAO_Widget
* CRM_Fake_DAO_Forbidden
* CRM_Fake_DAO_Sprocket

replaceGreetingTokens - Be more forgiving about $contactDetails

This should fix a failure in `TokenConsistencyTest`. Setting a breakpoint and monitoring
`$contactDetails`, it appears that the data would come in multiple formats. (Sometimes as
a nested array; sometimes as a flatter array.) This makes it work with both cases.

Fix token metadata to be clearable outside tests

Switch replaceGreetingTokens over

Merge pull request #21904 from braders/feature/remove-from-tag-translatable-errors

Make error message translatable - remove tag tasks.

Merge pull request #21905 from braders/feature/add-to-group-translatable-errors

Make error messages translatable.

Fix a few more tests to use a more realistic flow

Fix more tests to use the full form flow

Make error messages translatable.

Merge pull request #21892 from artfulrobot/artfulrobot-order-membership

Improve tests around the Order.create API and membership

fix unreplaced template vars when using legacy installer

Make error message translatable.

Also fix typo (space between 'at least')

Merge pull request #21848 from eileenmcnaughton/valid

Add financial validation check (to the extent we can)

Merge pull request #21863 from mariav0/patch-6

Update contribution_recurring_billing_html.tpl

Merge pull request #21850 from mariav0/patch-1

Update contribution_online_receipt_html.tpl

Merge pull request #21888 from mariav0/patch-26

Update test_preview_html.tpl

Merge pull request #21890 from mariav0/patch-28

Update contribution_invoice_receipt_html.tpl

Merge pull request #21889 from mariav0/patch-27

Update uf_notify_html.tpl