[REF] - Cleanup Contact BAO variables

Merge pull request #22619 from demeritcowboy/wrong-tz

dev/core#2122 - Manage event online registration page doesn't determine timezone correctly

Merge pull request #22538 from masetto/pdfletter

add `contact_id` to email params in `emailLetter` function

Merge pull request #22587 from colemanw/apiNull

APIv4 - Use empty string instead of 'null' to pass null values to the db

Merge pull request #22634 from eileenmcnaughton/pre_message

dev/core#2866 add pre-upgrade warning

Merge pull request #22652 from civicrm/5.46

5.46

Merge pull request #22669 from agileware/CIVICRM-1921

CiviCRM Event, the link to "On-line Registration (Test Drive)" is an admin-only URL to register in the backend, should be a frontend URL

Merge pull request #22668 from braders/profile-preview-template-formatting

[REF] Tidy up group.tpl

Tidy up group.tpl

This rationalises the indentation, and removes an extra closing </div> which was causing issues in some contexts.

Merge pull request #22653 from colemanw/searchKitOperatorFixes

SearchKit - Fix selecting operators and values in clauses

Merge pull request #22663 from braders/manage-event-listing-default-values

Set default values to avoid notices when rendering (event templates screen)

Merge pull request #22660 from braders/membership-dashboard-smarty-notices

Fix smarty notices in membership dashboard

Merge pull request #22661 from braders/contributionpage-php-notice

Avoid PHP notice on 'Manage Contribution Pages' screen

Set default values to avoid notices when rendering

Avoid PHP notice on 'Manage Contribution Pages' screen

Fix smarty notices in membership dashboard

Merge pull request #22658 from braders/phpdoc-fixes-3

Another set of PHPdoc corrections

Merge pull request #22659 from braders/const-type-hints

Correctly document types for class consts

Another set of PHPdoc corrections

Correctly document types for class consts

Merge pull request #22656 from seamuslee001/5.46

5.45.1 & 5.45.2 Release Notes

Add release-notes/5.45.2.md

Add release-notes/5.45.1.md

SearchKit - Fix selecting operators and values in clauses

Fixes dev/core#3046 which was caused by incomplete refactoring in
https://github.com/civicrm/civicrm-core/pull/22604

Merge pull request #22358 from colemanw/searchKitNotNull

SearchKit - Improve editable UX by not allowing required fields to be left blank

Merge pull request #22650 from totten/5.46-greeting

dev/core#3028 - For invalid greetings, return '' instead of failing

CIVICRM-1921 Event, the link to "On-line Registration (Test Drive)" is an admin-only URL to register in the backend, should be a frontend URL

dev/core#3028 - For invalid greetings, return '' instead of failing

Merge pull request #22649 from colemanw/api4CustomDateFormat

APIv4 - Correctly return date-only custom field values without the time

APIv4 - Correctly return date-only custom field values without the time

Merge pull request #22646 from braders/feature/tasktrait-issingle-docblock

Resolve docblock inaccuracy in CRM_Contribute_Form_Task_TaskTrait

Merge pull request #22645 from demeritcowboy/event-default-tz

dev/core#2122 - Add an event timezone for sample data

Merge pull request #22644 from demeritcowboy/fix-add-version

dev/core#2122 - Fix "add" version for event timezone dao field

Resolve docblock inaccuracy in CRM_Contribute_Form_Task_TaskTrait

default event timezone for sample data

fix add version

Merge pull request #22639 from civicrm/5.46

5.46

Merge pull request #22640 from totten/5.46-ext-check

Extensions - Add dependency status check

Merge pull request #22638 from totten/master-ext-prose

Extensions - Revise prose for status check

Extensions - Revise prose for status check

Extensions - Add dependency status check

Adds a system status check to ensure extension dependencies are met

Merge pull request #22623 from colemanw/extensionUpgrades

ExtensionUpgrades - Skip trying to upgrade missing dependencies

Extensions - Revise prose for status check

ExtensionUpgrades - Skip trying to upgrade missing dependencies

Before: Missing dependencies would be added and then the extension upgrader would try and fail to upgrade them
After: They are skipped, as missing extension dependencies are handled elsewhere

This allows the extension upgrade to proceed without error, even if there are missing dependencies.
The user will be prompted to install the missing dependencies afterward.

Merge pull request #22626 from colemanw/readOnlyRename

APIv4 - Rename 'ReadOnly' trait to 'ReadOnlyEntity' to avoid PHP error

Merge pull request #22631 from braders/calculateBaseScheduleDate-docblock

Fix inaccuracy in docblock for method calculateBaseScheduleDate

dev/core#2866 add pre-upgrade warning

Merge pull request #22629 from braders/attachmentInfo-docblock

Resolve inaccuracies in CRM_Core_BAO_File::attachmentInfo docblock

Merge pull request #22601 from braders/set-print-functions-tidy

Tidy up functions related to printing.

Fix inaccuracy in docblock for method calculateBaseScheduleDate

Resolve inaccuracies in CRM_Core_BAO_File::attachmentInfo docblock

Tidy up functions related to printing.

Calls to setPrint updated to use constant,
and inline documentation updated to reflect the values which are actually expected.

setWord and setExcel methods deprectated as they are not in use,
and they only support the older Microsoft Office formats (doc, not docx)

Merge pull request #22625 from eileenmcnaughton/comp

Do not pass in tablename

APIv4 - Add `nullable` property to getFields; improve SearchKit editable UX

