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

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

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

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

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 - 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.

SearchKit - use primary_key info instead of assuming 'id'

Merge pull request #20762 from eileenmcnaughton/mem2

dev/core#2115 prep - Remove silly if, silly rename

Merge pull request #20760 from eileenmcnaughton/dep

Remove deprecated function

Merge pull request #20749 from colemanw/apiPrimaryKey

 APIv4 - Rename id_field to primary_key and make it an array

Merge pull request #20757 from eileenmcnaughton/phone_strip

dev/core#27 Move check for phone function to a check

Remove silly if, silly rename

This removes an if which would only be true if civicrm_membership_type table had the
field is_reserved and it was set to true. Given condition 1 is not met, 2
cannot be met.

Also - we rename max_related to maxRelated for no-good-reason

Remove deprecated function

Merge pull request #20743 from eileenmcnaughton/option_type

Improve api consistency on custom field creation

Add primaryKey info to daos

APIv4 - Rename `id_field` to `primary_key` and make it an array

Some entities have a combination of primary keys instead of a single field.
This is allowed in the schema so the API needs to be able to handle it too.
Renaming it for consistency with the schema.

Merge pull request #20748 from seamuslee001/npm_package_updates

[NFC] Update various npm depdencies used only in unit testing

dev/core#27 Move check for phone function to a check

Fixes dev/core#27

Merge pull request #20759 from eileenmcnaughton/entry4

dev/core#2634 [REF] Move membership date calc from v3 api to BAO

Improve api consistency on custom field creation

Api supports a key 'option_values' by always setting the 'magic' param
option_type to 1. v4 doesn't - but creating option values when creating
a custom field is desirable. This fixes so that the form still
'opts out' if it passes in '2' but otherwise we create the option values
if passed in.

Just a teensy bit less magic in the api layer & bao

[REF] Move membership date calc from v3 api to BAO

This moves code to calculate membership dates for new memberships from the
v3 api Membership.create and the batch entry membership form to the membership bao.

I think more cleanup could follow this - in the api most obviously and
the BAO is a bit insane still

This has test cover in the v3 api membership test and in
CRM_Batch_Form_EntryTest

Merge pull request #20729 from demeritcowboy/assign-cleanup

[REF] Remove unnecessary variable wrangling in case roles section

Merge pull request #20754 from colemanw/fixContactSummarySmartyUndefinedVars

Fix undefined tpl vars on contact summary page

Afform - minor fixes

Fix undefined tpl vars on contact summary page

Merge pull request #20750 from mattwire/viewrecur

Add missing "View Recurring Contribution" link to contribution view

View recurring contribution button when displaying lineitems

Merge pull request #20747 from seamuslee001/part_guard

[REF] Add in template guard for  not being always set as per html format

Merge pull request #20586 from colemanw/is_current

APIv4 - add is_current as a pseudo (calculated) field

[NFC] Update various npm depdencies used only in unit testing

[REF] Add in template guard for  not being always set as per html format

Regen SQL

APIv4 - add is_current as a pseudo (calculated) field

This adds the first calculated field to APIv4. It works by injecting a SQL function and aliasing it
with the name of the fake field. This approach allows it to work as part of the WHERE clause
as well as the SELECT clause, even across joins.

Marking the old "is_current" api param deprecated because the field works better.

Merge pull request #20648 from eileenmcnaughton/ret1

[NFC] Improve use of return parameters in ContributionPage test class

Merge pull request #20745 from demeritcowboy/test-20602

[NFC] Unit test for 20602

test for 20602

Merge pull request #20602 from ixiam/dev#PreserveActivityTabFilter

Fix activity_date_time_relative filter when `preserve_activity_tab_fi…

Merge pull request #20627 from demeritcowboy/currency-fixation

dev/financial#175 - Unable to remove a currency once added

Merge pull request #20744 from deb1990/RSESPRT-64-fix-profile-submit-api

Support saving more than 25 records in Profile.submit api

fix activity_date_time_relative filter when `preserve_activity_tab_filter` setting is on

Merge pull request #20396 from eileenmcnaughton/entry3

[REF] Remove isPayLater from membership batch screen

Merge pull request #20394 from eileenmcnaughton/entry

[REF] Extend testing on membership batch entry & minor cleanup

Merge pull request #20676 from eileenmcnaughton/recur

Fix test to use valid financials

Merge pull request #20742 from eileenmcnaughton/throw

Throw exception rather than fatal

Merge pull request #20734 from seamuslee001/fix_patch_php8

[REF] Fix trying to access array offset from boolean value

Merge pull request #20731 from colemanw/apiExplorerWriteJoins

APIv4 Explorer: show joins for write actions

RSESPRT-64: Support saving more than 25 records in Profile.submit api

Merge pull request #20741 from civicrm/5.39

5.39

Merge pull request #20737 from colemanw/fixUndefinedGroupTree

Fix possibly undefined custom groupTree tpl variable

Merge pull request #20735 from eileenmcnaughton/tpl

Tpl enotice fix

Merge pull request #20740 from seamuslee001/5.39

Add release-notes/5.38.1.md

Throw exception rather than fatal

This makes it easier to debug if the entityName is not
resolved - in general this is a dev site issue as it is
to do with load order

Add release-notes/5.38.1.md

Merge pull request #20717 from seamuslee001/use_civi_event_dispatcher_setup

[REF] Alter new Setup process to use CiviCRM's Event Dispatcher

Remove references to savedFieldMapping in page 2 of the import

I do not believe this is ever assigned and I also don't think it makes sense
if assigned - the first page 'DataSource.tpl' is the one that displays the
option to load a field mapping. This code applies to the second page and
conditionally displays help text which seems to have stopped being true long ago

Fix possibly undefined custom groupTree tpl variable

Tpl enotice fix

I searched for hideStepNumbers & believe it is never assigned

[REF] Fix trying to access array offset from boolean value

[REF] Alter new Setup process to use CiviCRM's Event Dispatcher

Apply patch from Tim to limit what events are dispatched