Merge pull request #28826 from eileenmcnaughton/act_workflow

Use distinct test workflow message for workflow message tests

Disable check temporily for PR juggling

Merge pull request #27973 from colemanw/getLinks

SearchKit - New getLinks API action for advanced link handling

Merge pull request #28789 from eileenmcnaughton/message_template

Update `MessageTemplates` generation to work with Smarty3

Use distinct test workflow message for workflow message tests

The tests for the workflow message subsystem use an instance of a test
case_activity template attached the real message workflow template. They
test the text version - meaning that removing it, like we have with
others, causes various fails. In addition adding a real template that
does not match the test version causes mayhem.

In this case we want to ensure we keep all the tests for the subsystem
functionality but undo the tethering to a particular
template to allow that template to have updates without
having to go deep into this code

Merge pull request #28825 from eileenmcnaughton/no_ref

Remove function use of reference

Merge pull request #28822 from eileenmcnaughton/date_formats_test

Extend date formats being tested

Merge pull request #28823 from eileenmcnaughton/date_const

Use constants for date formats

Merge pull request #28556 from eileenmcnaughton/trait_meta

Fully deprecate MetadataTrait

Deprecate passing in month

Remove pass-by-reference

We have removed a bunch of these (added for php4.x support) with no
issues - it's only where there is inheritance we have
hit issues

Use constants for date formats

Extend date formats to be tested

Merge pull request #28821 from eileenmcnaughton/add

Fix error due to merges co-inciding

Fix error due to merges co-inciding

Merge pull request #28808 from colemanw/canAggregate

SearchKit - Allow aggregate functions to be used without GroupBy

Merge pull request #28818 from civicrm/5.69

Merge forward (5.69-rc to master)

Merge pull request #28718 from eileenmcnaughton/2_term

dev/core#3810 dev/core#3339 Fix handling of auto renew memberships where frequency_interval > 1

Merge pull request #28811 from colemanw/searchDisplayRemoveColFix

SearchKit - Fix wrong column being removed from displays

Merge pull request #28812 from colemanw/removeContactTypeField

APIv4 - Hide 'contact_type' from Individual,Organization,Household getFields

Merge pull request #28783 from braders/php8-last-participant

[REF][PHP8.2] Declare _lastParticipant property

Merge pull request #28815 from braders/event-additional-particiapant-properties

[REF][PHP8.2] Declare missing properties on CRM_Event_Form_Registration_AdditionalParticipant

Merge pull request #28775 from seamuslee001/dev_core_4841

dev/core#4841 Ensure that APIv4 have same default values which ensure…

Merge pull request #28816 from braders/php8-discounts

[REF][PHP8.2] Cleanup discounts property in eventcart ext

dev/core#4841 Ensure that APIv4 have same default values which ensures that all required selects are filled out when creating new bulk mailing from Search Kit

Update as per coleman and fix Test fails

Fix calls to defaultComponent to fix test fails

Merge pull request #28730 from eileenmcnaughton/recur_cancel_api_2

Stop bypassing hooks on ContributionRecur.cancel

[REF][PHP8.2] Cleanup discounts property in eventcart ext

[REF][PHP8.2] Declare missing properties on CRM_Event_Form_Registration_AdditionalParticipant

APIv4 - Remove 'contact_type' from Individual,Organization,Household

This field is irrelevant to these pseudo-entities and shouldn't need to be used at all.

Before: Creating a new SearchKit search for Individual includes Contact Type in the default columns.

After: Now it doesn't, and the field is completely hidden for all but the Contact entity.

Merge pull request #28799 from eileenmcnaughton/date_consolidate

Consolidate import date handling

Merge pull request #28753 from eileenmcnaughton/search_settings

Clean up search settings page, update group cache text

Merge pull request #28748 from eileenmcnaughton/dep

Bubble up deprecation if getVar used to get a deprecated property

Merge pull request #28814 from MegaphoneJon/anonymous-acl-cache

Don't try to insert 0 as a user_id to ACL contact cache

Merge pull request #28813 from colemanw/deletePhantomJsPolyfill

Remove unused phantomjs-polyfill package

Remove unused phantomjs-polyfill package

As of 23051b6 we no longer use PhantomJS so this package should not be needed.

Don't try to insert 0 as a user_id to ACL contact cache

SearchKit - Fix wrong column being removed from displays

Fixes recent regression from 4099cd6 causing the wrong column to be removed from a display.
To reproduce: Create a search display, go back to the main search criteria and delete a column,
then tab back to the display. The deleted column should have been removed but instead the wrong
one is still there.

Merge pull request #28776 from eileenmcnaughton/submit

Remove api annotation from getSubmittableFields()

Merge pull request #28809 from civicrm/5.69

5.69 to master

Merge pull request #28807 from alifrumin/5.69rn

[NFC] 5.69 Release Notes first pass

SearchKit - Allow aggregate functions to be used without GroupBy

Fixes https://lab.civicrm.org/dev/core/-/issues/2415

