Merge pull request #25690 from eileenmcnaughton/dep_test

Deprecations for delete on ACL entities

Deprecations for delete on ACL entities

Merge pull request #25688 from eileenmcnaughton/dep_test

Remove deprecated functions from tests

Remove deprecated functions from tests

Merge pull request #25686 from eileenmcnaughton/dep_2020_address

Remove 2020-deprecated handling of `legacyAddressCreate`

Merge pull request #25685 from eileenmcnaughton/dep_2020

Remove function noisily deprecated when the world was young (2020)

Merge pull request #25684 from eileenmcnaughton/address_delete

Remove deprecated function call

Merge pull request #25646 from colemanw/fixSearchDisplayFilter

Afform - Filter autocomplete fields belonging to a search display

Remove 2020-deprecated handline of legacyAddressCreate

Remove function noisily deprecated when the world was young (2020)

Merge pull request #25669 from eileenmcnaughton/smarty-syntax

dev/core#4146 Remove (old) Smarty-forward incompatible syntax from Address.tpl

Merge pull request #25683 from eileenmcnaughton/logging_dep

Add gentle deprecation to `debug_log_message`

Merge pull request #25682 from eileenmcnaughton/log_smarty

Log Smarty debug to it's own channel

Merge pull request #25680 from eileenmcnaughton/smarty-bank

Smarty notice fix on bank_account_number

Remove deprecated function call

Merge pull request #25678 from eileenmcnaughton/logging-ref

Remove unnecessary pass-by-reference

Add gentle deprecation to debug_log_message

Merge pull request #25679 from eileenmcnaughton/deprecated_mem_type

Remove code deprecated in 2018

Log Smarty debug to it's own channel

I just wrote the docs on the right way to log using this example
https://docs.civicrm.org/dev/en/latest/framework/logging/

- so we should make it the same. I tested locally
& this results in the smarty debugging going to a separate file (good).

I personally think we should send all the IPN files off to their
own channel but am loath to be pro-active on changing that

Smarty notice fix on bank_account_number

Also remove CRM_Utils_Array call

Remove code deprecated in 2018

Make function private

Remove unnecessary pass-by-reference

Merge pull request #25675 from eileenmcnaughton/notice_location

Remove empty on location - it should always be set now

Remove empty on location - it should always be set now

See https://github.com/civicrm/civicrm-core/pull/25668

Merge pull request #25618 from mattwire/caserelationship

Afform: Allow selecting case for relationship

Merge pull request #25668 from eileenmcnaughton/notice_location

Fix e-notice pattern around location not defined in formButtons.tpl

Merge pull request #25309 from MegaphoneJon/flexmailer-return-path

closes core#4070 - respect default return path

Merge pull request #25585 from herbdool/core-4127-4

dev/core#4127 move cms user parameters to userSystem

Merge pull request #25671 from colemanw/viewSql

SearchKit - Clarify how to view SQL output

SearchKit - Clarify how to view SQL output

Merge pull request #25670 from eileenmcnaughton/cust_it

Remove unused variables

Remove unused variables

Merge pull request #25466 from eileenmcnaughton/sep

Remove unused property - not used since import code refactor `_lineCount`

Merge pull request #25237 from eileenmcnaughton/mem_receipt

Remove conditional assignment around `is_deductible`

Merge pull request #25621 from eileenmcnaughton/event_test_cleanup

Event test cleanup, fix test to use submitted form values, rather than require a lot of contorting

Merge pull request #25270 from eileenmcnaughton/separate

Extend testing for ContributionConfirm & consolidate `isSeparateMembershipPayment`

Merge pull request #25667 from eileenmcnaughton/mixin

Fix mixin to use `addTemplateDir`

Remove (old) Smarty-forward incompatible syntax from Address.tpl

Fix e-notice pattern around location not defined in formButtons.tpl

Fix mixin

Merge pull request #25662 from eileenmcnaughton/token_dep

Add noisy deprecation to deprecated function, after universe search

dev/core#4127 move cms user parameters to userSystem

Merge pull request #25665 from totten/master-nfc-url

(NFC) crmURL - Add examples and whitespace. Crosslink docs.

Merge pull request #25664 from eileenmcnaughton/tpl_fix

Remove forward incompatible syntax from template

crmURL - Add examples and whitespace. Crosslink docs.

Merge pull request #25661 from eileenmcnaughton/smarty-url

Add `crmUrl function` to smarty in the standard way

Merge pull request #25663 from eileenmcnaughton/civix

run `civix update` on `recaptcha`, `legacyCustomSearches`, `financialacls`

Remove forward incompatible syntax from template

