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

Event registration activity target

Merge pull request #25652 from eileenmcnaughton/body

Php8.x compatibility - do not try to count NULL

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

Merge pull request #25642 from eileenmcnaughton/uffield

Fix add version for civicrm_custom_field.fk_entity

Fix add version for civicrm_custom_field.fk_entity

Per https://github.com/civicrm/civicrm-core/pull/25638#issuecomment-1439256431

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

Merge pull request #25600 from eileenmcnaughton/import_sql

Fix handling of invalid sql query during import

Merge pull request #25640 from civicrm/5.59

5.59 to master

Merge pull request #25638 from eileenmcnaughton/upgrade_fix

Fix master-only regression - upgrade script is in wrong version

Merge pull request #25633 from eileenmcnaughton/559

PEAR Exception handling - Mitigation of scenario where an import table has been deleted and the metadata is out of date

Merge pull request #25573 from herbdool/core-4127-3

dev/core#4127 move is_drupal to use userSystem logger

parsePrefix() - Don't confuse financialacls dynamic-permissions with permission-prefixes

Fix master-only regression

I merged https://github.com/civicrm/civicrm-core/pull/25471 yesterday,
well after 5.60 branch forked, into master. However, it
includes upgrade script targetting 5.59 - which means that
for anyone already on 5.59 it doesn't run. This moves
it to the correct branch (5.60 aka master)

Fix fatal on datasource error

Merge pull request #25637 from civicrm/5.59

5.59

Merge pull request #25602 from eileenmcnaughton/import_set_path

Add 'SavePoint' to import to statusBounce back to

Merge pull request #25549 from eileenmcnaughton/no_post

Fix custom field bug on UFMatch sync

Merge pull request #25457 from eileenmcnaughton/notice_cont_view

Move form specific code to the form

Merge pull request #25522 from eileenmcnaughton/dedupe

Make dedupe exceptions searchable in SearchKit

Merge pull request #25612 from eileenmcnaughton/ical_notice

E-notice fix Ical display

dev/core#4127 move is_drupal to use userSystem logger

Merge pull request #25392 from eileenmcnaughton/prev_next_divide

dev/core#4112 Privatise `prevNextCache` functions

Merge pull request #25446 from lemniscus/oauth-reduce-dns-queries

Reduce unneeded DNS queries during OAuth flow

Merge pull request #25631 from eileenmcnaughton/poly

composer require --dev yoast/phpunit-polyfills

Merge pull request #25511 from eileenmcnaughton/dep_token6

Extra deprecation for clarity on token function `convertPseudoConstantsUsingMetadata`

Merge pull request #25598 from colemanw/PledgeAutocomplete

Add APIv4 Pledge Autocomplete

Merge pull request #25630 from colemanw/phpQueryEncodeBug

Angular Coder: Fix unescaping of quotes breaking attributes

Merge pull request #25632 from colemanw/formatCustomValue

EntityRef - Format custom field display value on QuickForms

EntityRef - Format custom field display value on QuickForms

Mitigation of scenario where an import table has been deleted and the metadata is out of date

Angular Coder: Fix unescaping of quotes breaking attributes

Fixes the bug described in https://github.com/civicrm/civicrm-core/pull/25471#issuecomment-1423233785

Merge pull request #25471 from colemanw/entityReferenceField

Add new EntityReference custom field type

composer require --dev yoast/phpunit-polyfills

Merge pull request #25441 from MegaphoneJon/status-is-a-string

getStatus() should be returning a string

getStatus() should be returning a string

CustomFields - Add EntityReference field type based on APIv4 autocomplete

Merge pull request #25444 from michaelmcandrew/dev/core#4100

Add setting to disable Smarty in scheduled reminders

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

Merge pull request #25616 from jitendrapurohit/parse_address_fix

Regression: Fix DB syntax error on Parse address scheduled  job

Merge pull request #25620 from mattwire/afformrelationship

Afform: Fix broken syntax for saving reciprocal relationships

Merge pull request #25596 from ixiam/devSK-ShowMoreButtonsAlways

Always show "add more buttons" dropdown in searchkit view

Merge pull request #25622 from braders/unnecessary-entity-property

[REF][PHP8.2] Remove unnecessary dynamic property

Merge pull request #25625 from braders/deprecated-self-callables

[REF][PHP8.2] Fix use of self in callables deprecation

Merge pull request #25626 from braders/groupcontacttest

[REF][PHP8.2] Tidy up properties on api_v3_GroupContactTest

