Merge pull request #16594 from alifrumin/rn-5.23

First pass at release notes for 5.23

First pass at release notes for 5.23

Merge pull request #16588 from eileenmcnaughton/523partnotice

Fix enotice regression (unreleased)

Fix enotice regressionn

This was merged to 5.23 but I made a booboo in it
https://github.com/civicrm/civicrm-core/commit/a3aed42a453009733e8feb2c56fde5750b37cd14

I think the params are actually not used so it's just the notice it causes but this fixes

Merge pull request #16587 from seamuslee001/5.23

Add release-notes/5.22.1.md

Merge pull request #16583 from totten/5.23-gencode-no-l10n

xml/GenCode.php - Fix execution in default locale (without l10n data)

Add release-notes/5.22.1.md

xml/GenCode.php - Fix execution in default locale (without l10n data)

Overview
--------

This fixes a recent regression in which `xml/GenCode.php` fails to execute in certain configurations.

Initial report: https://github.com/civicrm/civicrm-buildkit/issues/503

Before
------

* If the folder `l10n` exists in its traditional location, and if you run `./bin/setup.sh -g`, then it
  correctly executes.
* If the folder `l10n` does not exist in its traditional locacation, and if you run `./bin/setup.sh -g`,
  then it fails with an error like this:
  ```
  + php -d mysql.default_host=127.0.0.1:3306 -d mysql.default_user=dmasterciv_abcde -d mysql.default_password=abcd1234 GenCode.php schema/Schema.xml '' Drupal

  civicrm_domain.version := 5.23.alpha1

  Parsing schema description schema/Schema.xml
  Extracting database information
  Extracting table information
  PHP Fatal error:  Uncaught RuntimeException: Invalid configuration: the [cms.root] path must be an absolute URL in /home/me/.../Civi/Core/Paths.php:174
  Stack trace:
  #0 /home/me/.../Civi/Core/Paths.php(151): Civi\Core\Paths->toAbsoluteUrl('/', 'cms.root')
  #1 /home/me/.../Civi/Core/Paths.php(176): Civi\Core\Paths->getVariable('cms.root', 'url')
  #2 /home/me/.../Civi/Core/Paths.php(151): Civi\Core\Paths->toAbsoluteUrl('/', 'civicrm.root')
  #3 /home/me/.../Civi/Core/Paths.php(224): Civi\Core\Paths->getVariable('civicrm.root', 'path')
  #4 /home/me/.../Civi/Core/Paths.php(84): Civi\Core\Paths->getPath('l10n')
  #5 [internal function]: Civi\Core\Paths->Civi\Core\{closure}()
  #6 /home/jo in /home/me/.../Civi/Core/Paths.php on line 174
  ```

After
-----

You can run `./bin/setup.sh -g` with or without the `l10n` folder.

Comments
--------

There's an argument to be made that this shouldn't be necessary: when running
`GenCode`, it should only create PHP files, and none of the output should depend
on the CMS URL - because that's liable to change when you deploy the PHP code.
If someone did try to generate URL at such an early stage, it's arguably good to
generate an error.  In point of fact, `GenCode` is not actually using the CMS
URL.

The oddity stems from the contract of `CRM_Utils_System_*` (specifically,
`getCiviSourceStorage()` and `getDefaultFileStorage()`) which do double-duty
providing both path and URL.  To avoid duplicate work, `Civi\Core\Paths` uses
the same grain of information - it tracks the pairs of path+URL.

A more aggressive fix might be to split `getCiviSourceStorage()` and
`getDefaultFileStorage()` so that it's possible to get the path and URL
separately; then revise `Civi\Core\Paths` to take advantage of the finer-grained
contract.  However, splitting those things would be a more invasive patch,
and we're currently in RC.

Merge pull request #16578 from seamuslee001/dev_core_1602

dev/core#1602 Fix join date filtering in search builder by standardis…

dev/core#1602 Fix join date filtering in search builder by standardising date field handling in Membership Query BAO

Merge pull request #16558 from eileenmcnaughton/523dupe

dev/core#1597 & dev/core#1595 fix regression when deduping on custom fields

dev/core#1597 & dev/core#1595 fix regression when deduping on custom fields

