APIv4 - Remove unnecessary custom getActions class

The class override was not needed; the generic function works fine with custom entities.

Merge pull request #20175 from eileenmcnaughton/nfc

[REF] Minor code cleanup

Merge pull request #20172 from eileenmcnaughton/pdf

[REF] Complete decommissioning of CRM/Contribute/Form/Task/PDFLetterCommon.php

Merge pull request #20070 from larssandergreen/Clarify-event-scheduled-reminder-options

dev/core#2541 Clarify scheduled reminder options: Event Start Date -> Event Start, etc

Merge pull request #20092 from larssandergreen/fix-is-recur-label

dev/core#1452 Put text inside label tags for recurring contribution selection

Merge pull request #20176 from civicrm/5.37

5.37 to master

Merge pull request #20158 from jitendrapurohit/pricefieldfix

Fix priceset widget on Membership tab

[REF] Minor code cleanup

Minor code cleanups - use strict, declare exceptions, use use statement

Merge pull request #20173 from colemanw/noCustomReplace

APIv4 - Remove unnecessary class override (handled by generic)

Merge pull request #20161 from eileenmcnaughton/single

Remove apparent copy & paste from Contribution pdf letter function

[REF] Complete decommissioning on CRM/Contribute/Form/Task/PDFLetterCommon.php

This removes the last function fromCRM/Contribute/Form/Task/PDFLetterCommon.php

The function was calling a protected function on the parent which had to be unravelled.

I used the method we used in
https://github.com/civicrm/civicrm-core/blob/5e67eb7ff3f1f202d5112d15ad5c6a5f23f35795/CRM/SMS/Form/Upload.php#L341
to replace the code with the part that we have tested as being done in via render - ie replacing
hook, contact, domain tokens & smarty parsing.

However, there are quite a few tests on this code & one checked that we don't over-call
the token hook. I added some caching for the results of this hook so we can start to
eliminate these but I also allowed the calls to increment by 1 because it is not 1 per row

I made the results of this hook optional to replaceHookTokens & in general I think that
is the only place it needs to be called - perhaps once it used to pass more parameters &
the results were dynamic but now they really aren't

APIv4 - Remove unnecessary class override (handled by generic)

Remove apparent copy & paste from Contribution pdf letter function

I have done a bit of digging and I can't see any way these url parameters could be passed in
to this form. I also can't think of any reason, other than our old friend copy
& paste, for us to be retrieving activity or case in the url

I kept the isSingle logic but redirected it to focus on a single contribution which
is something we do want

Merge pull request #20171 from eileenmcnaughton/actsh

[REF] Add strict types to Utils_Mail::Send

Merge pull request #20166 from mattwire/recaptchacontributionbase

Simplify reCAPTCHA on ContributionBase form

Merge pull request #20167 from mattwire/recaptchaclean

REF Cleanup ReCAPTCHA code

[REF] Add strict types to Utils_Mail::Send

The function only returns a bool - this makes that explicit / forced and also removes handling for it
to return a PEAR_Error in one place that calls it. A couple of unused params are removed

Merge pull request #20160 from eileenmcnaughton/pdf

[REF] Post move cleanup - switch from $form to $this

Merge pull request #20169 from ixiam/dev_issue#2574

Fixes issue with state/province & country not translating values

Merge pull request #20164 from mattwire/recaptcharegistrationbuildcustom

REF reCAPTCHA Simplify buildCustom function on Registration form

[REF] Post move cleanup - switch from form to this

Cleanup ReCAPTCHA code

Fixes issue with state/province & country not translating values

Merge pull request #20165 from mattwire/recaptchasignaturebuildcustom

REF reCAPTCHA Simplify buildCustom function on Signature form

Remove warning about reCaptcha missing key from contribution page

Use CRM_Utils_ReCAPTCHA::enableCaptchaOnForm directly in ContributionBase

viewOnly parameter is not used in buildCustom for Signature

viewOnly parameter is not used in buildCustom for Registration

Simplify buildCustom function

Fix typos in comments and remove unused lines

