Merge pull request #19726 from seamuslee001/5.35

5.35

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 #19720 from colemanw/afformSearchFilters

Improve search display filter validation

SearchKit - Validate all filters as belonging to select clause or afform

When viewing a SearchDisplay, this will verify all filters are permitted by first checking the SELECT clause,
and secondly checking for a containing Afform with exposed filters.

All other filter params will be silently ignored.

Afform - use form-level permissions in APIv4 Afform::get

Merge pull request #19723 from eileenmcnaughton/price

dev/core#2434 Fix price set id widget

Merge pull request #19709 from eileenmcnaughton/ss

dev/core#2422 Add created_id, modified_id, expires_date to saved search table

dev/core#2434 Fix price set id widget

Merge pull request #19722 from civicrm/5.35

5.35

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 #19476 from eileenmcnaughton/mem_tax

Fix membership form to correctly calculate tax when a discount is applied

Merge pull request #19715 from mattwire/tokenclean

NFC Update comments in token classes

Add description to saved search

Update comments in token classes

Merge pull request #19590 from totten/master-authx

AuthX - Extended authentication support (portable and router-friendly)

Also add created_date & modified_date per discussion on pr

dev/core#2422 Add created_id, modified_id, expires_date to saved search table

Per

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

These can be exposed in the search kit saved search listing as filters when
we get to that point but the earlier we start
saving them the better the data will be when we do expose

Merge pull request #19714 from civicrm/5.35

5.35

Merge pull request #19703 from colemanw/afformStandalone

Afform - Remove ngRoute from afformStandalone page

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

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

Merge pull request #19684 from totten/master-authors

info.xml - Allow multiple `<author>`s

Merge pull request #19712 from eileenmcnaughton/merge

5.35 to master

Merge branch '5.35' of https://github.com/civicrm/civicrm-core into merge

(REF) authx - Reorganize internal methods to use an auth-request object

This significantly trims down the `auth()` method and rearranges as three
methods. A data object is passed between the three methods. The main method:

```
    if ($principal = $this->checkCredential($tgt)) {
      $tgt->setPrincipal($principal);
    }
    $this->checkPolicy($tgt);
    $this->login($tgt);
```

This arrangement lays the groundwork for implementing more varied policies.
For example, we could have a policy where the ability to login via
username/password/api_key is dictated by the user's role or permissions.

(REF) Authenticator - Extract field $authxUf

authx - If using auto-login, then use a redirect to hide the `?_authx=...` gunk

authx - The "already logged in" check should be less sensitive to int-vs-string for user/contact ID

authx - Support Joomla users+sessions - Prevent leaks among stateless requests

AllFlowsTest - When executing on a new deployment, ensure demo user has a Civi contact

(REF) AllFlowsTest - Cleanup helper signature

AllFlowsTest - Add cases with multiple identities from the same browser

AllFlowsTest - Add test for checking contact+user vs contact-only

distmaker - Add authx to tarball

authx - Make it safe to load the 'authx.settings.php' file multiple times

authx - Support Joomla users+sessions

authx - Support Drupal 8 users+sessions

authx - Support Backdrop users+sessions

(REF) AllFlowsTest - Misc cleanup

authx - Support login/logout flow

authx - Support JWT credentials

authx - Primary implementation, including Drupal 7 and WordPress

authx - Add the AllFlowsTest

authx - Add page to check identity ('civicrm/authx/id')

authx - Declare settings for allowed flows and credentials

authx - Draft README and info.xml

authx - Generate skeleton

CRM_Core_Session - Add option 'useFakeSession()' for stateless requests

CRM_Core_Invoker - Emit authorization event

Afform - use search params instead of route params

Afform - Remove ngRoute from afformStandalone page

Afforms are meant to work as standalone directives so routing on the standalone page was meaningless

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

Merge pull request #19683 from colemanw/searchDisplayFixes

Search display fixes

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

Merge pull request #19708 from J0WI/patch-1

Fix migrateUtf8mb4 for special database names

Merge pull request #19675 from demeritcowboy/casedashlets

dev/core#1400 - Put dashlets back to only showing open cases by default

Fix angular error when user is not logged in.

Fix migrateUtf8mb4 for special database names

Merge pull request #19702 from seamuslee001/domain_petition_emails

[NFC] Add in unit test to prove that domain tokens are able to be inc…

Merge pull request #19706 from civicrm/5.35

5.35

Merge pull request #19705 from seamuslee001/update_schema_handler_test

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

Merge pull request #19704 from colemanw/removeDeprecatedAngularJs

Angular - Remove deprecated js file

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

[NFC] Add in unit test to prove that domain tokens are able to be incldued in peition emails

Merge pull request #19698 from eileenmcnaughton/custom

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

Angular - Remove deprecated js file

Merge pull request #19701 from civicrm/5.35

5.35

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 #19699 from colemanw/controlField

APIv4 - Rename 'controlField' property to 'control_field'

Merge pull request #19695 from civicrm/5.35

5.35

APIv4 - Rename 'controlField' property to 'control_field' for consistency

By convention, all keys returned from the api should be lowercase.

Merge pull request #19690 from colemanw/api3Operators

APIv3 - Improve array-based apis to support sorting and operators

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

Merge pull request #19629 from MegaphoneJon/core-2386

core#2386 - metadata-driven chain-select fields

APIv3 - Improve array-based apis to support sorting and multiple operators

This backports some APIv4 code to v3, for the purpose of supporting
entityRef widgets for Afform.

Merge pull request #19668 from eileenmcnaughton/empty_rows

dev/core#2420 Add text to report template saying no results found if that is the case

dev/core#2427 Fix user creation regression

Merge pull request #19693 from eileenmcnaughton/if

[REF] clean up if

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

Merge pull request #19689 from demeritcowboy/my-cases-listtype

dev/core#1207 - My Cases dashlet shouldn't crash for users with only my cases access

[REF] clean up if

This just makes 2 lines simpler. The issue is that if checks the same thing
twice - presumably the if empty check got added for the e-notice but
the other was not removed

don't crash when saving custom fields of type money

Merge pull request #19688 from demeritcowboy/lottery-win

Fix for missing function CodeGen_Util::removeDir

don't crash for users with only my cases access

dev/core#2418 - info.xml - Show full author list

dev/core#2418 - info.xml - Parse `<authors>` tag

missing function

Merge pull request #19686 from colemanw/searchKitClearSelection

SearchKit - Fix clearing selection behavior

SearchKit - Fix clearing selection behavior

The variables `allRowsSelected` and `selectedRows` were sometimes getting out-of-sync.
This adds a function to help ensure they both get updated at the same time when the selection needs to be cleared.

Merge pull request #19674 from demeritcowboy/caseclosed

dev/core#1400 - Failing test showing that My Cases users can't view their own closed cases

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 #19682 from colemanw/dashlet-spacing

Reduce wasted space in civicrm dashboard margins

SearchKit - Fix js error when adding label to buttons/links in List display