(security/core#104) CRM_Utils_System::authenticateKey - Use secure equality test

(security/core#97) PHP CLI guard is the opposite of correct

Escape api params in APIv4 Explorer

Purify PCP introductory text field

Escape information supplied by extensions to prevent XSS

security/core#97 Ensure that php scripts where applicable in sql and tools that should only be run in CLI can be run in CLI

Use more portable check for cli and add in 404 header as per Rich's comments

security/core#100 Escape uploaded data to prevent Reflected Cross site scripting from uploaded CSVs

Merge pull request #19814 from eileenmcnaughton/535

avoid error when reserving respondents in a survey.

avoid error when reserving respondents in a survey.

The full error is:

Error: Call to a member function getSelectedIDs() on null in CRM_Campaign_Form_Task->preProcess() (line 38 of /var/www/powerbase/sites/all/modules/civicrm/CRM/Campaign/Form/Task.php).

I'm not sure if we should somehow be passing &$form to this function
instead?

Merge pull request #19776 from demeritcowboy/18830-5.35

dev/core#2137 - Backport of 18830 and its followups to not crash when asset builder can't do its thing

backport of 18830 and its followups

Merge pull request #19748 from eileenmcnaughton/535a

dev/translation#58 don\'t make group title NOT NULL even for a moment

dev/translation#58 don\'t make group title NOT NULL even for a moment

Set version to 5.35.0

Merge pull request #19725 from totten/5.35-avail-perm

dev/drupal#156 - system_get_info() is gone in Drupal 9

(NFC) release-notes/5.35.0 - Last minute update

system_get_info is deprecated

Merge pull request #19721 from agh1/5.35.0-releasenotes-final

5.35.0 release notes: added late changes

5.35.0 release notes: added late changes

Merge pull request #19711 from totten/5.35-upg-smtp

Upgrader (5.34) - Handle unsavable characters in decoded SMTP password

Merge pull request #19694 from eileenmcnaughton/dash

dev/core#2426 Fix regression whereby the dashboard crashes (permission related)

Upgrader (5.34) - Handle unsavable characters

Overview
--------

In php-mysqli with utf8mb4, the escaping rules do not handle 8-bit
characters (`chr(128)`+). ([Demo](https://gist.github.com/totten/4083741b920113ffc569d40053ce849d))

Here's a situation reported by @agileware-justin which provokes this:

> 1. SMTP credentials (mailing_backend) were saved and had been encrypted using mcrypt, prior to PHP 7.1
> 2. SMTP outbound email was NOT enabled, but the SMTP credentials are in the database
> 3. Active PHP version was PHP 7.3, without mcrypt module
> 4. CiviCRM 5.34 upgrade triggers the database error

Before
------

The behavior can be viewed in two variables:

* Depending on whether `CIVICRM_CRED_KEYS` is set, the upgrader may be
  writing passwords as plain-text or as `^CTK?` tokens.
* Depending on what value is in `$setting['smtpPassword']`, what value is in
  `CIVICRM_SITE_KEY`, and whether `mcrypt` is active, we may or may not get
  8-bit characters when reading the password
  (`CRM_Utils_Crypt::decrypt($setting['smtpPassword'])`).

The fatal combination arises when using plain-text with 8-bit characters.
But other combinations (encrypted tokens and/or 7-bit plain-text) seem
fine.

After
-----

As before, combinations involving encrypted tokens and/or 7-bit plain-text
are fine.

We don't have a head-on soultion for escaping 8-bit plain-text for use with
php-mysqli-utf8mb4.  (Which is insane, right?) But now we manage the
symptoms better:

* If you aren't even using SMTP (like in Justin's example),
  then this is not legit.  We show a warning and simply discard the
  unneeded/corrupt value of `smtpPassword`.

* If you are using SMTP, then this might theoretically be legit.
  (We haven't confirmed, but it seems plausible in other locales.) We show a
  different warning and encourage the sysadmin to setup `CIVICRM_CRED_KEYS`
  (which will enable the more permissive `^CTK?` format.)

dev/core#2426 Fix regression whereby the dashboard crashes for contacts unable to view their own contact record.

As noted in the code comments this was not my preferred technical fix but I do lean
towards using this in the rc & reconsidering the other approaches in master.

Also, in master I got a different error that I think related to new work

Fix angular error when user is not logged in.

Merge pull request #19705 from seamuslee001/update_schema_handler_test

[NFC] Update Schema Handler to use the standard create table syntax o…

[NFC] Update Schema Handler to use the standard create table syntax of using ROW_FORMAT=dynamic

Merge pull request #19698 from eileenmcnaughton/custom

dev/core#2423 Fix quasi-regression around serialized custom fields

Bring back some madness

dev/core#2423 Fix quasi-regression around serialized custom fields

https://lab.civicrm.org/dev/core/-/issues/2423

Merge pull request #19692 from demeritcowboy/case-custom-money-5.35

dev/core#2394 - Don't crash when saving custom case fields of type money

Merge pull request #19680 from eileenmcnaughton/money

dev/financial#166 Fix for inconsistency around currency symbol

Merge pull request #19696 from eileenmcnaughton/user

dev/core#2427 Fix user creation regression

dev/core#2427 Fix user creation regression

dev/financial#166 Fix for inconsistency around currency symbol

From https://lab.civicrm.org/dev/financial/-/issues/166 we learn that the existing code
(tested via testFormatLocaleNumericRoundedByCurrency) is not consistent across
all platforms. I think this may be

Merge pull request #19685 from eileenmcnaughton/535

dev/core:2394 Fix for number formatting regression

don't crash when saving custom fields of type money

dev/core:2394 Fix for number formatting regression

This function has been picking up the formatting for the locale
meaning that the currency separator replacement is already done and
is swapped back by formatLocaleNumericRoundedByPrecision

Doing it via brickmoney is better than our custom
separator replacement  - but we need a quick fix for the rc

Merge pull request #19670 from eileenmcnaughton/aff

Switch afform back to '=' to unbreak deduper

Switch afform back to '=' to unbreak deduper

https://github.com/eileenmcnaughton/deduper/pull/9

Merge pull request #19659 from demeritcowboy/revert-18782

revert 18782

revert 18782

Merge pull request #19653 from eileenmcnaughton/535

dev/core#2360 - Escape the word `rows` in sql query

dev/core#2360 - Escape the word `rows` in sql query

Merge pull request #19593 from eileenmcnaughton/535

Fix Redis deprecated warning

Merge pull request #19594 from eileenmcnaughton/535m

dev/mail#89 Fix unreleased regression where civimember is not permitted/enabled

dev/mail#89 Fix unreleased regression where civimember is not permitted/enabled

Fixes a bug where a person with no access to CiviMember will get an exception thrown (and
not caught) rather than receive a result of 0 when checking the memberships
that a contact has access to

Fix Redis deprecated warning

It's not obvious why we would pass trapException here. I believe the issue is that
some custom searches result in 'acceptable' bad sql here but
https://github.com/civicrm/civicrm-core/commit/6dc40f3250ede0f38ce2aed7c3fabd3f1f667c1b
already fixed to catch any exception so we don't
need to convert. There is one other place that calls fillWithSql but there
is no evidence it is subject to the random issue on custom searches / needs
any attention at the moment

Merge pull request #19586 from alifrumin/5.35-releasenotes

[NFC] First Pass at 5.35 release notes

5.35 Release Notes - First Pass

Merge pull request #19573 from totten/5.35-rand37

dev/core#2370 - Installer - Bump up entropy for autogenerated cred keys

dev/core#2370 - Installer - Bump up entropy for autogenerated cred keys

This slightly expands the amount of entropy for certain auto-generated values.

Before
-----

~99% of generated values have >=232 bits

After
-----

~99% of generated values have >=260 bits

Technical details
--------

https://lab.civicrm.org/dev/core/-/issues/2370#note_53832

Merge pull request #19563 from eileenmcnaughton/5.35

Fix unreleased regression from mistake extracting function

Fix unreleased regression from mistake extracting function

Ports this line (merged to master) to 5.35 as the mistake turns out to affect 5.35
https://github.com/civicrm/civicrm-core/pull/19551/files#diff-447cfa0a0ec021e7cf54c6f207d94c3e3343eec930130a995355fec37a590a22R597

Merge pull request #19546 from colemanw/labelField

APIv4 - Fix label_field to use underscore instead of camelCase

APIv4 - Fix label_field to use underscore instead of camelCase

Merge pull request #19529 from agh1/5.35.0-releasenotes-initial

5.35.0 release notes - initial run

5.35.0 release notes: added boilerplate

5.35.0 release notes: raw from script

Set version to 5.35.beta1

Merge pull request #19521 from civicrm/5.34

5.34

Merge pull request #19519 from colemanw/afformGuiFixBlocks

Afform GUI - Fix errors when creating & saving blocks

Merge pull request #19520 from agh1/5.34.0-releasenotes-final

5.34.0 release notes: final edits and late changes

5.34.0 release notes: final edits and late changes

Afform Gui - Exclude self from list of available blocks when editing a block

Merge pull request #19464 from eileenmcnaughton/menu

dev/core#2340 Skip rather than fail on bad menu item

Merge pull request #19514 from colemanw/apiRelativeDateFix

APIv4 - Fix bug when using relative date filters in ON clause of a join

Afform GUI - Fix errors when creating & saving blocks

Merge pull request #19517 from seamuslee001/fix_user_load_deprecation

Override the DrupalBase getUserObject function with a Drupal8/9 compa…

Merge pull request #19516 from colemanw/apiGetFkFields

APIv4 - Enable getFields to find fields across implicit FK joins

Merge pull request #19510 from eileenmcnaughton/msg_tpl

[REF] Extract logical functions from sendTemplate

Override the DrupalBase getUserObject function with a Drupal8/9 compatible version

Merge pull request #19443 from eileenmcnaughton/ipn

[REF] Move sendNotification out of recur, remove unused related_contact

Merge pull request #19511 from eileenmcnaughton/mem_clean

[REF] Remove invalid attempt to load contriution id from invoiceID

Merge pull request #19498 from eileenmcnaughton/complete_order_test

[REF] Stop passing contributionPageID to isEmailReceipt

Merge pull request #19515 from demeritcowboy/amtg-notice

dev/core#2355 - E_NOTICE on Manage Contribution and Manage Event listings

APIv4 - Enable getFields to find fields across implicit FK joins

Now it is possible to retrieve field metadata for a joined entity

E_NOTICE on contribution pages

APIv4 - Fix bug when using relative date filters in ON clause of a join

Merge pull request #19496 from colemanw/afformBugFixes

Afform - Misc tweaks, validation & bug fixes

Merge pull request #19513 from eileenmcnaughton/kernel_error

Move require once in api kernel to support edge cases

Merge pull request #19492 from demeritcowboy/fatalerrorhandler

dev/core#2350 - Oauth extension - Setting a fatalErrorHandler might override the redirect url because typo

config-form.html - Fix rendering of client-side validation error for "server_route"

Move require once in api kernel to support edge cases

The function 'createError' on the kernel class calls 3 functions that are in the api v3 utils file.
It is possible to reach this from apiv4 using drush (I realise we
don't really know much about how to support api v4 with drush but it DOES work with a simple
api with this patch in my testing). We could go with only including utils.php in
v4 api AND the createError function. I think it's more readable this way.

To test this try

 drush  cvapi  Contribution.get version=4

it will legitimately fail without checkPermissions=0 but it will fail with a
require_once error without this patch

Search Kit - Improve display of afforms in search list

Remove defunct reference to skilCleanMoney

The BAO used to handle this param but it no longer does. The api does respond to it - but defaults to
true (we don't call the api here but anything other than setting it to false will always do nothing)

Remove attempt to load contriution id from invoiceID

This might make sense (maybe) on the front end code this was previously shared with but on the
backoffice form id would be specifically set in the url

[REF] Extract logical functions from sendTemplate

This simply makes it clear what the main chunks of code are doing

[REF] Stop passing contributionPageID to isEmailReceipt

By switching to passing in contributionID instead we can significantly simplify this.

It could cause an extra query in some cases but the query would be very quick based
on my previous query reduction efforts and this will help us
remove a bunch of other code

Merge pull request #19417 from eileenmcnaughton/test

[REF] Standardise methods of determining isTest

Merge pull request #19505 from mattwire/remotememrelated

Remove another use of loadRelatedObjects() function

Merge pull request #19500 from eileenmcnaughton/cancel

dev/core#2206 Unhide contributioncancelactions core extension

Merge pull request #19507 from eileenmcnaughton/trans

Comment intent to remove call to transition components

Comment intent to remove call to transition components

The other usages have been migrated to contributioncancelactions so only update-to-completed remains

Merge pull request #19502 from mattwire/sendReminder

CRM_Core_BAO_MessageTemplate::sendReminder() is not used anywhere

Merge pull request #19504 from colemanw/labelField

Add "labelField" metadata to entities

Unhide contributioncancelactions

With https://github.com/civicrm/civicrm-core/pull/19289 merged we
can now unhide this extension & people can disable it if they wish (as a
supported option)

Remove another use of loadRelatedObjects() as we call it again before using

Merge pull request #19503 from mattwire/casecomponentignoreexception

Use exceptions when enabling case component / checking for 'CREATE VIEW' permissions

Add "labelField" metadata to entities

This determines which field will be shown when displaying a single record
e.g. when viewing a Contact, show the display_name.

Afform - Fix repeatable blocks in GUI