Merge pull request #20132 from colemanw/renameSearchKit

Replace string org.civicrm.search -> org.civicrm.search_kit

Replace string org.civicrm.search -> org.civicrm.search_kit

Followup to renaming the extension, this fixes strings with the old name.

Merge pull request #20130 from colemanw/searchKitLinks

APIv4 - Deprecate nonstandard syntax for implicit joins

APIv4 - Deprecate nonstandard syntax for implicit joins

The APIv4 prototype used a string-manipulation hack to make implicit joins appear more friendly,
e.g. it would transform `contact_id.*` to `contact.*`.
However this made the generation of such joins less predictable,
as not all FK fields end in `_id` (notably, custom ContactRef fields do not).

This preserves the nonstandard syntax for backward-compat while favoring the new syntx
in the API Explorer.

Merge pull request #20124 from colemanw/renameSearchKit

dev/core#2556 Rename extension org.civicrm.search -> org.civicrm.search_kit

Merge pull request #20122 from eileenmcnaughton/mgd

Enotice fix

Merge pull request #20112 from francescbassas/patch-19

Add contact source column in some Reports

Refresh extension cache after renaming search_kit extension

dev/core#2556 Rename extension org.civicrm.search -> org.civicrm.search_kit

This avoids a namespace conflict in drupal 7, and is generally more descriptive
See https://lab.civicrm.org/dev/core/-/issues/2556

Add contact source column for basic contact fields

Enotice fix

Merge pull request #20119 from civicrm/5.37

5.37

Merge pull request #20117 from colemanw/searchKitFixContactRef

SearchKit - Fix display of contact reference fields (single-value)

Merge pull request #20085 from mlutfy/fixDrupal8MenuTranslation

Fix Drupal8 CiviCRM menu translation on multilingual (and ajax calls)

Add release notes for 5.36.1, 5.35.2

SearchKit - Fix display of contact reference fields (single-value)

This fixes the display of contact reference fields, giving feature parity
with other FK fields. Both ID and Display Name are shown as available columns.

This does not address the more difficult question of how to join
multi-valued contact reference fields with contact display names.

CRM.url - Encode url path if slashes in placeholder were also encoded

Merge pull request #20116 from totten/master-ext-uninst

Extensions - Setup <classloader> during "enable" and "uninstall"

Extensions - In case of dirty removal, skip classloader

Merge pull request #20114 from demeritcowboy/simplify-actionlink2

dev/core#2509 - Remove duplicate setting of activity subject field and replace a few CRM_Utils_Array::value

Extensions - Setup <classloader> during "enable" and "uninstall"

This is an analog/follow-up to https://github.com/civicrm/civicrm-core/pull/20091

Why is this necessary? Recall that a typical admin will go through this lifecycle:

1. Enable extension $x
2. Disable extension $x
3. Either:
   * (a) Re-enable extension $x
   * (b) Uninstall extension $x

Step `#2` disables the classloader for purposes of regular page-loading. However, when you
get to step `#3a` or `#3b`, then you need the classloader again.

simplification

Merge pull request #20090 from totten/master-ext-upgrader

dev/core#2547 - Add base-upgrader to core

Merge pull request #20105 from eileenmcnaughton/v4

Save entities created by api v4 to managed

Merge pull request #20106 from demeritcowboy/simplify-actionlink

dev/core#2509 - Remove unnecessary variable to make it easier to extract function that creates action links

Merge pull request #20107 from omarabuhussein/update-queue-item-data-field

Increase civicrm_queue_item.data field size

Merge pull request #20110 from eileenmcnaughton/err_more

Include more data in error message

ext/oauth-client - Switch to CRM_Extension_Upgrader_Base

ext/search - Switch to CRM_Extension_Upgrader_Base

Extensions - Add test-case for invoking upgrader classes

This is essentially a copy of CRM_Extension_Manager_ModuleTest, adapted to
use the upgrader class instead of lifecycle-hooks.

Extensions - Define the upgrader base class

This class is based on refactoring the civix template.

Major sections have been split out into traits.  This should make it more
readable (and potentially make it easier to remix).

This base-class should generally provide an equivalent DX for subclass authors:

