Fix undefined tpl vars

Merge pull request #20315 from mattwire/recaptchahelpers

(REF) dev/core#2571 - Add helper functions for reCAPTCHA extension

Add helper functions for reCAPTCHA extension

Merge pull request #20817 from colemanw/fixBasicGetColor

APIv4 - Fix dumb typo breaking color lookup

Merge pull request #20816 from colemanw/optionsPageFixes

PHP notice fixes on options page

Merge pull request #20399 from jaapjansma/dev_2624_2

dev/core#2624  display line items from the template contribution at the view recurring contribution screen.

Merge pull request #20819 from seamuslee001/fix_apiv4_syntaxtests

[REF] Add in APIv4 Product Entity and Fix APIv4 Syntax Conformance te…

[REF] Add in APIv4 Product Entity and Fix APIv4 Syntax Conformance test failures on ContributionProduct and EntityBatch APIs

update version

Fix dumb typo breaking color lookup

PHP notice fixes on options page

Merge pull request #20815 from civicrm/5.40

5.40 to master

Merge pull request #20814 from seamuslee001/dev_core_2680

dev/core#2680 Remove rows in the civicrm_contribution_product table t…

Merge pull request #20813 from colemanw/tplFixes

Fix undefined tpl variables

Merge pull request #20505 from JMAConsulting/contribution_product_api4

Add ContributionProduct and EntityBatch APIv4 Entity

dev/core#2680 Remove rows in the civicrm_contribution_product table that have bad product_ids in them

Merge pull request #20782 from mlutfy/forwardMail

Forward Mailing: remove duplicate buttons, cancel button

Fix undefined tpl variables

Merge pull request #20812 from nirodaonline/patch-1

Update contributor-key.yml

Update contributor-key.yml

Merge pull request #20805 from eileenmcnaughton/log_date

Add deprecation warning for modifiedDates

Merge pull request #20794 from nirodaonline/feature/core-2652

dev/core#2652 - Civi\Report - Add is null / is not null to OP_MULTISE…

Fix for dev/core#2624

Fix for dev/core#2624

Merge pull request #20758 from colemanw/addSearchAfformTest

SearchKit - Auto-apply filters passed in from Afform markup

Merge pull request #20786 from totten/master-monaco

crmMonaco - Multiple updates. Toward message-template editing.

Merge pull request #20806 from eileenmcnaughton/lines

Fix test to use order api & setup valid financials

Fix test to use order api & setup valid financials

Updates a test from 'weird setup' to using the order api

Set version to 5.41.alpha1

Set version to 5.40.beta1

Merge pull request #20810 from totten/master-e2e-timeout

AssetBuilderTest - Raise threshold for timeout

AssetBuilderTest - Raise threshold for timeout

Overview
--------

Raise the threshold beforer abandoning execution of this E2E test.

Technical Details
------------------

This aims to mitigate flaky failures like the following:

Example failure: https://test.civicrm.org/job/CiviCRM-Core-Matrix/BKPROF=max,CIVIVER=5.39,SUITES=phpunit-e2e,label=bknix-tmp/9766/testReport/junit/E2E.Core/AssetBuilderTest/testInvalid/

```
E2E\Core\AssetBuilderTest::testInvalid
GuzzleHttp\Exception\ConnectException: cURL error 28: Operation timed out after 1001 milliseconds with 0 bytes received (see https://curl.haxx.se/libcurl/c/libcurl-errors.html)

/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:200
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:155
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:105
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Handler/CurlHandler.php:43
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Handler/Proxy.php:28
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Handler/Proxy.php:51
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/PrepareBodyMiddleware.php:37
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Middleware.php:29
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/RedirectMiddleware.php:70
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Middleware.php:59
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/HandlerStack.php:71
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Client.php:351
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Client.php:162
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/vendor/guzzlehttp/guzzle/src/Client.php:182
/home/jenkins/bknix-max/build/build-0/web/sites/all/modules/civicrm/tests/phpunit/E2E/Core/AssetBuilderTest.php:167
/home/jenkins/bknix-max/extern/phpunit8/phpunit8.phar:671
```

Merge pull request #20809 from totten/master-fwd

Merge forward 5.39 => master

set-version.php - Fix autocommit for 'info.xml'

The 'set-version.php' script has an option `--commit` which should commit any updated files.

Before: Fails to commit changes to `ext/*/info.xml`

After: Does ocmmit changes to `ext/*/info.xml`

release-notes/5.39.0.md - Add recent PR

release-notes/5.39.0.md - Random tiny edits

dev/core#2659 Move group cleanup to latest release

Merge pull request #20803 from colemanw/addressSearchFilter

SearchKit - Remove automatic filter from address

Merge pull request #20600 from ixiam/dev#SettingsTraitCheckboxes

Force cleanup of setting value type "checkboxes" when is empty

Add deprecation warning for modifiedDates

This is only accessed from log_date

Merge pull request #20764 from colemanw/multiContactRefJoins

SearchKit - Add ability to join on multi-select ContactRef fields

Merge pull request #20785 from eileenmcnaughton/baseipn

Fix BaseIPN test to (mostly) pass invalid financials checks

Merge pull request #20746 from colemanw/locBlockFixes

dev/core#2676 SearchKit - Add LocBlock entity to display Event location data

SearchKit - Remove automatic filter from address

Merge pull request #20780 from eileenmcnaughton/pay_order

Fix test set up to user Order api

Merge pull request #20802 from civicrm/5.39

5.39 to master