Merge pull request #28804 from eileenmcnaughton/regen

Update regen for event message template

[NFC] 5.69 release notes first pass

Merge pull request #28803 from eileenmcnaughton/notice

E-notice fix in membership workflow examples

Merge pull request #28798 from eileenmcnaughton/date_test

NFC Superficial cleanup in test class

Merge pull request #28805 from eileenmcnaughton/code_notes

[NFC] Add todos to the invoice code

Add todos to the invoice code

This helps clarify that most of this code is slated for removal
in order to make the forms non-form-dependent

Update MessageTemplates generation to not use weird old methods

Update regen for event message template

Merge pull request #28801 from eileenmcnaughton/fork/mapping-edit-button-fix

Fix edit button for 'Import/Export Mapping'

E-notice fix in membership workflow examples

Fix edit button for 'Import/Export Mapping'

Consolidate import date handling

NFC Superficial cleanup in test class

Merge pull request #28797 from eileenmcnaughton/nfc

[NFC] Superficial cleanup

[NFC] Superficial cleanup

Merge pull request #28788 from eileenmcnaughton/assign

Fix codeGen assign to do what it says on the box

Merge pull request #28738 from mlutfy/maxSpin

menubar spin: stop after 10 seconds

Merge pull request #28794 from demeritcowboy/del

update deleted files list

deleted files list

Merge pull request #28793 from civicrm/5.69

5.69

Merge pull request #28781 from vingle/5.69-ux-regressions

Fixes two small accordion UX regressions

Merge pull request #28673 from bb/migrate-snappy-to-composer

Migrate Snappy to composer

Merge pull request #28792 from civicrm/5.69

5.69 to master

Merge pull request #28791 from eileenmcnaughton/569

Fix typo in smarty compatibility class

Merge pull request #28782 from mlutfy/extList

Manage Extensions: hide the type/module column

Manage Extensions: hide the type/module column

Fix typo in smarty compatibility class

Merge pull request #28790 from eileenmcnaughton/str_contains

Remove str_contains from civicrm.settings template

SearchKit - Show multiple 'create' links in toolbar

Merge pull request #28787 from eileenmcnaughton/get_template_vars

Standardise `getTemplateVars()`

APIv4 - Fix getLinks permission checks

The getLinks action checks all permissions internally, so should not have any
external gatekeeper checks applied.

SearchKit - Use getLinks api in formatLink

SearchKit - Preprocess links only once; use api getLinks action

Before: Links were being preprocessed by the getLinkInfo() function once-per-link-per-row.

After: Links preprocessed once-per-link. New API action used.

APIv4 - Alter activity links for CiviCase

Activities that belong to a case use a differently-structured url.
This adds a handler for it and moves it more appropriately into the
civi_case extension.

APIv4 - Make Activity::getLinks return multiple add links per activity type

SearchKit - Use getLinks API action

APIv4 - Add getLinks action to CustomValue API

APIv4 - Add getLinks action

Remove str_contains from civicrm.settings template

Fix codeGen assign to do what it says on the box

Standardise getTemplateVars()

Merge pull request #28785 from MegaphoneJon/delete-grant-false-error

remove incorrect message that grant could not be deleted

remove incorrect message that grant could not be deleted

Merge pull request #28780 from eileenmcnaughton/mut

Compiance with php8 & Marvel9

[REF][PHP8.2] Declare _lastParticipant property

Compiance with php8 & Marvel9

Merge pull request #28771 from eileenmcnaughton/blt_cms

Remove checks to bltID in cms classes

Merge pull request #28770 from eileenmcnaughton/blt

Stop referring to `_bltID` in AbstractEditPayment

Adds support for fieldset

In this commit: https://github.com/civicrm/civicrm-core/commit/1602ec3e104de40e7bd785496f0cde323fe870ed the `display: none` was narrowed to only target divs with the class '.collapsed'. This excluded a few examples of fieldsets being used (as identified in ThemeTest, Accordion 4 - https://lab.civicrm.org/extensions/themetest/-/blob/main/ang/themetest/snippets/accordion4.html)

Keeps search criteria fieldset closed when results found

Fixes search regression introduced in #28421 and identified in #28661.

Merge pull request #28760 from eileenmcnaughton/case_dec

Declare caseId, caseIds as public properties

Merge pull request #28773 from eileenmcnaughton/cart_82

[PHP8.2] Fix test fails on event cart

Remove api annotation from getSubmittableFields()

This should either be public or api-supported. Perhaps it should be public/api but
at least for now I will resolve by removing the annotation

[PHP8.2] Fix test fails on event cart

Stop referring to _bltID in AbstractEditPayment

Merge pull request #28758 from braders/php8-smsCommonTest

[REF][PHP8.2] Use correct form object to ensure properties exist

Remove checks to bltID in cms classes

Merge pull request #28768 from eileenmcnaughton/dep_test

Remove deprecated test-only function

[REF][PHP8.2] Use correct form object to ensure properties exist