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

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

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

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

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

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

SearchKit - Improve join labels

This fixes the order of joins, fixes a bug where the label fields were getting overwritten by
implicit joins before they could be used for explicit joins,
and uses the label field for explicit joins instead of just the entity name.

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

Cleanup labels in LocBlock entity and expose to SearchKit