Merge pull request #20798 from agh1/5.39.0-releasenotes-final

5.39.0 release notes: added late changes

Merge pull request #20765 from colemanw/apiDeprecatePassingIdField

APIv4 - Deprecate passing 'id' to basic actions

Merge pull request #20719 from mattwire/deprecatedwarningcaller

Add calling function to deprecatedWarning log message

Fix test set up to user Order api

Note this includes the order api ensuring that the contribution_recur_id is passed through to
any created memberships

Merge pull request #20797 from colemanw/reportTplFixes

Fix undefined tpl vars in CiviReport

CRM_Utils_JS::stripComments() - Mitigation for string which includes newline and comment

5.39.0 release notes: added late changes

Add deprecated flag to functions

Add calling function to deprecatedWarning log message

Fix undefined tpl vars in CiviReport

Merge pull request #20796 from demeritcowboy/smartactlinks

Unused smarty variable

Merge pull request #20795 from demeritcowboy/smartycasehlp

smarty notices - New case screen - Case.hlp

Merge pull request #20793 from eileenmcnaughton/order2

[Ref] Remove (more) unreachable code from order api

unused

sometimes legitimately missing

Further cleanup on order

Follow up to https://github.com/civicrm/civicrm-core/pull/20792
- although that can be closed if this is merged

[REF] Remove unreachable code from order api

dev/core#2652 - Civi\Report - Add is null / is not null to OP_MULTISELECT_SEPARATOR

Merge pull request #18517 from JMAConsulting/financial-issue-150

dev/financial#150: civicrm/payment/form url got empty currency argument in backoffice live CC form

Merge pull request #20392 from seamuslee001/ckeditor4

[REF] Extract CKEditor into its own core extension

Merge pull request #20790 from eileenmcnaughton/partor

Handle participant payment create in line item

Merge pull request #20789 from totten/master-crmui-selecting

(REF) crmUi - Support onCrmUiSelect for using select2 as a picklist

Extract CKEditor into its own core extension

Update civicrm_generated and shift js files into the new extension

Update to use E::path and E::url as appropriate

Add in Upgrade step to update option label to be consistant

Make changes for CKeditor 4 label now

Add in upgrade step to enable extension on upgrade

Handle participant payment create in line item

Remove participant payment create from order api

This is now done in the line item create & we went through to 'prove' that it works with
extra validation

Merge pull request #20783 from JMAConsulting/issue_2681

dev#2861: Prevent duplicate names in mailing labels

Merge pull request #20788 from totten/master-array-rotate

CRM_Utils_Array::asColumns() - Add helper to rotate a matrix (from rows to columns)

Merge pull request #20787 from totten/master-path-unset

CRM_Utils_Array - Implement pathUnset() method

crmUi - Add onCrmUiSelect for using select2 as a picklist

Fix BaseIPN test to (mostly) pass invalid financials

Merge pull request #20779 from eileenmcnaughton/batch2

 Clean up code to determine line items for membership batch entry

CRM_Utils_Array - Add helper to rotate an array (from rows to columns)

CRM_Utils_Array - Implement pathUnset() method

composer.json - Update Monaco

crmMonaco - Add option to receive inserted snippets

crmMonaco - Provide internal access to `$ctrl.editor` / `crmMonaco.editor` instances

crmMonaco - Accept option to specify relative height

Merge pull request #20781 from colemanw/tplFixes

Fix undefined indexes in tpls

Clean up code to determine line items for batch entry

This removes a chunk of weird code & further consolidates on the internal BAO_Order helper.

Note the test cover is in CRM_Batch_Form_EntryTest, which has
validateFinancials enabled

Afform - Support implicit joins in SearchDisplay filter fields

SearchKit - Fix array filters with serialized fields

SearchKit - Allow multi-valued contact ref fields to be formatted as links

This handles multivalued fields by looping through each item and formatting it as a link

Forward Mailing: remove duplicate buttons, cancel button

Prevent duplicate names in mailing labels

Merge pull request #20778 from eileenmcnaughton/batch

[REF] Tidy up param transforms

Merge pull request #20728 from demeritcowboy/missing-end-date

Error when sorting case roles table by end date

Merge pull request #20738 from eileenmcnaughton/tpl2

Enotice fix Remove references to savedFieldMapping in page 2 of the import

Fix undefined indexes in tpls

Merge pull request #20672 from eileenmcnaughton/mem_num_terms

dev/membership#29 Add civicrm_line_item.membership_num_terms

[REF] Tidy up param transforms

This just moves the transforms to their own function & comments them. I have previously
added test cover which should cover this well.

The parameters used for profiles are not the 'real' parameters so this does the transform

Merge pull request #20774 from civicrm/5.39

5.39

dev/membership#29 Add civicrm_line_item.membership_num_terms

Specify table in query (no change in query meaning)

Merge pull request #20727 from colemanw/deprecateApi4EntityLinks

APIv4 - Deprecate Entity::getLinks action

APIv4 - Deprecate passing 'id' to basic actions and standardize update actions

The primary_key is now known for every entity so it doesn't need to be passed
into the action classes, and both BasicUpdate and DAOUpdate can function off this key.

This accepts the old constructor arguments for backward-compatablity, for now.

Merge pull request #20773 from alifrumin/rn5.39

[NFC] First Stab at 5.39 Release Notes

Merge pull request #20770 from demeritcowboy/smartydash

smarty notice - case dashboard when no upcoming/recent cases

release notes 5.39 - first pass