RecurringEntity - Cleanup godawful form code

Before - Form code is terrible
After - Form code still deeply flawed but a bit easier to read

This form code belongs in a BAO where it can have an api and proper tests
but this at least cleans it up a bit to make it legible.

RepeatingEntity - Update tests with correct name for ActionSchedule

Follow-up from a47a0396 this updates the tests to match the (ugh) form code
(that form code really belongs in a BAO which can be properly tested!)

Merge pull request #27327 from demeritcowboy/weight

Set link weight on manage case

weight

Merge pull request #27320 from eileenmcnaughton/processor

Remove unused parameter, pass by reference in private function with one caller

Merge pull request #27323 from eileenmcnaughton/weight

Define weight for links on extensions page, financial types, financial accounts

Merge pull request #27326 from civicrm/5.65

5.65 to master

Merge pull request #27324 from eileenmcnaughton/565

Fix enotice (possible regression)

Add weights to contribution page links

Fix enotice (possible regression)

It's clear in the function that lineItem is only defined for add - I hit the enotice editing
the participant to be paid. Not sure if this is a regression but there was a related one
fixed in 5.65 or 5.64 so targetting the rc

Add weight to contribution tasks

Add weights to Manage Event links

Add weights on assign financial account page

Add weights to financial type screen

Add weights on financial accounts page

Define weight for links on extensions page

Merge pull request #27317 from eileenmcnaughton/process

Remove long-broken function

Remove unused parameter, pass by reference in private function with one caller

Merge pull request #27318 from eileenmcnaughton/legacy

Finish CiviCRM 2.1 deprecation - old style verp

Merge pull request #27259 from eileenmcnaughton/lookup

Use new lookup trait to eliminate use of undefined properties `userDisplayName`

Merge pull request #27315 from colemanw/actionScheduleFix

Fix dev/core#4550 - Crash when saving repeating event

Merge pull request #27288 from eileenmcnaughton/total

dev/3410#3410 Show 0 rather than nothing when paid amount is zero

Merge pull request #27281 from eileenmcnaughton/fin_test

Fix financial transfer test to use form flow, allow all pending to transfer

Finish CiviCRM 2.1 deprecation - old style verp

Remove long-broken function

Fix dev/core#4550 - Crash when saving repeating event

AdminUi - Exclude non-scheduled-reminders from schedule reminders table

The ActionSchedule table is overloaded for use by repeating events & activities.
This hides the things that aren't supposed to be shown.

Merge pull request #27173 from larssandergreen/Show-Contribution-Page-receipt-text-in-offline-contribution-receipt

Show Contribution Page receipt text in offline contribution receipt

Merge pull request #26243 from eileenmcnaughton/weight

Let unset weights give an e-notice

Merge pull request #27304 from eileenmcnaughton/split

Make splitjob function static & private, Remove a couple of unused variables

Merge pull request #27312 from civicrm/5.65

5.65

Merge pull request #27298 from eileenmcnaughton/82_import

Remove some unused undefined properties from participant export

Merge pull request #27255 from seamuslee001/dep_warning_participant_trait

[REF] Convert from session status to deprecation warning

Make split job static & private

This is a pre-requisite to https://github.com/civicrm/civicrm-core/pull/27303

Show Contribution Page receipt text in contribution receipts

Remove a couple of unused variables

Merge pull request #27314 from larssandergreen/Fix-token-consistency-test

Fix token consistency test with new lines and <br>s

Merge pull request #27295 from eileenmcnaughton/trxn

Switch to using token for trxn_id, online contribution receipt

Fix token consistency test with new lines and <br>s

Merge pull request #27081 from colemanw/savedSearchReminders

dev/core#2644 Add Scheduled Communications extension

Merge pull request #27313 from lemniscus/session-set-comment

[NFC] Be more confident and informative in the comment for CRM_Core_Session::set()

Be more confident and informative in the comment for CRM_Core_Session::set()

Merge pull request #27310 from artfulrobot/artfulrobot-fix-job-log-updater

Fix upgrader SQL (5.60) that updates civicrm_job_log

ActionSchedule - Don't crash when mapping type is missing

Merge pull request #27203 from eileenmcnaughton/nl2br

Convert new lines to page breaks when rendering text tokens to html

ScheduledCommunications - Delete when deleting saved search

Merge pull request #26051 from eileenmcnaughton/nfc2

Unconditionally assign smarty-var, remove unused variables

Fix upgrader SQL (5.60) that updates civicrm_job_log

This upgrader has broken SQL, joining civicrm_job.id = civicrm_job_log.id causing
crashes on upgrades.

After: fixed the SQL

Merge pull request #27307 from eileenmcnaughton/processor

Trivial cleanup in EmailProcessorTest

Merge pull request #27309 from eileenmcnaughton/private

Make function private, after universe search in EmailProcessor

Merge pull request #27308 from eileenmcnaughton/no_ref

Make function private to reflect usage

Merge pull request #27306 from eileenmcnaughton/except

Declare CRM_Core_Exception in cleanDir, add type hints

