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

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

Merge pull request #28787 from eileenmcnaughton/get_template_vars

Standardise `getTemplateVars()`

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

Merge pull request #28764 from eileenmcnaughton/cart_form

Use local variables for the billing location type ID in event cart code

Merge pull request #28759 from braders/cancel-url

[REF] Tidy up use of _cancelURL property

Merge pull request #28767 from eileenmcnaughton/abstract

Remove bltID as a parameter from getPaymentProcessorReadyAddressParams

Merge pull request #28456 from braders/php82-option

[REF][PHP8.2] Resolve dynamic property usage in CRM_Price_Form_Option

Merge pull request #28739 from MegaphoneJon/contact-fields-to-remove

Unit test for fix to illegal offset error when grouping by contact in SK

Merge pull request #28765 from eileenmcnaughton/map

Fix a few more places that pass bltID to mapParams

Merge pull request #28761 from eileenmcnaughton/cart

Stop setting `_bltID` on the `UpdateBilling` form

Merge pull request #28763 from eileenmcnaughton/payment_form

Use lookup not form property for BillingID in payment form class

Merge pull request #28755 from braders/local-var-taggroup

[REF][PHP8.2] Local variable for tagGroup

Remove deprecated test-only function

Remove bltID as a parameter from getPaymentProcessorReadyAddressParams

Fix a few more places that pass bltID to mapParams