Merge pull request #20193 from kartik1000/5.37

dev/core#2571: Removed Forced Recaptcha function as it called only once

Merge pull request #20192 from colemanw/deprecateAclCheck

Add deprected warning to apparently unused functions

Changes made in ref to master

Add deprected warning to apparently unused functions

It looks like one check function is only ever called by the other,
and the other isn't called from anywhere.

What these functions are supposed to do is unclear.
The code comment appears to be wrong; the variable $str
is not a permission string but is inserted into a query
as civicrm_acl.object_table.

Merge pull request #20138 from eileenmcnaughton/load

[REF] Remove redundant code

Merge pull request #20185 from civicrm/5.37

5.37

Merge pull request #20174 from demeritcowboy/api-caseactivity-perm

dev/core#2573 - Set appropriate default permissions to use api4 CaseActivity

Merge pull request #20181 from colemanw/anotherOneBitesTheDust

APIv4 - Remove unnecessary custom getActions class

Merge pull request #20183 from ufundo/patch-1

add me

add me

for first merged PR as requested by @demeritcowboy :)

Merge pull request #20162 from lemniscus/oauth-client

oauth-client - Add test for existing Auth Code flow behavior

Merge pull request #20182 from mattwire/sendconfirmationreturn

Contribution.sendconfirmation API3 function should return

API functions should return otherwise you get 'cannot interpret return values from function'

APIv4 - Remove unnecessary custom getActions class

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

Merge pull request #20147 from seamuslee001/afform_redirect_html

dev/core#2569 Add in Redirect field into the html afform editor

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

without this you need administer CiviCRM permission to use api4 CaseActivity

[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

(NFC) making the whitespace how Jenkins wants it

(NFC) put code into Drupal style

Add test for existing Auth Code flow behavior

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

Add in Redirect field into the html afform editor

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] Remove redundant code

[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.