Civix upgrade on financialAcls

Civix update on legacyCustomSearches

Civix update on Recaptcha

Merge pull request #25659 from totten/master-ext-up

ext/* - General update to civix v23.02.0

Add crmUrl function in the standard way

Even during Smarty2 this method of registering functions was being phased out
for putting the file in the folder

Add noisy deprecation to deprecated function, after universe search

Merge pull request #25653 from totten/master-htxt

Smarty - {htxt} blocks should not be evaluated unless needed

Merge pull request #25655 from MegaphoneJon/sub-history-date

HTML definition for subscription history date

Merge pull request #25660 from civicrm/5.59

5.59

Merge pull request #25658 from demeritcowboy/contact-params

Use correct var for joomla synchronizeUFMatch

Smarty - {htxt} blocks should not be evaluated unless needed

ext/civigrant - Update to civix v23.02.0

ext/search_kit - Update to civix v23.02.0

ext/sequentialcreditnotes - Update to civix v23.02.0

ext/oauth_client - Update to civix v23.02.0

ext/message_admin - Update to civix v23.02.0

ext/ckeditor4 - Update to civix v23.02.0

ext/civiimport - Update to civix v23.02.0

ext/civicrm_admin_ui - Update to civix v23.02.0

ext/flexmailer - Update to civix v23.02.0

ext/greenwich - Update to civix v23.02.0

ext/afform/admin - Update to civix v23.02.0

ext/afform/core - Update to civix v23.02.0

ext/authx - Update to civix v23.02.0

use correct var

Merge pull request #25656 from eileenmcnaughton/notice_lang

Notice fix on preferred_language, when null

Merge pull request #25654 from eileenmcnaughton/hacky

Remove legacy handling of locks for discontinued mysql/mariaDB versions

Merge pull request #25650 from reflexive-communications/event-registration-activity-target

Event registration activity target

Notice fix on preferred_language, when null

Merge pull request #25652 from eileenmcnaughton/body

Php8.x compatibility - do not try to count NULL

HTML definition for subscription history date

Remove legacy handling of locks for discontinued mysql/mariaDB versions

Merge pull request #25651 from totten/mbtruncate

Smarty - Fix warnings about 'mb_truncate' modifier

Php8.x compatibility - do not try to count NULL

Merge pull request #25649 from demeritcowboy/case-search

dev/core#4126 - Fix force-url parameters on case dashboard

mb_truncate - Save 0.00000001s

If it's null, then we don't really have to call `$strlen()` or evaluate the `>` operation.

Smarty - Fix warnings about 'mb_truncate' modifier

Steps to reproduce
------------------

* Use PHP 8.1 and D7 with a default logging policy (eg "All Messages" or "Errors and Warnings")
* Use "Search > Find Contacts"
* On the listing page, observe a bunch of warnings

Before
------

~25 warnings, including ~10 related to modifier.mb_truncate.php

After
-----

~10 warnings, none involving `mb_truncate`

CRM_Activity_BAO_Activity::addActivity() remove duplicate target_contact_id

pass participant contact_id as target_contact_id when creating event registration activity

fix force-url parameters

Merge pull request #23782 from totten/master-finperm

Allow (some) permissions with colons

Merge pull request #25248 from eileenmcnaughton/add_template

Update Smarty addTemplateDir function signature to future smarty

Merge pull request #25629 from yashodha/dev-4106

(dev/core#4106) Add filter on pledge payment in contribution report t…

Merge pull request #25648 from eileenmcnaughton/cheaper_failure

Reduce processing load in test `assertAPIFailure`

Merge pull request #25641 from eileenmcnaughton/token_name

Deprecate unused, exception class with non-standard name-spacing

Merge pull request #24746 from mattwire/joblogsearch

Make job ID accessible to searchkit for joblog

Reduce processing load in test assertAPIFailure

In trying to figure out why adding 'too much' (an exception) to the
return caused a memory out I realised that the print_r here is
realised regardless of whether it is needed. This fixes to only
resolve if it is going to be displayed. Arguably this would have
been a case where an inline function would have made sense - but
at the cost of readability - ie it's more helpful
to add code comments to explain the if than to make the codd hard to
follow. The only downside is the hypothetical
possibility of is_error being something other than 1,'1',TRUE - I
think if we thought that was a remote possibilty we would add
a unit test to add it - not cover it in an assertion

Merge pull request #25587 from herbdool/core-4127-7b

dev/core#4127 move Drupal stuff in getCiviSourceStorage to Drupal use…

Make job ID accessible to searchkit for joblog

Afform - Filter autocomplete fields belonging to a search display

Fixes dev/core#4138