Merge pull request #25623 from braders/event-permissions-test

[REF][PHP8.2] Tidy up properties in CRM_Event_BAO_EventPermissionsTest

Merge pull request #25624 from braders/remove_dedupeRuleGroupID

(REF) Remove test reference to property which no longer exists

Afform: Fix syntax for saving reciprocal relationships

[REF][PHP8.2] Tidy up properties on api_v3_GroupContactTest

[REF][PHP8.2] Fix use of self in callables deprecation

(REF) Remove test reference to property which no longer exists

[REF][PHP8.2] Tidy up properties in CRM_Event_BAO_EventPermissionsTest

[REF][PHP8.2] Remove unnecessary dynamic property

Merge pull request #25619 from eileenmcnaughton/php82_test

Php8.2 test fix, remove one instance of undeclared property

Php8.2 test fix, remove one instance on undeclared parameter

Merge pull request #25617 from eileenmcnaughton/matt_dep

Stop loading unused `relatedObjects`

Merge pull request #25610 from eileenmcnaughton/doc_block

[NFC] doc block fix

Unset ids for clarity

Stop loaded unused relatedObjects

[NFC] doc block fix

Merge pull request #25586 from herbdool/core-4127-6

dev/core#4127 move mailing workflow check to userSystem

Merge pull request #25615 from eileenmcnaughton/matt_dep

Remove function deprecated a year ago

Fix DB syntax error on Parse address sched job

Remove function deprecated a year ago

E-notice fix Ical display

Merge pull request #25608 from eileenmcnaughton/test_fix

Fix test to test the thing it was written to test

Merge pull request #25607 from eileenmcnaughton/prop

[NFC] php8.2 support in test class `ActivitySearchTest`

Merge pull request #25599 from totten/5.59-patchset-wp

distmaker - Fix export of WordPress patches

Fix test to test the thing it was written to test

Per https://github.com/civicrm/civicrm-core/pull/23929 the point of the test was
to check that any override coming from parameters or the recurring contribution
record would be respected.

Also see
https://issues.civicrm.org/jira/browse/CRM-16397

However, the test did not explicity change the recurring contribution record amount
field, having been written after a hook was added that was changing the recurring amount
based on the received payment. So it was not testing it's goal - that where they differed
then, for single line item contributions, the recurring record took precedence.

This fixes to specifically alter them to be different, thus testing the override works
and does not rely on the hook (which is likely to change it's behaviour)

Merge pull request #25604 from eileenmcnaughton/parse_private

Make private function private

php8.2 support in test class ActivitySearchTest

Merge pull request #25550 from totten/master-abbr2

Disambiguate `Address.state_province_id:abbr`

Make private function private

Merge pull request #25570 from kurund/profile-api-fixes

update entity mapping logic for profile fields to include contact subtypes

Merge pull request #25445 from lemniscus/oauth-allow-client-to-set-redirect-uri

Allow OAuth redirect URI to be overridden in client or provider config

Merge pull request #25322 from braders/unused-params-contribution-soft-test

(REF) Tidy up unused params in api_v3_ContributionSoftTest

Disambiguate `Address.state_province_id:abbr` (MySQL; simpler version)

Consider `ContactJoinTest::testCreateWithPrimaryAndBilling` which writes the value:

  'address_billing.state_province_id:abbr' => 'AK',

The symbol 'AK' can map to three places: Akwa Ibom (Nigeria), Atakora
(Benin), and Alaska (USA).  This is an ambiguous choice.  It should be
resolved in a consistent way.

One flavor of ambiguity comes from MySQL.  When loading abbreviations, Civi
queries with `ORDER BY abbreviation`.  This is a *typically* stable, but it
has no *guaranteed* resolution.  Adding a secondary sort key makes the
outcome clear/unambiguous.

Merge pull request #25595 from totten/master-api4-cast

APIv4 - Limited support for casting

Merge pull request #25601 from eileenmcnaughton/import_constants

Remove unused constants left over from import cleanup

Merge pull request #25589 from lemniscus/api4-explorer-cv-syntax-short-boolean

api4 explorer: make boolean params work in cv (short syntax)

castTypeSoftly() - Support magic floats and magic ints

Merge pull request #25552 from totten/master-abbr-php

Disambiguate `Address.state_province_id:abbr` (PHP asort)

Use provided params not POST

Add 'SavePoint' to import to statusBounce back to

Remove unused constants left over from import cleanup