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…

APIv4 - Fix pseudoconstant matching function to pass params correctly across joins

The getPseudoconstantList() function passes an array of params to core BAO::buildOptions,
that list needs to be transformed to match the prefix on the field, in case that field
belongs to a joined entity rather than the main entity.

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

Enotice fix on extensions page

I dug around and I think upgradable is an obsolete row value - I couldn't find where it
was set - including when I marked an extension as an older version in info.xml

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

[REF] Simplify isRenew handling on batch for membership

This starts the process of using getters to get data from the row and also
marks the no-longer shared emailReceipt function as no longer deprecated
(since it was copied back to this form early).

Clean Money moved to 'stdiseRow'

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

[REF] Minor auth.net ipn simplification

This removes some property setting on the contribution
which is then discarded & moves a bit of validation to be
with the other validation

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

Merge pull request #20769 from eileenmcnaughton/acl_test

dev/core#2115 Move financial acl code out of v3 Contribution create

Merge pull request #20771 from demeritcowboy/smartyrep2

smarty notice - title is only set for a list of existing report instances

Merge pull request #20772 from demeritcowboy/smartyrep

smarty notice - confirm_message is usually irrelevant in report listings

title is only set for a list of existing instances

sometimes it's legitimately missing

ensure var exists

Merge pull request #20733 from seamuslee001/is_deleted_unit_test

[NFC] Add in unit test to lock in the fix for the is_deleted in where…

Merge pull request #20321 from mattwire/loggedincontactid

Deprecate duplicate getLoggedInContactID() function