Merge pull request #16567 from eileenmcnaughton/parti_form

Fix theoretical regression

Merge pull request #16568 from JGaunt/patch-4

Update 5.23.0.md

Update 5.23.0.md

Added organisation name :)

Fix theoretical regression

This PR https://github.com/civicrm/civicrm-core/pull/16363/commits/44237a295dc5eab4f6110fb0556b1af742364f6e
(merged to 5.23) calls self::::getDefaultRoleID() in a way that was always flawed but definitely woudl
not work now. I think it is never reached but better to put this fix into 5.23 & maybe reemove the line in master
later since it assumes participant_status_id is optional - which is not true of the backoffice form

Merge pull request #16547 from seamuslee001/5.23

[NFC] Ensure that annon permissions are correctly assigned when enabl…

Merge pull request #16549 from seamuslee001/style_fix_property_bag

[NFC] Fix style error

[NFC] Fix style error

Merge pull request #16532 from eileenmcnaughton/5.23

dev/core#1588 Fix regression where empty string is passed to PropertyBag

[NFC] Ensure that annon permissions are correctly assigned when enabling civicrm webtest module in backdrop

Merge pull request #16542 from JGaunt/patch-3

Update CONTRIBUTORS.txt

Update CONTRIBUTORS.txt

Added in mine and my colleagues name.

Merge pull request #16535 from eileenmcnaughton/523rel

dev/core#1592 fix regression on relation active period

Merge pull request #16534 from eileenmcnaughton/523m

dev/core#1551 regression fix

dev/core#1592 fix regression on relation active period

Per https://lab.civicrm.org/dev/core/issues/1592 the custom query  for this field is being
ignored now it's using datepicker with relevant format changes. This  causes the
custom function for the field to  be called

dev/core#1551 regression fix

Now that mailing_id is defined as field metadata the form main metadata fn is trying to handle it, However
the form is casting it to the wrong array format -https://github.com/civicrm/civicrm-core/blob/86f20cc17dde028977130f84feaf3b1d760ca57d/CRM/Contact/Form/Search.php#L649

We could fix  this & rip it out of Mailing_BAO_Query::where but there is still a gap  in  that
there is no pseudoconstant recorded for the field so the qill doesn't work & there is a gap in
pre-parsing for the form 'IN'

Hence this seems like a good approach to remove the regression in the immediate term

dev/core#1588 Fix regression where empty string is passed to SettingsBag

We have a scenario where the checkbox is presented but is optional. This is then in 'params' which is passed through to
PropertyBag. The value is equal to '' so it fails to validate when we use it to set the value on the PropertyBag.

I don't think we lose anything meaningful by not setting an empty string and it avoids this error so we should
merge & release as a regression fix IMHO. If we want to revist then we should do that in master

https://lab.civicrm.org/dev/core/issues/1588

Merge pull request #16521 from seamuslee001/dev_core_1489_2

dev/core#1489 Include hook_civicrm_managed into the list of upgrade f…

dev/core#1489 Include hook_civicrm_managed into the list of upgrade freindly hooks

Merge pull request #16520 from agh1/5.23.0-release-notes

5.23.0 release notes

Merge pull request #16518 from greenpeace-cee/fix-xoauth-imap

dev/mail#59 - Fix use of useXOAUTH2 constant instead of var

dev/mail#59 - Fix use of useXOAUTH2 constant instead of var

This fixes an issue where a missing $ before the useXOAUTH2 variable
prevents IMAP authentication with XOAUTH2 from working.

5.23.0 release notes: added boilerplate

5.23.0 release notes: raw from script

Merge pull request #16483 from seamuslee001/utf8mb4_min

[NFC] Skip utf8mb4 test on MySQL versions lower than 5.7

Set version to 5.23.beta1

[NFC] Skip utf8mb4 test on MySQL versions lower than 5.7

Merge pull request #16485 from seamuslee001/master

release-notes: Late changes for 5.22 release notes

release-notes: Late changes for 5.22 release notes

Merge pull request #16457 from colemanw/api4doc

APIv4 documentation & code cleanup

Merge pull request #16472 from eileenmcnaughton/cn_for_now

Revert "Remove unreachable code lines."

