From: Michael Devery <michael@compucorp.co.uk>
Date: Mon, 26 Mar 2018 10:59:54 +0000 (+0100)
Subject: CRM-21853: Do not change 'is_multiple' unless included in the params
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=e2cb881ce39d22d2169d3df551817872638ce92e;p=civicrm-core.git

CRM-21853: Do not change 'is_multiple' unless included in the params

Also remove use of CRM_Utils_Array call since we're sure it's set
---

diff --git a/CRM/Core/BAO/CustomGroup.php b/CRM/Core/BAO/CustomGroup.php
index 1ec8b47144..21e4563142 100644
--- a/CRM/Core/BAO/CustomGroup.php
+++ b/CRM/Core/BAO/CustomGroup.php
@@ -128,8 +128,8 @@ class CRM_Core_BAO_CustomGroup extends CRM_Core_DAO_CustomGroup {
       $fields[] = 'is_public';
     }
     foreach ($fields as $field) {
-      if (isset($params[$field]) || $field == 'is_multiple') {
-        $group->$field = CRM_Utils_Array::value($field, $params, FALSE);
+      if (isset($params[$field])) {
+        $group->$field = $params[$field];
       }
     }
     $group->max_multiple = isset($params['is_multiple']) ? (isset($params['max_multiple']) &&