Unlike the 'required' field property, which only determines if the API requires a value to Create,
the 'nullable' property tells a UI whether a field is allowed to be set to NULL in Create OR Update.

SearchKit uses this property during in-place edit and bulk edit operations to determine whether a
field can be left blank.

Merge pull request #22627 from eileenmcnaughton/config

Remove deprecated language construct

APIv4 - Rename 'ReadOnly' trait to 'ReadOnlyEntity' to avoid PHP fatal error

Fixes dev/core#3043
The word 'readonly' is reserved as of php 8.1

Remove deprecated language construct

Per https://github.com/totten/civix/pull/235 this language construct
appears to be getting more noisy about being bad....

Merge pull request #22615 from eileenmcnaughton/online

Remove isset from online contribution receipt

Merge pull request #22551 from eileenmcnaughton/phpcs

More phpcbs code cleanups

Merge pull request #22606 from colemanw/relationshipPseudoFields

SearchKit - Expose relationship description, created/modified_date & permissions

Merge pull request #22620 from demeritcowboy/adjust-version

dev/core#2122 - Fix version specified in message template upgrade

Merge pull request #22592 from colemanw/searchKitSmarty2

SearchKit - Allow smarty in field rewrite

Do not pass in tablename

Tablename is fetched if not passed in so no need to do the fetch first

Merge pull request #22622 from demeritcowboy/unused-tz

dev/core#2122 - prevent E_NOTICEs and blank dates being converted to current datetime

Updated generated sql

Remove isset from online contribution receipt

This removes the isset checks that re failing around
totalTaxAmount (solved the same way as offline
https://github.com/civicrm/civicrm-core/pull/22560/files#diff-fd5668d5492e5f0ec55b7f8a10eccfa4ea9de249e4cf2383a2e0d36dbd92ebe0R134
)

It also removes the isset around contribution.amount_level (switching
to a token) and the enotice on isQuickConfig - this
switches to loading that information & also using a more meaningful variable.
Note that line items should always be assigned so that check is silly.

Merge pull request #22519 from colemanw/customGroupCleanup

[REF] CustomField Admin - Decouple page code from form code

prevent E_NOTICE and prevent blank dates being converted to current date/time

CustomField Admin - Decouple page code from form code

This moves the forms for creating/editing/previewing/deleting custom data
to their own url paths instead of sharing the same path as the page.

wrong variable

fix version

Merge pull request #20476 from agileware/CIVICRM-404

dev/core#2122 Add timezone support for events

Merge pull request #22618 from eileenmcnaughton/batch

Fix batch summary to use the api

Merge pull request #22617 from eileenmcnaughton/view

Split View Relationship tpl ito own file

Merge pull request #22548 from eileenmcnaughton/raw

Add option to display money tokens without formatting

Fix batch summary to use the api

In looking at @colemanw's work to clean up the retrieve function it
turned out that in at least one place NULL is being passed to Batch::retrieve

Since I dug into that 1 place I fixed it to use v4 api (it is
the screen accessed after creating a new accounting batch
from contribution->accounting batches->new batch)

However, it seems safer to continue to accept NULL

CIVICRM-1773 Add Event timezone tests.

CIVICRM-1763 Add Event component check for missing timezones.

CIVICRM-1761 Add event_tz default to API spec.

CIVICRM-1753 Add event timezone information to message templates.

CIVICRM-1758 Add timezone output for Event Tokens.

CIVICRM-1756 Update Event ICalendar & List output to use timezone

CIVICRM-1749 Add timezone output to Event Registration forms

CIVICRM-1749 Add time elements and timezone to Event Info output.

CIVICRM-1749 Implement UI for timezone in Event management forms

CIVICRM-1754 Add timezone to Event Schema.

Split View Relationship tpl ito own file

The inclusion of  templates/CRM/Contact/Form/Relationship.tpl simply contains a chunk
of code for the view file in a big IF - this separates that out it's own file & is
more thoughtful about the inclusion....

Merge pull request #22547 from eileenmcnaughton/smart

Only format smarty aliases as money if specified

Merge pull request #22613 from eileenmcnaughton/notice

Fix smarty notices in footer files

Merge pull request #22314 from braders/core-3004-remove-fontsize-classes

Remove use of .font-size11pt and .font-size12pt

Fix smarty notices in footer files

Only format smarty aliases as money if specified

smarty aliases allow tokens to be assigned to the template as smarty variables. They are very new,
have only been implemented in these 2 places and are not accessible from outside of core, which
both have unit test cover.

When trying to extend it to cover another variable (totalTaxAmount) in another template I found
that a raw format is appropriate. I think it makes sense to explicitly specify.

Code comment

Co-authored-by: Tim Otten <totten@civicrm.org>

Merge pull request #22611 from yashodha/report_cleanup

report cleanup - no functional change

Remove use of .font-size11pt and .font-size12pt

Merge pull request #20845 from herbdool/issue-899

Issue #899: don't show PCPs where the events or contribution pages are disabled or past the end date

update doc block, work on `PDFLetterCommonTest` unit test compatibility with `contactId` parameter

Merge pull request #20251 from larssandergreen/change-registration-button-text

dev/core#1613 Change event registration button text based on if there are additional participants

Merge pull request #22604 from colemanw/searchKitOperators

SearchKit - Improve field/operator/value selection UI

report cleanup

Merge pull request #22610 from colemanw/deprecatedUnusedFunction

Contact BAO - Deprecate unused function