Merge pull request #21095 from colemanw/searchDisplayFix

[REF] SearchKit - Use non-deprecated join syntax when loading standalone displays

Merge pull request #21098 from totten/master-action-batch-nfc

(NFC) MailingQueryEvent - Add more docblocks about query-writing and `tokenContext_*`

(NFC) MailingQueryEvent - Add more docblocks about query-writing and `tokenContext_*`

Merge pull request #21088 from totten/master-action-batch

ActionSchedule - Pass real batches into TokenProcessor. Simplify CRM_Activity_Tokens.

Merge pull request #21091 from totten/master-schedui-acttok

Scheduled Reminders UI - Show more activity tokens in admin GUI

Merge pull request #21089 from colemanw/afformCleanup

[REF] Afform - Code cleanup in LoadAdminData API action

SearchKit - Use non-deprecated join syntax when loading standalone displays

Merge pull request #21081 from colemanw/upgradeAngularFileUploader

Upgrade angular-file-uploader to v2.6.1

Merge pull request #21092 from totten/master-acttok-test

(NFC) Expand test coverage for scheduled-reminders with `{activity.*}` tokens

Upgrade angular-file-uploader to v2.6.1

Merge pull request #21090 from totten/master-tokproc-diag

(NFC) TokenProcessorTest - Add scenario inspired by dev/core#2673

Scheduled Reminders UI - Show activity tokens in admin GUI

Add actionschedule test for activity tokens

Merge pull request #21086 from colemanw/removeUnusedCode

CRM_Core_Component - Remove unused code

TokenProcessorTest - Add scenario inspired by dev/core#2673

Merge pull request #21087 from seamuslee001/pear_db_upgrade

Upgrade Pear/DB package to be version 1.11.0

Afform - Code cleanup in LoadAdminData API action

Merge pull request #21083 from eileenmcnaughton/camp_pseudo

Fix caching on campaign pseudoconstant

CRM_Activity_Tokens - Simplify prefetch. Remove special-cases for `actionSearchResult`.

Before: There are two distinct ways in which `CRM_Activity_Tokens` can be called, eg

* For scheduled reminders, it's given `$row->context[actionSearchResult]`. The query is
  inspected for data.
* For everything else, it's given `$row->context[activityId]`. The value is

After: `CRM_Activity_Tokens` always receives activities as `$row->context[activityId]`.

Comment: There are pre-existing tests for activity-based reminders and tokens in
`CRM_Core_BAO_ActionScheduleTest` and `CRM_Activity_ActionMappingTest`.

(REF) ActionSchedule - Pass through SQL fields named "tokenContext_*"

This will help us to consolidate the prefetching logic in `CRM_*_Tokens`.

Currently, `CRM_*_Token::alterActionScheduleQuery()` updates the query to select all fields for the entity.
This is regardless of how many fields there are, whether they are needed, etc. This is also prone to naming conflicts.

With this patch, `CRM_*_Token::alterActionScheduleQuery()` only needs to select one field (`tokenContext_fooId`). This
will then propagate to the `TokenProcessor` which can do its own optimized data-loading.

ActionSchedule - Fill TokenProcessor will real batches

Background: `ActionSchedule::sendMailings()` fetches a batch of pending reminders (per
some specific schedule/rule). Then it composes and sends a message for each.

Before: For each item in the batch, it makes a new `TokenProcessor`.  This
means that the `TokenProcessor` only sees one pending reminder. Therefore, the
`TokenProcessor` cannot meaningfully fetch batched data.

After: It creates one `TokenProcessor` and adds rows for each pending
reminder.  This means that `TokenProcessor` can fetch batched data.# On
branch master-action-batch

Comments: This part of a longer plot to simplify the `CRM_*_Tokens`.
Currently, `CRM_*_Tokens` implements duplicate prefetch mechanisms -- the
general-purpose `prefetch()` and the special-purpose
`alterActionScheduleQuery()`.  This patch makes it possible to use
`prefetch()` for all the real work of prefetching (and phasing-out
`alterActionScheduleQuery()`).

(REF) ActionSchedule - Remove unused `try`/`catch(TokenException $e)`

This try/catch block purports to catch any exceptions of type `TokenException`.

However, if you grep the source tree, you will find that `TokenException` is never thrown.

Merge pull request #21085 from totten/master-action-lang

Scheduled Reminders - Pass locale through to TokenProcessor

Merge pull request #21078 from eileenmcnaughton/con

[Ref] Simplify IF clause

Upgrade PearDB to 1.11.0 following upstream merge of some of CiviCRM's patches

Update pear db patch following new release upstream

