Merge pull request #27074 from civicrm/5.65

5.65

Merge pull request #27066 from eileenmcnaughton/565

dev/core#4494 Fix e-notice by removing unused variables

Merge pull request #27013 from eileenmcnaughton/auth

Fix permission check on Contribution form, clarify underlying functions

Merge pull request #27068 from eileenmcnaughton/fee_block

Move form-specific check to form

Merge pull request #27067 from eileenmcnaughton/form_test_fix

Test fix, towards full form flow

Move form-specific check to form

Test fix, towards full form flow

Rather than set _isQuickConfig it should be loaded from the db by the form
(which this switches to)

Merge pull request #27060 from colemanw/dontGetFields

Api4 - Remove unused key from getFields

dev/core#4494 Fix e-notice by removing unused variables

Merge pull request #27063 from eileenmcnaughton/context

Do the todo (move the function to the trait as suggested

Merge pull request #27042 from colemanw/copyPastaFix

[REF] SearchKit - Cleanup upgrade code

Merge pull request #27064 from eileenmcnaughton/remove_notice

Remove line that gives error & does nothing

Merge pull request #27008 from colemanw/booleanSearch

Simplify queries of NOT NULL fields

Remove line that gives error & does nothing

I went back to 5.55 & did receipt testing and
1) this notice is there already back then
2) it results in a resolved value of 'Array - Display name
3) it does not appear anywhere in the final receipt

Merge pull request #27061 from demeritcowboy/searchevent-title

Manage event - fix too many brackets

