From 8e2480de287adb42f4d8f39f054cc8b798380cf4 Mon Sep 17 00:00:00 2001 From: Andrew Hunt Date: Tue, 31 Dec 2019 15:20:21 -0500 Subject: [PATCH] 5.21.0 release notes: added late changes --- release-notes/5.21.0.md | 67 ++++++++++++++++++++++++++++++----------- 1 file changed, 50 insertions(+), 17 deletions(-) diff --git a/release-notes/5.21.0.md b/release-notes/5.21.0.md index 5db72c5571..1f5371160c 100644 --- a/release-notes/5.21.0.md +++ b/release-notes/5.21.0.md @@ -37,8 +37,9 @@ Released January 1, 2020 [15823](https://github.com/civicrm/civicrm-core/pull/15823), [15826](https://github.com/civicrm/civicrm-core/pull/15826), [15820](https://github.com/civicrm/civicrm-core/pull/15820), - [15825](https://github.com/civicrm/civicrm-core/pull/15825) - and [15876](https://github.com/civicrm/civicrm-core/pull/15876))** + [15825](https://github.com/civicrm/civicrm-core/pull/15825), + [15876](https://github.com/civicrm/civicrm-core/pull/15876), and + [16155](https://github.com/civicrm/civicrm-core/pull/16155))** Begins work to enforce a common naming convention for temporary tables. @@ -138,14 +139,16 @@ Released January 1, 2020 - **Develop getter & setter structure for Payment classes (Work Towards [dev/financial#82](https://lab.civicrm.org/dev/financial/issues/82): - [15697](https://github.com/civicrm/civicrm-core/pull/15697))** + [15697](https://github.com/civicrm/civicrm-core/pull/15697), + [16052](https://github.com/civicrm/civicrm-core/pull/16052), + [16060](https://github.com/civicrm/civicrm-core/pull/16060), and + [16081](https://github.com/civicrm/civicrm-core/pull/16081))** - This PR adds a new class, `Civi\Payment\PropertyBag` which will eventually - replace the wild `$params` array that gets thrown at payment processors, with - a standardized, at least minimally validated and type cast object. It uses the - new class in one very minimal way and adds some comments to outline future - work. This is a non functional change that lays the groundwork for future - work. + This PR adds a new class, `Civi\Payment\PropertyBag` which is designed to + offer a consistent object for holding payment parameters as they are sent to + and from payment processors. For now, the legacy array of parameters is still + valid, so this will not have much immediate impact, but it lays the groundwork + for future work. - **Always display amount paid even if it is 0.00. Blank looks like it's missing data rather than nothing paid @@ -188,6 +191,22 @@ Released January 1, 2020 ### Core CiviCRM +- **CiviCRM Option Group with a name containing spaces cannot have any options + added, error message is shown also impacts the in-place option editor + ([dev/core#1405](https://lab.civicrm.org/dev/core/issues/1405) and + [dev/core#1447](https://lab.civicrm.org/dev/core/issues/1447): + [15937](https://github.com/civicrm/civicrm-core/pull/15937) and + [16033](https://github.com/civicrm/civicrm-core/pull/16033))** + + Option group names are no longer exposed for editing, as they are now + automatically generated from the title. On upgrade, existing option group + names that contain spaces will be rewritten. *Extensions that provide or rely + upon option group names that contain spaces will need to be rewritten.* + + Upon upgrade, if rewriting an option group name will duplicate another option + group's name, that option group will be skipped and a post-upgrade message + will show the details. + - **Lybunt report - remove broken chart functionality ([dev/core#523](https://lab.civicrm.org/dev/core/issues/523): [15989](https://github.com/civicrm/civicrm-core/pull/15989))** @@ -274,15 +293,22 @@ Released January 1, 2020 - **Fix activity search to work by url params ([15990](https://github.com/civicrm/civicrm-core/pull/15990))** -- **CiviCRM Option Group with a name containing spaces cannot have any options - added, error message is shown also impacts the in-place option editor - ([dev/core#1405](https://lab.civicrm.org/dev/core/issues/1405): - [15937](https://github.com/civicrm/civicrm-core/pull/15937))** - - **Quicksearch with phone filter doesn't work with non-numeric character ([dev/core#1420](https://lab.civicrm.org/dev/core/issues/1420): [15955](https://github.com/civicrm/civicrm-core/pull/15955))** +- **Trying to edit the settings for a reserved option group gives a network + error can't connect to server + ([dev/core#1477](https://lab.civicrm.org/dev/core/issues/1477): + [16107](https://github.com/civicrm/civicrm-core/pull/16107))** + +### CiviCampaign + +- **Activities not visible if they have a campaign_id & contact does not have + 'administer CiviCampaign' + ([dev/core#1444](https://lab.civicrm.org/dev/core/issues/1444): + [16057](https://github.com/civicrm/civicrm-core/pull/16057))** + ### CiviCase - **Case Activities Report includes core activities *always* @@ -307,13 +333,20 @@ Released January 1, 2020 ([dev/financial#100](https://lab.civicrm.org/dev/financial/issues/100): [15771](https://github.com/civicrm/civicrm-core/pull/15771))** - To avoid users accidentally setting invalid statuses Removes the statuses - "Partially Paid", "Cancelled", and "Failed" from the `Payment Status` field - when recording a payment for a Contribution related to a Membership. + To avoid users accidentally setting invalid statuses, this removes the + statuses "Partially Paid", "Cancelled", and "Failed" from the `Payment Status` + field when recording a payment for a Contribution related to a Membership. Additionally, removes the statuses "Partially Paid", "Overdue" and "In Progress" from the `Payment Status` field when recording a payment for a Contribution related to an Event. +- **hook_civicrm_post() implementation results in DB Error: already exists for + custom field values + ([dev/core#1443](https://lab.civicrm.org/dev/core/issues/1443): + [16061](https://github.com/civicrm/civicrm-core/pull/16061))** + + The post hook is invoked after storing custom values when adding contributions. + - **Fix api Payment.create to support overpayments ([15909](https://github.com/civicrm/civicrm-core/pull/15909))** -- 2.25.1