Switch to throwing errrors rather than soft validation

diff --git a/CRM/Contact/BAO/Group.php b/CRM/Contact/BAO/Group.php
index 1c9b4756aab4fcc258b2998fa93b1275fa49ca2b..fcb7b61cb4715eef90663cd677611e32428365ad 100644 (file)
--- a/CRM/Contact/BAO/Group.php
+++ b/CRM/Contact/BAO/Group.php
@@ -392,15 +392,11 @@ class CRM_Contact_BAO_Group extends CRM_Contact_DAO_Group {
     if (isset($params['parents'])) {
       if (is_array($params['parents'])) {
         foreach ($params['parents'] as $parent => $dc) {
-          if (!CRM_Utils_Type::validate($parent, 'Integer', FALSE)) {
-            unset($params['parents'][$parent]);
-          }
+          CRM_Utils_Type::validate($parent, 'Integer');
         }
       }
       else {
-        if (!CRM_Utils_Type::validate($params['parents'], 'Integer', FALSE)) {
-          unset($params['parents']);
-        }
+        CRM_Utils_Type::validate($params['parents'], 'Integer');
       }
     }
     $group = new CRM_Contact_BAO_Group();
@@ -460,7 +456,7 @@ class CRM_Contact_BAO_Group extends CRM_Contact_DAO_Group {
 
       if (!empty($params['parents'])) {
         foreach ($params['parents'] as $parentId => $dnc) {
-          if (CRM_Utils_Type::validate($parentId, 'Integer', FALSE)) {
+          if (CRM_Utils_Type::validate($parentId, 'Integer')) {
             if ($parentId && !CRM_Contact_BAO_GroupNesting::isParentChild($parentId, $group->id)) {
               CRM_Contact_BAO_GroupNesting::add($parentId, $group->id);
             }

diff --git a/tests/phpunit/api/v3/GroupTest.php b/tests/phpunit/api/v3/GroupTest.php
index f08edb45fb4747c6dbef50ba41350ad94033edec..311e1f93bc22902f4727924ecaf41eaca6b22902 100644 (file)
--- a/tests/phpunit/api/v3/GroupTest.php
+++ b/tests/phpunit/api/v3/GroupTest.php
@@ -154,6 +154,8 @@ class api_v3_GroupTest extends CiviUnitTestCase {
       'is_active' => 1,
       'parents' => "(SELECT api_key FROM civicrm_contact where id = 1)",
     );
+    $this->callAPIFailure('group', 'create', $params);
+    unset($params['parents']);
     $this->callAPISuccess('group', 'create', $params);
     $group1 = $this->callAPISuccess('group', 'get', array(
       'title' => 'Test illegal Group',
@@ -164,6 +166,8 @@ class api_v3_GroupTest extends CiviUnitTestCase {
     $params['parents'] = array();
     $params['parents'][$this->_groupID] = 'test Group';
     $params['parents']["(SELECT api_key FROM civicrm_contact where id = 1)"] = "Test";
+    $group2 = $this->callAPIFailure('group', 'create', $params);
+    unset($params['parents']["(SELECT api_key FROM civicrm_contact where id = 1)"]);
     $group2 = $this->callAPISuccess('group', 'create', $params);
     $this->assertEquals(count($group2['values'][$group2['id']]['parents']), 1);
   }