CRM_Core_Component - Remove unused code

Fix caching on campaign pseudoconstant

Merge pull request #21084 from colemanw/fixGetInfoItem

APIv4 - Silently ignore errors in CoreUtil::getInfoItem()

ActionSchedule - Set locale during mail hooks / delivery / etc

From a core POV, this should be unnecessary.  And I'm not sure that it's a
good idea for contrib to assume that the locale has been set to the email
receipient.  (It probably isn't done that way in some contexts.)

Never-the-less, this should provide greater continuity with the pre-existing
behavior.  If anyone, say, uses `Hook::alterMail()` to append a translated
footer, then it should continue to work in the same way as before.

ActionSchedule - Convert from global `setLocale()` to `$context['locale']`

Before: Runs `setLocale()` and then executes the entire pipeline for `TokenProcessor`

After: Leaves the global locale alone. Instead, rely on `TokenProcessor` to switch locale
as it visits each recipient.

TokenProcessor - If there is a `locale`, then use it

There are two likely ways in which a tokenized email winds up with localized strings - either
the Smarty `{ts}...{/ts}` tags define it, or a custom/hook-based tag uses `ts()`.

This ensures that the locale is set in both cases.

It's hypothetically possible that some other `civi.token.eval` listeners
need to use the `row->context['locale']`.  However, I grepped universe and
couldn't find anything that would be affected.  (There were two contrib
listeners for `civi.token.eval` and neither seemed to be affected.)

(REF) ActionSchedule - Convert setCommunicationLanguage(...) to setLocale(pickLocale(...))

CRM_Utils_AutoClean - Add variant of `swap()` tuned for setting locale

This can currently be done with `swap()`, but we may be doing this several
times, and `swapLocale()` is a little more pithy and a little more
microoptimal.

APIv4 - Silently ignore errors in CoreUtil::getInfoItem()

Fixes dev/core#2751

Merge pull request #21077 from johntwyman/fix-setPreUpgradeMessage-function

Fix the check to see if the financialAclExtension is installed

[Ref] Simplify IF clause

This if is only reached if contributionStatus is completed - ergo it can't be anything else - poof.

Also previousStatus will be empty if previous_status_id is not set - so the in_array(previousStatus)
check is enough

Fix the check to see if the financialAclExtension is installed

Merge pull request #21074 from demeritcowboy/nocc

Remove no longer used variable in Email.tpl / smarty warning

Merge pull request #21067 from eileenmcnaughton/email2

[Ref] extract function to getEmailDefaults

Merge pull request #21073 from totten/master-sendtpl

MessageTemplate::sendTemplate() - Accept `array $messageTemplate` and `array $tokenContext`

Merge pull request #21072 from colemanw/apiTableName

APIv4 - Throw exception instead of munging illegal join aliases

[Ref] extract function to getEmailDefaults

Part of terminally deprecating the EmailCommon class

no longer used

Merge pull request #21069 from colemanw/searchKitAdminResultsTable

SearchKit - Merge admin results table with searchDisplay code

APIv4 - Throw exception instead of munging illegal join aliases

The problem with "fixing" an illegal join alias is that it's then mysterious
what the correct alias will be, leading to bugs when the incorrect alias
gets used throughout the rest of the api params.

Throwing an exception seems like a better way to ensure developers
are alerted to the error.

(NFC) MessageTemplate - Add "INTERNAL" disclaimers for messageTemplate and tokenContext

Merge pull request #21057 from eileenmcnaughton/tok_resolve

 dev/core#2747 REF] Move all the generic functions to the parent

Merge pull request #21063 from eileenmcnaughton/vars2

[Ref] Clarify what parameters are passed in

MessageTemplate::sendTemplate() - Accept `array $tokenContext` option (unit-test)

MessageTemplate::sendTemplate() - Accept `array $messsageTemplate` option (unit-test)

MessageTemplate::sendTemplate() - Accept `array $messsageTemplate` option

When using `sendTemplate()`, how does it resolve the literal content of the template?

* (Before+After) You may give `string $valueName` (identify+load template by workflow-name)
* (Before+After) You may give `int $messageTemplateID` (identify+load template by ID)
* (Before+After) You may give `string $subject` (override the msg_subject)
* (After) You may give `array $messageTemplate` (specify the content for any/all fields;
  eg msg_subject, msg_text, msg_html)
* (After) Using `string $subject` is deprecated

MessageTemplate::sendTemplate() - Accept `array $tokenContext` option

When using `sendTemplate()`, what values are exported to the TokenProcessor? This option allows
you pass options directly through to tokenContext.

