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

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

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

Merge pull request #27286 from civicrm/5.65

5.65

Merge pull request #27282 from eileenmcnaughton/wei

Minor cleanup in test

Merge pull request #27280 from eileenmcnaughton/565

Fix bug exposed by invalid data warning

Fix event receipt regression

Remove unused profiles from offline event receipts

Move _REQUEST cleanup to tearDown

Add weights to links on multipleRecordFieldsListing

Add weights to scheduled jobs

Add weights to event search

Pass specific ids rather than ids array

It is unclear when we pass ids what ids we care about (& in general how many ids we are using from the
various functions). In fact 2 specific IDs are used (one from the _relatedObjects array) so pass
these in and stop passing in ids. Also stop setting an id that we never use

Convert html to text, keeping line breaks, when using html-based tokens

Fix bug exposed by invalid data warning

The invalid data warning exposed a bug where the email
is being sent before the line item has been fixed - so
line item data in the email would be incorrect. This is
not a regression but the potential for someone to
be aware of it is.

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

Merge pull request #27277 from eileenmcnaughton/form_test

Update one of our complex confirm tests to use full form flow, fix discovered failure to send to additional participants

Merge pull request #27140 from colemanw/groupSelectWhereClause

API - Switched to cached function for looking up pemissioned groups

Merge pull request #27276 from colemanw/trackingFieldsMsg

Add pre-upgrade message about the removed TrackingFields.tpl

Afform - Support multiple permissions in the GUI

Merge pull request #26987 from mattwire/lineitem

Fix Financial Items incorrectly recorded when using Payment API

Merge pull request #27273 from eileenmcnaughton/no_component

Remove some unnecessary variables & pass by ref

Update one of our complex confirm tests to use full form flow, fix discovered additional payments bug

Add pre-upgrade message about the removed TrackingFields.tpl

Merge pull request #27197 from colemanw/getSqlFunctions

SearchKit - Expose sql functions provided by extensions

Merge pull request #27233 from colemanw/trackingFields

Delete probably-unused TrackingFields.tpl

Merge pull request #27275 from colemanw/isDefined

EntityLookupTrait - Add isDefined and getDefinition methods

EntityLookupTrait - Add isDefined and getDefinition methods

Remove some unnecessary variables

Merge pull request #27202 from colemanw/searchKitCurrency

SearchKit - Improve handling of money currency

Merge pull request #27269 from colemanw/undefined

ajax.js - Fix undeclared variable

Merge pull request #27199 from colemanw/fixSkTotals

SearchKit - Fix broken totals row and query output

Merge pull request #27267 from eileenmcnaughton/form_test

Add form helper for tests

SearchKit - Fix broken totals row and query output

This regressed in ef1cbd

ajax.js - Fix undeclared variable

CiviCampaign - Move serialized contents of civicrm_survey.recontact_interval into civicrm_option_value.filter

Refactors out a thoroughly strange data structure.

Each option value associated with a survey has an associated recontact_interval, which is an integer
storing the number of days before the respondent should be recontacted.

Before: A serialized array was being stored in the survey with keys matching  the option LABEL!

After: There was an unused integer column, civicrm_option_value.filter which works fine for storing
the number directly for each option_value with no messy mapping needed.

Add form helper for tests

Merge pull request #27262 from colemanw/phpstormApi4

[NFC] PhpStorm - Add more APIv4 function argument hints