Merge pull request #16466 from mattwire/recurringentity_copy

Call hook_civicrm_copy for RecurringEntity

Merge pull request #16471 from totten/master-psr-log

composer.json - Relax psr/log constraint. Improve D8 compatibility.

Revert "Remove unreachable code lines."

In the course of https://github.com/civicrm/civicrm-core/pull/16462
I concluded that there might be (untested scenarios) where this would still be hit.

In #16462 I adopted a cleaner (tested) approach. However, I think it's unlikely that
will be merged before 5.23 is cut so I propose reverting this change & re-reverting it
once the rc is cut as a more conservative approach. We can focus on getting #16462
wrapped up for 5.24

This reverts commit d7943ca7f7107c1466e687e876bddaf0f1998d99.

Merge pull request #16460 from colemanw/noInclude

APIv3 - Only scan files for deprecation checks in Entity.get

composer.json - Relax psr/log constraint. Improve D8 compatibility.

Overview
--------

This addresses a composer conflict that's reported when trying to install
on Drupal 8.7. For example, run these commands:

```
drush8 dl drupal-8.7.x
cd drupal-8*
composer require civicrm/civicrm-core:5.22.x-dev
```

This is a port of #16470 for `master`.

Before
------

The install fails because `civicrm-core` requires `psr/log:~1.1`, and something else
is prodding us to use `psr/log:1.0.2`.

```
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for civicrm/civicrm-core 5.22.x-dev -> satisfiable by civicrm/civicrm-core[5.22.x-dev].
    - Conclusion: remove psr/log 1.0.2
    - Conclusion: don't install psr/log 1.0.2
    - civicrm/civicrm-core 5.22.x-dev requires psr/log ~1.1 -> satisfiable by psr/log[1.1.0, 1.1.1, 1.1.2, 1.1.x-dev].
    - Can only install one of: psr/log[1.1.0, 1.0.2].
    - Can only install one of: psr/log[1.1.1, 1.0.2].
    - Can only install one of: psr/log[1.1.2, 1.0.2].
    - Can only install one of: psr/log[1.1.x-dev, 1.0.2].
    - Installation request for psr/log (locked at 1.0.2) -> satisfiable by psr/log[1.0.2].

Installation failed, reverting ./composer.json to its original content.
```

After
-----

It should work.  However, this is hard to demonstrate via `r-run` without merging.

Comments
--------

The substantive differences between `psr/log` in v1.0 and v1.1 relate to `LoggerInterfaceTest` and `TestLogger`:

https://github.com/php-fig/log/compare/1.0.2...1.1.2

However, `civicrm-core` does not use `LoggerInterfaceTest` or `TestLogger`, so it shouldn't matter.

For the standard tarballs which use `composer.lock`, this does have the side-effect of bumping up from 1.1.0 to 1.1.2.

Merge pull request #16458 from twomice/1573_extra_columns_contribution_detail_report

Fix for #1573: Extra columns for Contribution Detail report.

APIv4 Organize debug sql into a single array

Merge pull request #16469 from civicrm/5.22

5.22 to master

Merge pull request #16464 from WeMoveEU/core-1564-against-5.22

dev/core#1564 Fix category name for a/b test temporary table

Merge pull request #16463 from eileenmcnaughton/obscure

Fix obscure bug

Call hook_civicrm_copy when we copy entities using RecurringEntity

dev/core#1564 Fix category name for a/b test temporary table

Fix obscure bug

After upgrading to MacOS Catalina & MAMP 5.6 (latest) with php version 7.4.1 or 7.3.9
my site was unable to load CiviCRM with a segfault error. I traced it to this line & found that
phpstorm was highlighting the line it red as possibly subject to https://bugs.php.net/bug.php?id=62577

- taking phpstorm's recommendation allowed the site to load - the failure had been on parsing info.xml files

Merge pull request #16461 from eileenmcnaughton/api_test

Add require_once to api3TestTrait

Add require_once to api3TestTrait

I used this trait from an extension but it was used with apiv4 before apiv3 (it supports both).

I got an error because utils.php is not included by this file. It 'freeloads' off the fact apiv3 initialises
the file - but in fact the file that includes a function requiring a function
should have the require once - reduces risk & cognitive load

