Minor cleanup in test class

Merge pull request #28844 from eileenmcnaughton/date_non_us

Fix handling of non-padded day in import date in dd/mm/yyy format

Merge pull request #28843 from civicrm/5.69

5.69

Merge pull request #28827 from colemanw/getReferenceColumns

DAO - Return reference columns based on field metadata

Fix handling of non-padded month in dd_mon_YY format

Fix time handling for us-style

Add consistent time-format support for US formats, add test for our magic year-handling

Merge pull request #28842 from MegaphoneJon/price-field-weights

Price field and value weights not respected

Merge pull request #28647 from eileenmcnaughton/delete

Move implementation of financial type acl out of core (leverage existing extension hooks)

fix price field and value weights

Remove duplicate check

Do not call core function from extension

Move financial type check from event register page

Move link handling to extension

Move check permissioned line items to acl now it is not used in core

Move implementation of financial type acl out of core (leverage existing extension hooks)

Merge pull request #28668 from colemanw/utilsArrayCollect

REF - Swap CRM_Utils_Array::collect with vanilla array_column

Merge pull request #28828 from eileenmcnaughton/date_iso

Fix time validation for iso date format

Merge pull request #28834 from colemanw/phpStormGetLinks

[NFC] Add 'getLinks' action to phpStorm extension

Merge pull request #28837 from colemanw/group_notice2

Fix e-notice on contact create for low-permissioned user

Merge pull request #28786 from colemanw/oauthLinks

dev/core#4674 - Add Oauth links for AdminUi MailSettings

Merge pull request #28838 from spalmstr/Issue_4879

Issue 4879 second attempt

Following suggested additional changes.

Fix e-notice on contact create for low-permissioned user

This cleans up some very messy code which was emitting e-notices when the 'group' key was missing.
This can happen when the current user does not have permissions to add contacts to groups.

Issue 4879

Merge pull request #28831 from eileenmcnaughton/template_vars_sample

Remove smarty2 functions from sample code

Merge pull request #28829 from eileenmcnaughton/template_vars

Use getTemplateVars in smarty help function

Merge pull request #28830 from eileenmcnaughton/template_smarty

Replace getTemplateVars() in test, assignTemplateVariables

Merge pull request #28621 from artfulrobot/issue-4857

standalone: implement cms-prefixed permissions as concrete ones

Merge pull request #28833 from artfulrobot/standaloneusers-dependency

Remove missingStandaloneExtension() checks which were pre-alpha scaffold

Add 'getLinks' action to phpStorm extension

Remove missingStandaloneExtension() checks which were pre-alpha scaffold

Merge pull request #28784 from mlutfy/floatLeftButton

Merging as no objections

Remove smarty2 functions from sample code

Replace getTemplateVars() in test, assignTemplateVariables

Use getTeplateVars in smarty help function

Combine convertToDefaultDate back into only calling function

It is not a logical separation of concerns & makes code more complex
rather than simpler

Fix time validation for iso date format

As this is a very simple format strtotime will get it right
so we just fix our validation & simplify

Merge pull request #28826 from eileenmcnaughton/act_workflow

Use distinct test workflow message for workflow message tests

Disable check temporily for PR juggling

dev/core#4674 - Add Oauth links for AdminUi MailSettings

This fixes https://lab.civicrm.org/dev/core/-/issues/4674 by adding a LinksProvider
for the new GetLinks action from #27973.

It uses the CRM_Core_BAO_MailSettings::getSetupActions() function to generate the extra links,
but there is a mismatch between the two functions:

- getSetupActions() returns a fully-formed url
- API::getLinks() returns a path

However, I noticed that while in theory hook_civicrm_mailSetupActions could return various urls,
in practice they are all the same: a generic civicrm/ajax/setupMailAccount redirect.
So this returns that as the path and it works.

Perhaps we should deprecate the option to return a url from hook_civicrm_mailSetupActions
(or just remove it, since it appears to be unused & YAGNI).

Regenerate Ext DAO files without boilerplate getReferenceColumns()

Regenerate Core DAO files without boilerplate getReferenceColumns()

DAO - Return reference columns based on field metadata

This deletes getReferenceColumns() from the DAO boilerplate and adds all the necessary field metadata
so that they can be generated by the parent class.

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

Extract date regex validation

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