Merge pull request #27201 from eileenmcnaughton/page_tokens

Add contribution page tokens to contribution tokens

SearchKit - Add column for scheduled communications to savedSearch listing

Merge pull request #27299 from eileenmcnaughton/pledge

Move function to only form that calls it

Make function private, after universe search

Make function private to reflect usage

A universe search shows it is only called from this class

Trivial cleanup in EmailProcessorTest

Declare CRM_Core_Exception in cleanDir

The only information our exception imparts is that it is
coming from CiviCRM Core  - but we should at least be consistent in that.

Also - I think passing something other than a string here would be bad - so
adding a type hint is good. We were already checking not target so the empty string check is redundant

APIv4 - Add UNIQUE flag to GROUP_CONCAT

Unlike DISTINCT which dedupes by the value of a field, UNIQUE will dedupe by the id of the record

APIv4 - Improve dataType handling within GROUP_CONCAT

Supports sqlEquations and function suffixes within the aggregated array

Add Scheduled Communications extension

Permits scheduled reminders to be created based on a SavedSearch

Merge pull request #27300 from eileenmcnaughton/mailing_archive

Unconditionally assign smarty variable

Merge pull request #27302 from colemanw/api4RequiredFix

APIv4 - Fix setting nullable/required/default_value field metadata

Merge pull request #27204 from eileenmcnaughton/br2nl

Pass html text through a formatter when translating it to tokens

ActionSchedule - Enable `getEntityTable()` to return dynamic values

This functionality is needed for SavedSearch-based reminders, which
do not have a fixed base-table.

Merge pull request #27268 from colemanw/recontact_interval

CiviCampaign - Refactor civicrm_survey.recontact_interval

Merge pull request #27105 from colemanw/inPlaceEdit

SearchKit - In-place edit without refreshing results

APIv4 - Fix setting nullable/required/default_value field metadata

Merge pull request #27301 from eileenmcnaughton/mailing_const

Remove unused CONST

Remove unused CONST

I can't see any evidence flexmailer or anything else still uses it...

Unconditionally assign smarty variable

Move function to only form that calls it

There was a time when we were moving stuff into AbstractEditPayment but
really it is better to have private functions when we have done
the work to determine that they are only used from one place - because then
we know we can edit  / fix them

I did a universe search & this is a straight move to a child class

Remove some unused undefined properties from participant export

Merge pull request #27297 from eileenmcnaughton/colon

Remove, extraneous: punctuation?

Unconditionally assign smarty-var, remove unused variables

Remove, extraneous: punctuation?

Merge pull request #27254 from colemanw/coreManaged

[REF] - Cleanup managed entities to all go through `hook_civicrm_managed` and deprecate unnamed entities

Switch to using token for trxn_id, online contribution receipt

Start to find the ones not set

Merge pull request #27291 from civicrm/5.65

Merge 5.65 to master

Merge pull request #27211 from colemanw/afformSubmissionLimit

Afform - Allow submissions to be disabled and limited

Merge pull request #27270 from colemanw/fixTabCountFrRlz

SearchKit - Fix assigning searchDisplay tab count

SearchKit - Fix assigning searchDisplay tab count

Incorectly fixed with https://github.com/civicrm/civicrm-core/pull/23759, this is the correct fix.
Enables assignment of the value (because you can't assign scalar values using one-way binding)
but avoids the $compile:nonassign error by making it optional.

See https://docs.angularjs.org/api/ng/service/#-scope-

Merge pull request #27290 from kurund/file-type-custom-data-fixes

Fixes for setting defaults for custom data field of type file

Merge pull request #27283 from eileenmcnaughton/weigh_job

Add weights to scheduled jobs

Afform - Enforce submission open/closed status

Afform - Add submit_currently_open calc field

Afform - Add submission_count and submission_date calc fields

Afform - Add submit_enabled & submit_limit fields

Fixes BasicGetAction handling of default_value - it should be used to set a default value not a default where clause.

Afform - Add field titles to metadata

Merge pull request #27272 from colemanw/afformPermission

Afform - Support multiple permissions in the GUI

Fixes for setting defaults for custom data field of type file

Merge pull request #27159 from larssandergreen/Fix-event-receipt-regression

Fix incorrect event receipt regression

Merge pull request #27150 from larssandergreen/Remove-profiles-from-offline-event-receipts

Remove unused profiles from offline event receipts

Merge pull request #27274 from eileenmcnaughton/gather

Pass specific ids rather than ids array

Merge pull request #27284 from eileenmcnaughton/weigh_multi

Add weights to links on multipleRecordFieldsListing

Merge pull request #27285 from eileenmcnaughton/test_nfc

[test] [cleanup] Move _REQUEST cleanup to tearDown

Merge pull request #27279 from eileenmcnaughton/form_test

Update further tests to additional form flow, lock in bug fix

dev/3410#3410 Show 0 rather than nothing when paid amount is zero

Per https://lab.civicrm.org/dev/core/-/issues/3410 the incorrect value is no longer
showing - but it is pretty confusing having the title but not the value when
the value is 0