Do the todo (move the function to the trait as suggested

This makes the currency of the contribution available to the template
for other templates - notably offline event receipts

Merge pull request #27062 from civicrm/5.65

5.65

Add test + external support for getAuthenticatedContactID()

Fix permission check on Contribution form, clarify underlying functions

This is a reviewer response to https://github.com/civicrm/civicrm-core/pull/22951

In that PR it seeks to set UserID to something derived from a function that is doing
a lot of work. I dug into that function which someone (ahem) wrote many years ago
and teased out some of the underlying chunks & cleaned up the variable names for clarity
along with a doc block cleanup.

I decided that rather than set _userID in contributionBase we should remove it
(it is only referred to in 4 places) and call the more specific function at each usage.

Otherwise we change the meaning of userID that is used somewhat differently later.

Also I rather prefer functions that will perform the same whenever called over relying
on a variable having been set correctly the first time

Merge pull request #27005 from eileenmcnaughton/end_date

Fix event templates to use end_date token

Api4 - Remove unused key from getFields

too many brackets

Merge pull request #27057 from larssandergreen/Fix-event-fee-help

Fix event fee help

Fix event fee help

Merge pull request #27056 from larssandergreen/Use-statusBounce-instead-of-redirect-when-can't-delete-custom-group

dev/core#4094 Use statusBounce instead of redirect when cannot delete custom group

Merge pull request #27052 from totten/master-data-nonphp

(REF) civicrm_data.tpl - Stop using Smarty's {php}

Merge pull request #27055 from totten/nfc-eventtest

(NFC) Test class - Fix inconsistent namespace

Merge pull request #27032 from eileenmcnaughton/dep

Unit tests - replace deprecated function with newer one

Use statusBounce instead of redirect when cannot delete custom group

(NFC) Test class - Fix inconsistent namespace

Most tests in this folder have a namespace...

Merge pull request #27053 from mlutfy/fixContributionBaseImplode

Fix PHP 8.0 implode on ContributionBase

Fix PHP 8.0 implode on ContributionBase

Merge pull request #27041 from kurund/afform-submission-schema-changes

(dev/core#4232) Afform submission schema changes

Merge pull request #27051 from eileenmcnaughton/event_price

Clarify doc-block for form variable

Merge pull request #27047 from eileenmcnaughton/is_required

Remove do-nothing code.

Merge pull request #27028 from larssandergreen/Show-which-extension-has-a-parse-error-in-info.xml

Show which extension has a parse error in info.xml

Merge pull request #27038 from bugfolder/80_global_language

Issue dev/backdrop#80: global $language should be an object.

add status column to afform submissions table

(REF) civicrm_data.tpl - Stop using Smarty's {php}

This should make it easier to try out Smarty 4

Clarify doc-block of for form variable

Merge pull request #27048 from eileenmcnaughton/notices

E-notice fix, don't skip loading NULL fields

E-notice fix, don't skip loading NULL fields

This addresses e-notices (e.g) when help_pre is NULL & assigned to the template

Remove do-nothing code.

This code conditionally adds the field is_required to the array that already
contains the field is_required

Show which extension has a parse error in info.xml

Merge pull request #27045 from civicrm/5.65

5.65

Merge pull request #27043 from seamuslee001/dev_core_4492

dev/core#4492 Ensure that when determining if tokens should be show c…

dev/core#4492 Ensure that when determining if tokens should be show check if the Entity is in list of Avaliable Entities

Merge pull request #27033 from colemanw/fixSkEditable

Fix dev/core#4483 SearchKit crash when in-place-editing a field in a joined entity

SearchKit - Cleanup upgrade code

Merge pull request #27039 from sebalis/contributor-details-sebalis

added details for sebalis in contributor-key.xml

added details for sebalis in contributor.xml

Merge pull request #27037 from larssandergreen/Fix-paymentBlock-url-concat

dev/core#4487 Alternative option: Fix paymentBlock url concat

Merge pull request #27024 from yashodha/dev-4485

(dev/core#4485) Replace Event Total by Total Amount

Issue #80: global $language should be an object.

Fix event templates to use end_date token

Unit tests - replace deprecated function with newer one

Also minor clean ups

Fix paymentBlock url concat

Merge pull request #27036 from eileenmcnaughton/phpunit

Use phpunit when running on php 7.3+

Use phpunit when running on php 7.3+

Fix dev/core#4483

Merge pull request #27027 from allinappliadmin/patch-1-AIA

E:: to make it look after the extension's translation

E:: to make it look after the extension's translation

Merge pull request #27026 from aydun/path_fixes

Path fixes

Path fixes

(dev/core#4485) Replace Event Total by Total Amount

Merge pull request #27023 from eileenmcnaughton/test_twice

Re-enable any queues that were disabled for background processing, on end

Add queue runner test

Re-enable any queues that were disabled for background processing, on end

This addresses what is likely an artificial scenario but

essentially if these steps are followed twice within the same function
- create a queue
- run the queue via runAll

The second time will fail because the queue was disabled & no re-enabled, despite
being perisistent

Merge pull request #26292 from larssandergreen/Remove-more-unneeded-descriptions---pledges,-etc

Remove unneeded descriptions and clean up on search forms

Merge pull request #27022 from eileenmcnaughton/unsubscribe

Wording changes in the Unsubscribe form

code clean up

ts variable method updated and OptOutURL removed

Merge pull request #27015 from eileenmcnaughton/extend

Separate getStatusMessage function

Merge pull request #27021 from civicrm/5.65

5.65 to master

Merge pull request #26969 from demeritcowboy/view-only-merge-hook

dev/core#4481 - View-only custom fields should be merged during merges

Merge pull request #27019 from eileenmcnaughton/user_name

Remove excessive data handling & some unused variables

Merge pull request #27020 from civicrm/5.65

5.65

Merge pull request #27004 from eileenmcnaughton/offline_tokens

Update a couple more tpl variables to tokens

Merge pull request #27018 from larssandergreen/Fix-MailSettings-add-path

dev/core#4480 Fix MailSettings add path

Remove excessive data handling & some unused variables

Fix MailSettings add path

Separate getStatusMessage function

The  CRM_Event_Form_Task_Register form was once part of CRM_Event_Form_Participant

It now extends it & the goal is to disentangle it from the parent.

In general _single is only set for the parent & describes actions for that
whereas the Register form is a task that can act on many

Merge pull request #27014 from civicrm/5.65

5.65 to master

Merge pull request #27010 from demeritcowboy/fintype

dev/core#4473 - Add/edit financial type screen broken

Update a couple more tpl variables to tokens

event.is_public is already converted in the event online template
- event.email_confirm_text is already tested as a token available
in the offline template. The token layer should take
care of the htmlise question

fix broken add/edit form

Merge pull request #27011 from larssandergreen/Remove-inbound-email-parse-function

Remove unused parse function for Incoming Mail

Remove unused parse function

Merge pull request #27009 from civicrm/5.65

5.65

Simplify queries of NOT NULL fields

Merge pull request #27007 from colemanw/fixLink

Fix dev/core#4461 missing links causing SK to return no results

view-only custom fields should be merged

Revert "hide view only custom fields on merge screen"

revert 3b9d4007540d19d58c1fed209e88a01fd5084026.

add real-world example test

Fix dev/core#4461 missing links causing SK to return no results

Merge pull request #27002 from eileenmcnaughton/enotice

Enotice fix

Merge pull request #27001 from eileenmcnaughton/participant_test

E-notice fix

Another e-notice fix

Enotice fix

Merge pull request #26985 from colemanw/recPHPver

Bump recommended PHP version to 8.1, min recommendation to 8.0

Merge pull request #24657 from larssandergreen/always-show-change-selections-for-participant-view

dev/core/#3361 Add Change Selections button when viewing price set registrations with no contribution associated