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

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

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

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

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

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

Merge pull request #20751 from artfulrobot/artfulrobot-fix-email-hold-reset

Fix invalid defaults for Mailing.update_email_resetdate API

Merge pull request #20681 from eileenmcnaughton/order_easy

Make Order api easier to use for default price set

Merge pull request #20656 from eileenmcnaughton/cont_mode

[REF] Follow up clean up - remove contribution_mode

Fix invalid defaults for Mailing.update_email_resetdate API

temp

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

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

Move Unit test into its own class and use normal apiv4 wrapper rather than internal class as per Coleman

Merge pull request #20763 from colemanw/joinIdField

SearchKit - use primary_key info instead of assuming 'id'

Merge pull request #20761 from eileenmcnaughton/mem_type

dev/core#2115 Add test cover for membership type page

Merge pull request #20755 from colemanw/afformTweaks

Afform - cosmetic fixes in palette panel

Add test cover for membership type page

APIv4 - Support returning join data from serialized fields

This allows the API to return e.g. an array of display names from a serialized contact reference field.

APIv4 - Remove unused backreferences from SchemaMap

APIv4 - Deprecate Entity::getLinks action

The action returns a bunch of deprecated information which is no longer needed.
The APIv4 Explorer had been using a small portion of it which was actually redundant
with existing info from getFields.

APIv4 - Fix saving multi-contactRef field values

SearchKit - use primary_key info instead of assuming 'id'