* Variables have the same names.
* Most method signatures are identical (e.g.  `executeFoo()`)
* Some methods have been redeclared in equivalent form (`addTask()` - using splat instead `func_get_args()`).
* Two internal functions have slightly diff signatures (`enqueuePendingRevisions()`, `_queueAdapter()``).

Extensions - If there is an `<upgrader>`, then send it lifecycle notifications (w/top-sort)

info.xml - Parse the `<upgrader>` tag

Extensions - Define the upgrader interface

Include more data in error message

Merge pull request #19989 from olivierh65/patch-1

dev/core#2490 Drupal8: Fix CMS intergration table mapping

Merge pull request #20108 from colemanw/contact-actions-ribbon

Add 'contact-actions-ribbon' region

Merge pull request #20101 from JKingsnorth/patch-18

dev/core#2549 Fix undefined index when merging websites

Add 'contact-actions-ribbon' region

Add upgrade to update civicrm_queue_item.data field to LONGTEXT

Increase civicrm_queue_item.data field size

civicrm_queue_item.data field holds information about the task in the
job queue that is going to be excuted. But if we are using queues to
import large data (as we might do in an extension such as
nz.co.fuzion.csvimport) then the current field type might not be able to
handle the data size. So here I am changing it from TEXT to LONGTEXT to
accomdate for such cases.

Merge pull request #20053 from eileenmcnaughton/money

dev/core#2505 Fix formatLocaleNumeric

Save entities created by api v4 to managed

This fixes an issue where entities that only have v4 apis were not saving to
civicrm_managed because the id was being incorrectly fetched

Change-Id: I9b20de6dcc665d22824743b033acbdc889663045

simplification

APIv4 - Fix gatekeeper checks across joins

APIv4 - Add test to ensure gatekeeper permissions are checked across joins

Merge pull request #20103 from civicrm/5.37

5.37

Merge pull request #20104 from seamuslee001/regen_grant_dao

[NFC] Regenerate Grant DAO

[NFC] Regenerate Grant DAO

Merge pull request #20002 from jaapjansma/dev_53_2

dev/report#53: search on relationship and case (2)

Merge pull request #20099 from eileenmcnaughton/ref

Remove unused functions

Merge pull request #20058 from larssandergreen/re-use-mailing

dev/mail#90 - Allow re-use mailing on draft mailings and adhoc mailings

Merge pull request #20094 from seamuslee001/fix_install_search_display_table_56

[REF] Fix installation of the search display table on MySQL 5.36 in t…

Merge pull request #20100 from eileenmcnaughton/afform

Fix the MOST out-of-date parts of the afform readme

Fixed failing test

dev/core#2549 Fix undefined index when merging websites

Fix the MOST out-of-date parts of the afform readme

I was looking for afform docs and this documentation still appears to be the only
documentation there is :-(

This PR addresses some of the most out-of-date parts but I think we want to
agree how to document it & start addressing that

Merge pull request #20091 from totten/master-ext-instclass

Extensions - Setup `<classloader>` during installation

Remove unused functions

These protected functions were extracted as part of the process of testing and
transitioning to using the token compatibility subscriber - however, the
cleanup of actually removing them once we switched got missed.

As they are protected their unusedness is easy to confirm

Merge pull request #20036 from eileenmcnaughton/dedupe

Alter dedupe code to call api rather than bao->save()

[REF] Fix installation of the search display table on MySQL 5.36 in tests

Merge pull request #20089 from demeritcowboy/approx

[NFC/Unit Test] - Help avoid problems when using assertApproxEquals with integers instead of fractions

Merge pull request #20088 from seamuslee001/nfc_test_fix

(NFC) Fix possible flacky tests by splitting the parts of the checksu…

Extensions - Setup `<classloader>` during installation

Change Re-use to Copy

avoid problems

Update CRM/Mailing/Controller/Send.php

(NFC) Fix possible flacky tests by splitting the parts of the checksum up so that the timestamp part can be tested using ApproxEquals assertion

Update test as per Dave D

Merge pull request #20086 from civicrm/5.37

5.37

Merge pull request #20076 from eileenmcnaughton/token

Simplify mailing check

Merge pull request #20083 from mlutfy/dev2543on537

dev/core#2543 Fix Membership Edit PHP fatal when no recorded payment

Merge pull request #20084 from agh1/5.37.0-releasenotes-initial

5.37.0 release notes initial run

Changed " - Copy" append to "Copy of " prepend for consistency

Add " - Copy" to the end of the new mailing name when cloning

Fix Drupal8 CiviCRM menu translation on multilingual

Simplify mailing check

This should do the same thing - note org is never defined so replaceOrgTokens does
nothing (& is not used anywhere else)

5.37.0 release notes: added boilerplate

5.37.0 release notes: raw from script

dev/core#2543 Fix Membership Edit PHP fatal when no recorded payment

Merge pull request #20018 from seamuslee001/dev_core_2522

dev/core#2522 Add in Submit Action and redirect handling post form su…

Merge pull request #20079 from totten/master-logmgr

community/feature-request#12 - Allow named logging channels

Merge pull request #20080 from seamuslee001/dev_core_403

dev/core#403 Fix Petition Confirmation Email template by removing the…

dev/core#2522 Add in Submit Action and redirect handling post form submit

Move fields to the bottom of the config form and also rework the submission code as per suggestion from coleman and also ensure that the submit_action and redirect are exposed in the front end metadata

Remove debugging

Revmoe submit action field as no longer required

Make improvements as per Coleman's review

Update text as per Coleamn and fix regex to allos for ? = # & options in urls

Fix Regex to do some wildcard matching

Merge pull request #20064 from eileenmcnaughton/info

dev/core#2536 Trim strings from info.xml in extensions

Merge pull request #20067 from civicrm/5.37

5.37

dev/core#403 Fix Petition Confirmation Email template by removing the trailing space on the confirmation url from the href tag

community/feature-request#12 - Allow named logging channels

Overview
----------------------------------------

Make it easier to route log messages based on their topic (e.g. CiviContribute-related logs vs CiviMail-related logs).

Before
------

`Civi::log()` always returns the same instance of `LoggerInterface`, with no
clear way to differentiate logs of different business subsystems.

After
-----

`Civi::log(...)` allows you to optionally request a `LoggerInterface` for a specific theme, e.g.

```php
Civi::log('mail')->error('Failed to connect to SMTP server');
Civi::log('ipn')->warning('Transaction rejected by payment processor');
```

Technical Details
-----------------

A few things going on here:

* Extensions may start using their own logs (`Civi::log('myext')`) without any special effort.
* It is possible to replace or customize specific logs by defining a service `log.CHANNEL_NAME`.
* The `psr_log_manager` is a service. An extension like https://lab.civicrm.org/extensions/monolog/
  can replace the `psr_log_manager` and use the channel-name in its own way.

There is a limitation here in that the list of channels is open-ended.  It
will be impossible to (eg) detect that a log-user has made a typo in the
channel-name.  However, this seems like the better trade-off if the
alternative is that extensions face races during
installation/uninstallation.

Merge pull request #19886 from eileenmcnaughton/import

Fix for importing preferred language (and other fields) by label

Merge pull request #20071 from eileenmcnaughton/537d

dev/core#2498 dedupe rule saving fix

Merge pull request #20055 from davidjosephhayes/fix-authx-setting-display

Fix AuthX Setting Labels

Merge pull request #20074 from seamuslee001/fix_format_money

REF Ensure that getAmount includes 0s in decimal places up to 2 places

Merge pull request #20041 from eileenmcnaughton/mem_cont_switch

Switch to Call contribution api

Merge pull request #20073 from seamuslee001/dev_event_53

dev/event#53 Fix issue where by Sold out option was not being properl…

Merge pull request #20014 from eileenmcnaughton/mem_params

[Ref] Simplify getMembershipParameters

Merge pull request #20072 from colemanw/apiv4SettingFix

APIv4 - Fix Setting api to work with index param

REF Ensure that getAmount includes 0s in decimal places up to 2 places

dev/event#53 Fix issue where by Sold out option was not being properly frozen

APIv4 - Fix Setting api to work with index param

The api tries to force index params into the "Select" clause
which doesn't make sense for the oddball "Setting" api.

dev/core#2498 dedupe rule saving fix

Fixed bad bracket indent