APIv3 - Only scan files for deprecation checks in Entiy.get

APIv4 - Pluralize entity name in docblocks

Fix for #1573: Extra columns for Contribution Detail report.

Merge pull request #16441 from eileenmcnaughton/event_pre

[REF] Interim code cleanup - make the usage of addPayments clearer

Api4 Replace - simplify by removing unnecessary fallback

Clearer docblock description of the defaults param

APIv4 Explorer - use OOP format for chained api calls

Passing an array into setChain works, but the more "pure" way is to pass an object into addChain.

Merge pull request #16446 from mattwire/wp_passlanguage

Pass language code to CiviCRM for Wordpress

Merge pull request #16397 from eileenmcnaughton/cont_setting3

dev/core#1558 retrieve setting credit_notes_prefix as  a 'real' setting.

Merge pull request #16449 from colemanw/docVars

APIv4 - Variable substution in docblocks

Merge pull request #16454 from colemanw/explorerDefaults

APIv4 Explorer - Better UI for defaults param

APIv4 Explorer - Better UI for defaults param

Merge pull request #16452 from mattwire/tokenrow_docblock

NFC Fix TokenRow docblock

Fix TokenRow docblock

APIv4 - Variable substution in docblocks

Replaces $ENTITY and $ACTION in docblocks to improve help text in the api explorer.

Merge pull request #16450 from eileenmcnaughton/ev_test

[NFC] Minor test cleanup.

Minor test cleanup.

Merge pull request #16438 from eileenmcnaughton/ev_clean

[REF] Pass params into function rather than this weird check for whether params exists.

Merge pull request #16447 from samuelsov/lab#1319

dev/core#1319 Fix regression with shared household address

dev/core#1319 UI - different label for household and employer

Pass language code to CiviCRM

Merge pull request #16267 from jitendrapurohit/dev-1471

dev/core#1471 - Add system status alert for deleted custom fields in …

[REF] Interim code cleanup - make the usage of addPayments clearer

This brings the condition back to the calling function which clarifies what is going on with it

Merge pull request #16437 from eileenmcnaughton/ev_back

Add unit test to back office form when a payment is made against a partially paid contribution

Merge pull request #16439 from eileenmcnaughton/part_odd

Respect form value for register_date on partipant form

dev/core#1558 retrieve credit_notes_prefix as a 'real' setting.

    This is part of work to convert the non-standard contribution_invoice_settings array of settings into settings in line
    with our standards

Extend checks in test

Merge pull request #16425 from colemanw/markdown

Use markdown in php docblocks & display in APIv4 Explorer

Merge pull request #16098 from MegaphoneJon/event-23-correct

event#23 - don't count non-counted participants on Confirm Registrati…

Respect form value for register_date on partipant form

The backoffice event registration form has a register_date field. In credit card mode it
does an awful lot of handling which is mostly for legacy code reasons but the net result is to ignore it
& set it to now. However, the field defaults to now & if someone actually changed it they would expect their change to
be respected. This removes that handling & just uses the field value entered. The field is always present from my
digging

Array formatting

[REF] Pass params into function rather than this weird check for whether params exists.

This function used to be shared with the online form & hence this form had to work to make itself 'look' like the other form.

It is now free to be it's own form

Merge pull request #16434 from eileenmcnaughton/cont_form_settings

[REF] Remove unused variable

Add unit test to back office form when a payment is made against a partially paid contribution

Merge pull request #16374 from colemanw/api4Delete

APIv4 - don't throw exception when updating/deleting 0 items

Merge pull request #16432 from colemanw/comments

Cleanup API_Exception usage

event#23 - don't count non-counted participants on Confirm Registration screen

Remove unused setting

Use markdown in php docblocks & display in APIv4 Explorer

Reformats some docblocks to use markdown, and uses the marked.js library
to display them in the APIv4 Explorer.

Merge pull request #16424 from eileenmcnaughton/cont_setting_handling

dev/core#1558 Data conversion for non-standard setting.

Merge pull request #16433 from eileenmcnaughton/fatal

dev/core#560 Replace fatal with exception

APIv4 - don't throw exception when updating 0 items

APIv4 - don't throw exception when deleting 0 items