avoid error-suppression operator

Merge pull request #21154 from civicrm/5.41

5.41

Merge pull request #21151 from eileenmcnaughton/541m

Membership batch processing error

Membership batch processing error

This fixes a fatal because of incorrect typing

Affects 5.40

Merge pull request #21148 from civicrm/5.41

5.41

Merge pull request #21143 from seamuslee001/fix_enotice_php8

[REF] Fix e-notice in APIv3 Profile Test on PHP8

Merge pull request #21147 from eileenmcnaughton/541

dev/core#2758 fix task tiles

Merge pull request #21146 from demeritcowboy/smarty-default-caserole

[NFC] Update testCaseActivityCopyTemplate to provide variable that would usually be present

dev/core#2758 fix task tiles

[REF] Fix e-notice in APIv3 Profile Test on PHP8

Remove e-notice issue found by dave d

missing variable in template

Merge pull request #21140 from demeritcowboy/smarty-long

Use convenience function for one-off token evaluations to avoid too-long filenames and possible privacy issues

avoid fails for file too long, and possible privacy issues

Merge pull request #21075 from eileenmcnaughton/email3

[Ref] Move id fetching to the classes

Merge pull request #20168 from larssandergreen/add-recurring-filter-to-contribution-summary-report

dev/search#63 Add recurring contributions to contribution reports

Merge pull request #21138 from civicrm/5.41

5.41

Merge pull request #21136 from agh1/5.41.0-releasenotes-initial

5.41.0 release notes initial run

Merge pull request #21114 from totten/master-refl-finders

(REF) ReflectionUtils - Add findStandardProperties() and findMethodHelpers()

Merge pull request #21129 from civicrm/5.41

5.41

5.41.0 release notes: added boilerplate

5.41.0 release notes: raw from script

Merge pull request #20907 from demeritcowboy/report-entitylabel

dev/core#2691 - On logging detail civireport show words instead of numbers (part 2)

Merge pull request #21125 from colemanw/refCommentFix

NFC - Fix docblock in CRM_Core_Transaction

Merge pull request #21099 from eileenmcnaughton/camp54

dev/core#2743 fix api v3 to not unnecessarily load options

Merge pull request #21102 from artfulrobot/ui/37

dev/user-interface#37 Reinstate jQueryUI visible focus effect in quicksearch

[Ref] Move id fetching to the classes

Merge pull request #21127 from civicrm/5.41

5.41 to master

Merge pull request #21124 from MegaphoneJon/fix-event-validation

fix crmValidate() not loading on event registration pages

NFC - Fix docblock in CRM_Core_Transaction

fix crmValidate() not loading on event registration pages

Merge pull request #21116 from eileenmcnaughton/nfc

[NFC] {Test} Minor cleanup

Merge pull request #21103 from mlutfy/ui38

dev/user-interface#38 Contact Edit: Only display signatures if contact has a CMS account

Merge pull request #21121 from totten/master-rendertpl-rem

Replace deprecated calls to `renderMessageTemplate()`

CRM_Core_BAO_MessageTemplate - Remove `renderMessageTemplate()`

CRM_Core_BAO_MessageTemplateTest - Convert `renderMessageTemplate()` to `renderTemplate()`

CRM_SMS_Form_Upload - Convert `renderMessageTemplate()` to `CRM_Core_TokenSmarty::render()`

CRM_Contribute_Form_Task_PDFLetter - Convert `renderMessageTemplate()` to `CRM_Core_TokenSmarty::render()`

Merge pull request #21115 from totten/master-rendertpl

MessageTemplate - Add renderTemplate(). Deprecate renderMessageTemplate().

Merge pull request #21117 from civicrm/5.41

5.41 to master

Merge pull request #21107 from MegaphoneJon/clean-override-total

delocalize total amount before passing to setOverrideTotal

[NFC] {Test} Minor cleanup

MessageTemplate - Deprecate renderMessageTemplate(). Prefer renderTemplate() and TokenSmarty::render(). Add tests

Merge pull request #21113 from seamuslee001/master

5.41

Merge in 5.41

(REF) ReflectionUtils - Add findStandardProperties() and findMethodHelpers()

Merge pull request #21108 from seamuslee001/5.41

[REF] Upgrade CKEditor to 4.16.2

Merge pull request #21111 from civicrm/5.41

5.41

Merge pull request #21104 from totten/5.41-typo

(NFC) release-notes - Fix typo

Merge pull request #21100 from eileenmcnaughton/exceptless

dev/core#2749 remove exception when no payment processor configured when no payment processor configured

[REF] Upgrade CKEditor to 4.16.2

Merge pull request #21046 from eileenmcnaughton/camp

dev/core#2747 Reconcile remaining fields between scheduled reminders and legacy tokens

delocalize total amount before passing to setOverrideTotal

(NFC) release-notes/5.39.1.md - Fix typo

Typo was autogenerated from a template. Template is also fixed.

dev/user-interface#38 Contact Edit: Only display email signatures for contacts that have a user/CMS account

Merge pull request #21094 from mattwire/hideextensionkeyupgrade

Replace extension key with label during install/upgrade/disable/uninstall

Merge pull request #21096 from mattwire/guzzletimeout

Respect http_timeout core setting for Guzzle HTTP requests

Merge pull request #20936 from eileenmcnaughton/mem_order

dev/core#2717 Use Same order ->payment flow for non recurring back of…

Reinstate jQueryUI visible focus effect in quicksearch user-interface issue 37

Merge pull request #21076 from eileenmcnaughton/camp_name

Token Parser - Allow tokens with multiple dots (eg {contribution.contribution_recur_id.amount})

Merge pull request #20991 from eileenmcnaughton/invoice

dev/core#2719 [REF] Remove a couple more (tested) references to legacy contribution_invoice_settings

TokenProcessorTest - Ensure that tokens with dots and colons are processable

dev/core#2749 remove exception when no payment processor configuredhen no payment processor configured

This is an attempt to fix a regression https://lab.civicrm.org/dev/core/-/issues/2749
when there are legitimately no payment processors assigned to a page.

I feel like the configuration screens are adequate to enforce this & having an exception like
this in the assign function is the wrong place.

The original issue is that the is_monetary box was not checked & hence the
processor wasn't assigned. This felt like asking users to understand that
a payment method when money options are configured

Merge pull request #21080 from twomice/2673_subject_token_empty_body

(dev/core#2673) Email Tokens - Custom tokens in `subject` block similar tokens in `body`

Fix one test & remove 2 that covered the error message

Simplify calls to is_email_pdf

Per reviewer feedback - we don't need to check more than just the one setting. The
UI (for better or worse) ties the 2 together so under normal operation
they will be set together but we don't need to check that

dev/core#2719 [REF] Remove a couple more (tested) references to legacy contribution_invoice_settings

Merge pull request #21095 from colemanw/searchDisplayFix

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

dev/core#2743 fix api v3 to not unnecessarily load options

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

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

tpl notice fixes

Replace extension key with label during install/upgrade/disable/uninstall

(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

Respect http_timeout core setting for Guzzle HTTP requests

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

Permit parsing of related entity tokens

This fixes the regex portion for related entities tokens eg.
{participant.event_id.title}

Note the reason for this originally
is not needed but keeping the regex parts

Add no-prefetch campaign psuedoconstant

Reconcile remaining fields between scheduled reminders and legacy tokens

This adds the last fields that were in legacy tokens but not scheduled reminders and now
the same code is deriving the list for each. I wound up UI testing rather than unit
testing that custom fields are still advertised without this line
as the test uses rollback & adding custom fields would break that.

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