Merge pull request #21366 from eileenmcnaughton/filename

[REF] dev/core#2790 Move pdf getFileName functionality to the trait

Merge pull request #21365 from eileenmcnaughton/act_send

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

Merge pull request #21367 from eileenmcnaughton/export

dev/core#2814 migrate export processor to use MessageRender

Merge pull request #21364 from eileenmcnaughton/prof_edit

dev/core#2814 Fix profile edit to use messagetemplate:render

Merge pull request #21335 from eileenmcnaughton/pdft

[REF] dev/core#2790 dev/core#2814 Start migration to MessageTemplate::render

Merge pull request #21373 from eileenmcnaughton/depp

Fix test to test trait not deprecated function

Merge pull request #21375 from mattwire/settitlelegacycustomsearch

Update use of CRM_Utils_System::setTitle() for legacycustomsearches

Merge pull request #21368 from mattwire/settitle

Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 4

Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 4

Update use of CRM_Utils_System::setTitle() for legacycustomsearches

Merge pull request #21372 from eileenmcnaughton/ddep

Remove deprecated input variable

Fix test to test trait not deprecated function

Merge pull request #21371 from eileenmcnaughton/depre

Remove unused private function

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

Note that this does create a situation where more queries could occur but
1) this is only called in core from the email task - which is limited to 50 contacts so it
is low-volume
2) this is really the start of a cleanup - there are other places where
queries can be removed when we look at the total flow but I feel we need to get
some simplification happening first & hence have picked on this focus
as the first step (ie removing the calls to replaceContactTokens)
3) this seems to be called in extensions to some extent - and it should still work
- there might be more queries but I think the 'support contract' for calling an
unsupported function does not include it working in ways not required by
core code

Remove deprecated input variable

Remove unused function

Merge pull request #21157 from mlutfy/fixMonthnames

dev/translation#71 getFullMonthNames: do not rely on the OS for translation

Merge pull request #21369 from demeritcowboy/case-token-take-2

[NFC/Unit test] - Fix secondary issue with case token consistency test

Merge pull request #21301 from greenpeace-cee/add-where-functions

Add SqlFunction support in addWhere

dev/translation#71 getFullMonthNames: do not rely on the operating system locale for translation

fix timing issue

Add SqlFunction support in addWhere

This adds an optional $isExpression parameter in addWhere that
allows use of SqlFunctions in WHERE expressions:

Contact::get(FALSE)
  ->addWhere('last_name', '=', $last_name)
  ->addWhere('LOWER(first_name)', '=', "BINARY('ALICE')", TRUE)
  ->execute()->indexBy('id');

This also adds a few tests to the existing expression support for
ON and HAVING clauses as well as a new BINARY function.

Merge pull request #21363 from colemanw/searchDownloadTest

SearchKit - Add 'array' option to download API and add test

dev/core#2814 migrate export processor to use MessageRender

Note that test cover is in
I used  to step through it
as that specifically covers this change

Merge pull request #20985 from eileenmcnaughton/fprocessor

dev/core#2715 [REF] Move another 2 functions to the financial processor class

[REF] dev/core#2790 Move getFileName functionality to the trait

This moves the recently added functionality to standardise the file name to PDFTrait

For code not using the trait (naughty extensions that are using core code in unsupported ways)
the functionality will be the same as prior versions.

(note extensions should just copy & paste the code they want from core into their extensions
rather than call core functions - it's not 'DRY' if it's not supported).

dev/core#2814 Fix profile edit to use messagetemplate:render

If the api were merged I would have used that - but this adds the test & when can test converting
to the api when it is merged. It gets the hard lifting of this conversion out of the way

Merge pull request #21361 from eileenmcnaughton/deprecated_gone

Remove deprecated function

SearchKit - Add 'array' option to download API and add test

Merge pull request #21359 from eileenmcnaughton/pdf_move_cleanup

[REF] dev/core#2790 Follow on on code move, make not static

Merge pull request #21355 from eileenmcnaughton/surver

dev/core#2486 Add survey v4 api

[REF] Follow on on code move, make not static