Merge pull request #20159 from eileenmcnaughton/actsh

[REF] Minor tidy up - use writeRecord

Merge pull request #20157 from lemniscus/oauth-client

Fix a copy-and-paste error in the field definitions for OAuthSysToken

Merge pull request #20155 from eileenmcnaughton/pdf

[REF] Move more functions back to pdfLetter class

[REF] Minor tidy up - use writeRecord

This tidy up gets rid of 2 unused variables & an obsolete pass-by-ref and switches to a
preferred function call. Note that as a by-product hooks will now be called

Fix a copy-and-paste error in the field definitions for OAuthSysToken

Fix priceset widget on Membership tab

[REF] Move more functions bask to pdfLetter class

As with the previous moves this is
1) covered by unit tests
2) just code moving around

But note that this doesn't move whole functions but moves the
contents of postProcess & buildQuickForm back into the same functions
on the 'actual' class

Merge pull request #20134 from colemanw/searchKitLinks

Search Kit links improvements

Merge pull request #20143 from eileenmcnaughton/pdf

Move more functions to the pdfLetter class

Merge pull request #20154 from civicrm/5.37

5.37

Merge pull request #20111 from seamuslee001/wordpress_security_status_check_fix

[REF] Fix Security status check urls to work on WordPress

Merge pull request #20148 from seamuslee001/dev_core_2564

dev/core#2564 Fix class constants for when using phpseclib v1

Merge pull request #19991 from eileenmcnaughton/order

dev/core#2493 Default to not cleaning money for order.create api

Merge pull request #20151 from civicrm/5.37

5.37 to master

Merge pull request #20142 from colemanw/fixMultiValueContactRefFieldDedupe

Fix hard crash when deduping if any multiValued ContactRef fields exist

Fix hard crash when deduping if any multiValued ContactRef fields exist

Fixes https://lab.civicrm.org/dev/core/-/issues/2561

Merge pull request #20026 from totten/master-authx-details

Authx - Retain authentication outcome/metadata

dev/core#2564 - PhpseclibCipherSuite - Support v1 constants

Merge pull request #20137 from colemanw/fixCustomSerialize

dev/core#2506 Fix unserializing custom values during form reload

dev/core#2564 Fix class constants for when using phpseclib v1

Move more functions to the pdfLetter function

This 'common' form isn't really common - it's silly. This PR removes all
the gotcha instances of 'self::' to make it easier to decommission this file

Note that there are no calls to the class outside core in git universe

Merge pull request #20136 from eileenmcnaughton/dom

[REF] Move functions back to class that uses it

Merge pull request #20141 from colemanw/genericTpl

AfformGui - Specify admin tpl in entity config file & fall-back to ge…

AfformGui - Specify admin tpl in entity config file & fall-back to generic

Merge pull request #19986 from scardinius/core-2516

dev/core#2516 Invalidate only smart groups

Merge pull request #20140 from civicrm/5.37

5.37

Merge pull request #20127 from agileware/CIVICRM-1718

Set default terms = 1 in (edit) Membership form javascript

Merge pull request #20118 from demeritcowboy/actionlinks-extract

[REF] dev/core#2509 - Pull activity actionLinks creation into separate function

[REF] Move functions back to class that uses it

Fix unserializing custom values to work during form reload

Fixes https://lab.civicrm.org/dev/core/-/issues/2506
MultiValued custom ContactRef fields were losing their value
during form reload if there was a validation error.

SearchKit - Improve link titles to match entity titles

SearchKit - Improve UX for adding links to a search display

When adding links to a column, it will now only show relevant links to the entity in the column.
When adding a column of links/buttons, it will make all links available as before,
but by default will only show the ones relevant to the main entity.

SearchKit - Fix links to implicitly joined entities

This improves link construction to propery replace tokens in explicit joins and implicit joins.

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

CIVICRM-1718: Default terms = 1 in (edit) Membership form javascript

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

pull actionLinks creation into separate function

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

[REF] Fix Security status check urls to work on WordPress

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