Make sure no empty group is submitted on project permissions page

app/Controller/ProjectPermissionController.php

@@ -135,10 +135,14 @@ class ProjectPermissionController extends BaseController
             $values['group_id'] = $this->groupModel->getOrCreateExternalGroupId($values['name'], $values['external_id']);
         }
 
-        if ($this->projectGroupRoleModel->addGroup($project['id'], $values['group_id'], $values['role'])) {
+        if (empty($values['group_id'])) {
-            $this->flash->success(t('Project updated successfully.'));
+            $this->flash->failure(t('Unable to find this group.'));
         } else {
-            $this->flash->failure(t('Unable to update this project.'));
+            if ($this->projectGroupRoleModel->addGroup($project['id'], $values['group_id'], $values['role'])) {
+                $this->flash->success(t('Project updated successfully.'));
+            } else {
+                $this->flash->failure(t('Unable to update this project.'));
+            }
         }
 
         $this->response->redirect($this->helper->url->to('ProjectPermissionController', 'index', array('project_id' => $project['id'])));

app/Locale/bs_BA/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/ca_ES/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/cs_CZ/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/da_DK/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/de_DE/translations.php

@@ -1376,4 +1376,5 @@ return array(
     'This task is now linked to the task %s with the relation "%s"' => 'Diese Aufgabe ist jetzt verknüpft mit der Aufgabe %s mit der Relation "%s"',
     'The link with the relation "%s" to the task %s has been removed' => 'Die Verknüpfung mit der Relation "%s" zur Aufgabe %s wurde entfernt',
     'Custom Filter:' => 'Benutzerdefinierter Filter:',
+    // 'Unable to find this group.' => '',
 );

app/Locale/el_GR/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/es_ES/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/fi_FI/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/fr_FR/translations.php

@@ -1376,4 +1376,5 @@ return array(
     'This task is now linked to the task %s with the relation "%s"' => 'Cette tâche est maintenant liée à la tâche %s avec la relation « %s »',
     'The link with the relation "%s" to the task %s has been removed' => 'Le lien avec la relation « %s » de la tâche %s a été supprimé',
     'Custom Filter:' => 'Filtre personalisé :',
+    'Unable to find this group.' => 'Impossible de trouver ce groupe.',
 );

app/Locale/hr_HR/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/hu_HU/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/id_ID/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/it_IT/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/ja_JP/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/ko_KR/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/my_MY/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/nb_NO/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/nl_NL/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/pl_PL/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/pt_BR/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/pt_PT/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/ro_RO/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/ru_RU/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/sr_Latn_RS/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/sv_SE/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/th_TH/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/tr_TR/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/vi_VN/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/zh_CN/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );

app/Locale/zh_TW/translations.php

@@ -1376,4 +1376,5 @@ return array(
     // 'This task is now linked to the task %s with the relation "%s"' => '',
     // 'The link with the relation "%s" to the task %s has been removed' => '',
     // 'Custom Filter:' => '',
+    // 'Unable to find this group.' => '',
 );