Merge pull request #20153 from eileenmcnaughton/mem

[REF] Cleanup interaction with membership & membership id

Merge pull request #20203 from seamuslee001/dev_core_2582

dev/core#2582 Fix PCP and Pledge template urls in message templates t…

Merge pull request #20207 from seamuslee001/5372

5.37

Merge remote-tracking branch 'origin/5.37' into 5372

dev/core#2582 Fix PCP and Pledge template urls in message templates to go to front end site rather than backend

Merge pull request #20186 from colemanw/searchDisplayPerms

SearchKit - Pass-thu permission checks from SearchDisplay::run to api.get

SearchKit - Pass-thu permission checks from SearchDisplay::run to underlying API

The SearchDisplay::run api action essentially wraps api.get for the search entity,
so it makes sense to allow open access to the run api but require permission checks
on the delegated api call.

Merge pull request #20195 from eileenmcnaughton/537

dev/core#2577 Fix regression - payment edit to new type failing

Merge pull request #20201 from civicrm/5.37

5.37 to master

Merge pull request #20198 from alifrumin/rn5.37

[NFC] Release Notes 5.37

Merge pull request #20200 from jmcclelland/access-ajax-api

fix capitalization to allow proper access to 'access AJAX API' perm.

fix capitalization to allow proper access to 'access AJAX API' perm.

Maybe this is platform dependent? On my dev instance the access AJAX API
permission to the path /civicrm/ajax/api4/X/X is always denied for anon
users because anon users are granted "access AJAX API" yet the check is
done against "access Ajax API".

This only really matters in CRM/Api4/Permission.php but to avoid any
confusion I'm changing it everywhere the string appears.

5.37 release notes - first pass

Merge pull request #20180 from mattwire/recaptchacontribalways

Make reCAPTCHA not conditional on paymentprocessor on contribution page

Merge pull request #20194 from eileenmcnaughton/tok

[REF] Minor tidy up

If reCAPTCHA is enabled on contribution page always display (even if we don't have a paymentprocessor)

Merge pull request #20187 from eileenmcnaughton/dom

Make send letter function available to searchkit, individual contributions

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

dev/core#2577 Fix regression - payment edit to new type failing

Changes made in ref to master

[REF] Minor tidy up

This just removes some unused params & gets rid of the & before the function (it isn't called
in any weird ways and we have removed these before)

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

Make send letter function available to searchkit, individual contributions

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] Cleanup interaction with membership & membership id

This is intended to simplify https://github.com/civicrm/civicrm-core/pull/20077
by switching to using functions to retrieve membership (as an array) and membershipID
rather than passing around variables

[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