Merge pull request #21360 from eileenmcnaughton/case_pdf

Remove testing hack

Merge pull request #21358 from eileenmcnaughton/deprecated

Remove deprecated function

Merge pull request #21362 from eileenmcnaughton/prof_edit

[REF] Fix form to call this->setTitle()

[REF] Fix form to call this->setTitle()

Remove deprecated function

It was deprecated in 5.35 (and never supported to be called from outside
of core)

Remove testing hack

dev/core#2486 Add survey v4 api

Remove deprecated function

dev/core#2715 [REF] Move another 2 functions to the financial processor class

The process involves
- checking the selfs
- temporarily making public
- making private again once the calling function is also moved

Merge pull request #21318 from michaelmcandrew/dev-core-2798

activity import: prepend custom group title to custom fields

Merge pull request #21210 from colemanw/deprecateDel

[REF] Deprecate 2 del() functions in favor of generics and hooks (dev/core#2757)

Merge pull request #21352 from mattwire/settitle3

Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 3

Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 3

Merge pull request #21202 from mattwire/settitle2

Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 2

Merge pull request #21349 from mattwire/ckeditor4

NFC Clarify description for ckeditor4 plugin

Merge pull request #20856 from eileenmcnaughton/paypal

dev/core#2716 [REF] Stop 'distributing' the main flow

Merge pull request #21311 from eileenmcnaughton/poof

[REF] Do not set input['line_item'] pointlessly

Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 2

Clarify description for ckeditor4 plugin

Merge pull request #21135 from seamuslee001/dev_core_1618

dev/core#1618 Trigger a reconciliation of logging tables after Extens…

Set version to 5.43.alpha1

Set version to 5.42.beta1

Merge pull request #21328 from colemanw/downloadCSV

SearchKit - Add download CSV action

Merge pull request #21336 from totten/master-smarty-alias

TokenProcessor - Allow defining Smarty variables which are populated via token

Merge pull request #21279 from eileenmcnaughton/upsub

[Ref] Deprecate Core_Error handling

Merge pull request #21287 from colemanw/afformButton

Afform - Fix button appearance and block form during submission

Merge pull request #21345 from seamuslee001/fix_afform_test

[NFC] Fix E-notice in Afform unit tests

SearchKit - Add download CSV action

Merge pull request #21344 from seamuslee001/fix_page_hook_test

[REF] Fix Page Hook test on php8 by putting in guard into customDataB…

Merge pull request #21343 from colemanw/searchKitLinks

SearchKit - Add links to admin table and refresh after popups

[NFC] Fix E-notice in Afform unit tests

Merge pull request #21342 from vingle/vingle-patch-1

Tidies Joomla 4 integration (menu, padding) after final release

Merge pull request #21285 from mattwire/contributionviewlineitems

Simplify ContributionView form. Always display "lineitems"

[REF] Fix Page Hook test on php8 by putting in guard into customDataBlock around the SubType variable

Merge pull request #21340 from colemanw/upgradeFunctions

[NFC] Cleanup boilerplate code in extension upgrader classes

Merge pull request #21337 from eileenmcnaughton/tokens

dev/core#2806 Fix accidental exposure of v4 tokens

SearchKit - Add links to admin table and refresh after popups

J4 menu fixes

Adjustments in response to changed Joomla 4 admin menu height; removal of smallest screen behaviour as no longer needed.

J4 release tidying

Merge pull request #21341 from demeritcowboy/token-time

[NFC/Unit test] Update flaky test CRM_Utils_TokenConsistencyTest::testCaseTokenConsistency

created and modified are backwards

Cleanup boilerplate code in extension upgrader classes

Merge pull request #21334 from eileenmcnaughton/pdf_tokens

[REF] dev/core#2790 deprecate preProcessSingle

Merge pull request #21316 from colemanw/afformGet

Afform - Optimize Get by checking type

Merge pull request #21286 from vingle/patch-13

Fixes unusable modals in Joomla 4

dev/core#2806 Fix accidental exposure of v4 tokens

TokenProcessor - Allow defining Smarty variables which are opulated from tokens

Overview
--------

This allow more interoperability between Smarty expressions and tokens.  For
example, suppose one had a contribution-related message that could use the
Smarty variable `$theInvoiceId` and/or the token `{contribution.invoice_id}`.
This revision allows the Smarty variable to function as an alias for the token.

Before
------

The caller would need to precompute Smarty values, eg

```php
$theInvoiceId = civicrm_api4('Contribution', 'get', [
  'select' => 'invoice_id',
  'where' => [['id', '=', $contributionId]]
]);
$p = new TokenProcessor($this->dispatcher, [
  'controller' => __CLASS__,
  'schema' => ['contributionId'],
  'smarty' => TRUE,
]);
$p->addMessage('example', 'Invoice #{$theInvoiceId}!', 'text/plain');
$p->addRow(['contributionId' => 123]);
```

After
-----

The caller can declare a Smarty=>Token alias and leverage token data-loader.

```php
$p = new TokenProcessor($this->dispatcher, [
  'controller' => __CLASS__,
  'schema' => ['contributionId'],
  'smarty' => TRUE,
  'smartyTokenAlias' => [
    'theInvoiceId' => 'contribution.invoice_id',
  ],
]);
$p->addMessage('example', 'Invoice #{$theInvoiceId}!', 'text/plain');
$p->addRow(['contributionId' => 123]);
```

Comments
--------

The target token must be populated via `civi.token.eval` (e.g `$e->token('foo', 'bar', 'value')`).
This would work with `CRM_*_Tokens`.  But if the token is evaluted by other means (eg
`CRM_Utils_Token::replaceGreetingTokens()`), then it won't currently be resolved.

[REF] dev/core#2790 deprecate preProcessSingle

This is only called from here & is short & can just be part of the main function.

[REF] dev/core#2790 Start migration to MessageTemplate::render

Merge pull request #21270 from colemanw/searchKitListing

SearchKit - Use a search display to display searches

Merge pull request #21333 from colemanw/searchKitTableBug

SearchKit - Fix pager count and add 'None Found' text in empty tables

Merge pull request #21331 from eileenmcnaughton/pdf

[REF] dev/core#2790 move preProcess static to the trait

[REF] dev/core#2790 move preProcess static to the trait

Merge pull request #21305 from eileenmcnaughton/mem_pdf

[REF] dev/core#2790 Deprecate CRM/Member/Form/Task/PDFLetterCommon

SearchKit - Fix pager count and add 'None Found' text in empty tables

SearchKit - Use a searchDisplay instead of ad-hoc table to display main list of saved searches

SearchKit - only show sortable help text for sortable columns in table display

SearchKit - Add custom template column type

Merge pull request #21332 from civicrm/5.41

5.41 to master

Merge pull request #21330 from demeritcowboy/contrib-token-warning

dev/core#2807 - Contribution thank-you letters give Undefined index: contribution when you don't use any contribution tokens

put loop back inside the if

Merge pull request #21310 from eileenmcnaughton/prepro

[REF] dev/core#2790 Pre process cleanup on pdf tasks

Merge pull request #21276 from eileenmcnaughton/pdf

[REF] dev/core#2790 towards pdf task trait

Merge pull request #21254 from colemanw/afformLocationType

Afform - selectable location type for address, email, etc

Merge pull request #21320 from colemanw/searchKitRun

 dev/core#2732 SearchKit - Move field formatting from client-side to server-side

Merge pull request #21256 from demeritcowboy/support-link

Fix support link just added in oauth-client extension info.xml

[REF] Pre process cleanup

[REF] dev/core#2790 towards pdf task trait

Deprecate CRM_Activity_Form_Task_PDFLetterCommon

This gets us to having 2 classes rather than 3 that manage the activityPDF task
functionality.

CRM_Activity_Form_Task_PDFLetterCommon doesn't really add anything from a structure POV
but it does make it more confusing. There are also functions on the parent
that are only used by this class - which makes switching to a trait
harder. This untangles that part.

Note that once we have the trait (& some more token cleanup done) we will be well placed
to re-share some of these functions again