Merge pull request #21070 from demeritcowboy/relative-label

Smarty notice - Explicitly set hideRelativeLabel var on Find Cases form

SearchKit - Switch results table to use a search display

This greatly simplifies the SearchKit admin code by creating a
specialized searchDisplay (copied from crmSearchDisplayTable) which
eliminates all the code for the admin screen to fetch, format and display results.

SearchKit - Add optional search button to displays

This allows search displays to present a button to the user instead of running immediately

be explicit

Merge pull request #21062 from eileenmcnaughton/vars

Move make-sure-single-set out of shared function

Merge pull request #21049 from colemanw/searchDisplaysRef

[REF] SearchKit - display code refactor + pager options

Merge pull request #20976 from eileenmcnaughton/mem_status

Fix Membership.create in BAO to respect passed in status_id

Merge pull request #21060 from demeritcowboy/test-includable

dev/core#2730 - Replace fopen call in CRM_Utils_File::isIncludable with one that doesn't need error-supression to avoid problems in php8

Merge pull request #21066 from eileenmcnaughton/email

[Ref] Move rule to email trait

replace fopen with stream_resolve_include_path

[Ref] Move rule to email trait

This is part of dis-establishing EmailCommon - the function is not called from elsewhere

Merge pull request #21061 from eileenmcnaughton/trait

Remove unused assignment

SearchKit - Refactor search displays to share more code

SearchKit - Improve searchDisplay pager with additional options

Adds 'show_count' and 'expose_limit' as options to searchDisplay pagers

SearchKit - Refactor sort-related functions into a trait

SearchKit - Refactor task-related functions into a trait and move base trait to its own file

[Ref] Clarify what parameters are passed in

this function is called from 2 places with 4 keys in the array always set & no others

Move make-sure-single-set out of shared function

3 places in the code call this - in 2 _single is declared on the class as false
& hence don't need this - ergo it is only being done for the one remaining place
so it's not shared it's just a helper for that code & we should return it

Remove unused assignment

Merge pull request #21056 from civicrm/5.41

5.41

[REF] Move all the generic functions to the parent

This is for clarity, note that I need agreement on
https://lab.civicrm.org/dev/core/-/issues/2745
to decide whether to keep 'getExposedFields' or not - hence
that is not moved at this stage

[Ref] Extract getFieldValue, make function generic

Merge pull request #21047 from eileenmcnaughton/tok_return

[Ref] cleanup alterActionSchedule

Merge pull request #21001 from seamuslee001/fix_utf8mb4_test

[REF] Fix utf8mb4 test in APIv4 and re-enable the altering of databas…

[REF] Fix utf8mb4 test in APIv4 and re-enable the altering of database in the v3 version of the test and fix handling for first name field

Set version to 5.42.alpha1

Set version to 5.41.beta1

Merge pull request #21053 from eileenmcnaughton/group_odd

dev/core#2742 fix group type on group create from form

Merge pull request #21051 from eileenmcnaughton/allan

dev/core#2740 fix dashboard regression on links

Merge pull request #21050 from eileenmcnaughton/notice

Tpl notice fixes

dev/core#2742 fix group type on group create from form

dev/core#2740 fix dashboard regression on links

Merge pull request #21045 from colemanw/fixSearchKitTaskPermissions

SearchKit - Fix permission to access task list

Tpl notice fixes

These fields are both conditionally assigned to the template

[Ref] cleanup alterActionSchedule

We can simplify this function now as basicFields is keyed by the real field name and
all pseudofields are already in basicFields.

Merge pull request #21044 from mlutfy/fixContribSymbiotic

contributor-key.yml: updates for Symbiotic

SearchKit - Fix permission to access task list

This regressed in 06ea32c, as a new api action was added but permissions were not declared.

contributor-key.yml: updates for Symbiotic

Merge pull request #21039 from eileenmcnaughton/tok_pseudo

Reconcile tokens misnamed or missing on legacy class

Merge pull request #21038 from eileenmcnaughton/tok_names

Switch to using apiv4 for metadata

Merge pull request #21041 from eileenmcnaughton/notice2

Enotice fix

Merge pull request #21040 from eileenmcnaughton/tpl

Enotice fix

Merge pull request #20752 from samuelsov/ReportGroup

Make report subtotals work for custom fields

Enotice fix

This one is a bit of a brain twister but I'm pretty sure the re-ordered IF has the
same result - but one less enotice

Merge pull request #20989 from mattwire/mailtasks

Fix mailing task when there are no tasks

Enotice fix

Reconcile tokens misnamed on legacy class

This switches to using the same metadata based record for both classes