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.

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

Merge pull request #21325 from eileenmcnaughton/build

Remove extraneous buildQuickForm

Merge pull request #21324 from seamuslee001/fix_php8_testing

[NFC] Fix UpdateSubscriptionTest on php8 by creating a Payment Processor

Remove extraneous buildQuickForm

Merge pull request #21297 from eileenmcnaughton/pdftrait

[REF] dev/core#2790 move buildForm to pdfTrait

[NFC] Fix UpdateSubscriptionTest on php8 by creating a Payment Processor

Merge pull request #21323 from civicrm/5.41

5.41 to master

Merge pull request #21322 from alifrumin/rn5.41

[NFC] 5.41 Release Notes - First pass

Merge pull request #21177 from colemanw/rand

SearchKit - Allow random sorting (Fixes dev/report#75)

Merge pull request #21321 from colemanw/basicCriteria

[REF] Fix undefined smarty vars in Advanced Search

release notes 5.41 - first pass

SearchKit - Move field formatting from client-side to server-side

Formattig field values server-side allows the view values to be reused in exports.
It also opens the door for more complex formatting to happen on the server, if needed.

SearchKit - Break part of Run action into a base class

Fix undefined smarty vars in Advanced Search

Merge pull request #21317 from michaelmcandrew/dev-core-2797

better target multivalue checkbox and multiselect import validation

better target multivalue checkbox and multiselect import validation

Afform - Optimize Get by checking type before parsing html or loading custom data

Sometimes afforms are fetched by type, e.g. `cv api4 Afform.get +w type=search`
Previously this would load all html files and custom data groups before
discarding the ones not of type search.
Now it will skip irrelevant files.

Merge pull request #21120 from eileenmcnaughton/acl_setting

Move financial acl setting to the extension

Merge pull request #21315 from deb1990/fix-searchkit-add-button-ui

Fix Searchkit "Add" columns button UI

Merge pull request #21314 from seamuslee001/fix_php8_testing

[NFC] Fix undefined array key when running CRM unit test suite in php8

Fix searchkit add button ui

[NFC] Fix undefined array key when running CRM unit test suite in php8

Merge pull request #21313 from seamuslee001/address_custom_data

[REF] Improve Custom data insert performance when using the copyCusto…

Merge pull request #21312 from colemanw/entityOptions

SavedSearch - Add pseudoconstant for api_entity field

Fix Address test failure by passing the hook through to copyCustomFields and fix Contribution Recur test failures by doing some cleanup

Merge pull request #21241 from eileenmcnaughton/entity_batch

dev/core#2682 Entity batch declare option group for pseudoconstant

[REF] Improve Custom data insert performance when using the copyCustomFields

APIv4 - Add caching in Entity.get to avoid repeated file scanning

SavedSearch - Add pseudoconstant for api_entity field

This makes it easier to display in the UI what a search is for,
e.g. `->addSelect('api_entity:label')` would return "Contacts" for a search of Contacts.

Merge pull request #21309 from eileenmcnaughton/get

Use getter to get subscription id

Merge pull request #21306 from eileenmcnaughton/prepro

[REF] Copy preProcessFromAddress back into the pdf function

Regenerate Generated sql file and also set civicrm_financial_trxn to be default entity_table when created via APIv4

Merge pull request #21282 from eileenmcnaughton/test_sub

Add test to UpdateSubscription form

Use getter to get subscription id

These places all effectively do the same as

I was tempted to update that function to set the type to 'Integer'
since it is retrieved that way once it must be the case that
it could only be an integer (or that line will fail).

dev/core#2682 Entity batch declare option group for pseudoconstant

Merge pull request #21308 from mattwire/deprecatedjoi

Fix deprecated API4 Join on Email in dynamic profile

Add test to UpdateSubscription form

Fix deprecated API4 Join

Merge pull request #20115 from larssandergreen/fix-internal-anchor-URLs-in-mailings

Do not add tracking to internal anchor URLs

Merge pull request #20135 from larssandergreen/add-settings,-disable,-delete-to-group-page

dev/core#2546 Add settings button to group page

Merge pull request #21307 from civicrm/5.41

5.41

Merge pull request #21295 from eileenmcnaughton/541test

Fix restoreMembershipTypes to use the api

REF Copy preProcessFromAddress back into the pdf function

This is really hard to unravel without splitting it between classes -
mostly because it does stuff to do with setting properties on the form
that are not even relevant to all forms. This is a divide & conquer
case to cleaning up the function from
the deprecated class

Note that I removed the bounce on no-email. It felt like we would still want
to be able to download pdfs if the logged in user had no email. The bounce param seems
to have been added as an after thought - presumably to block
the bounce in a flow where they realised it made no sense

Merge pull request #21303 from civicrm/5.41

5.41

Merge pull request #21302 from seamuslee001/fix_conformance_tests

[NFC] Fix APIv4 Conformance tests on php8

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

This gets rid of the pseudo oo model on this class
and gets us to only having 2 classes in play

We are moving towards a trait

Merge pull request #21291 from eileenmcnaughton/541r

dev/core#2786 Ensure non-English installs have a default financial type for membership default price set

[NFC] Fix APIv4 Conformance tests on php8

Merge pull request #21289 from eileenmcnaughton/tokens

[Ref] intial testing on case tokens, make knownTokens optional

dev/core#2786 Ensure non-English installs have a default financial type for membership default price set

Fix Restore Membership types to create price set

Merge pull request #21300 from colemanw/imageField

SearchKit - Image field handler implementation

SearchKit - Image field handler implementation

Fixes dev/core#2781

Merge pull request #21298 from eileenmcnaughton/silly

[REF] Remove duplicate IF

Merge pull request #21292 from eileenmcnaughton/camps

Fix for new prefetch key

[REF] Remove duplicate IF

Earlier in the function there was already a RETURN if
so this IF is always true