Add tab for create, update and remove user group

refs #5543

application/controllers/GroupController.php

@@ -156,8 +156,7 @@ class GroupController extends AuthBackendController
         $form->setRepository($backend);
         $form->add()->handleRequest();
 
-        $this->view->form = $form;
+        $this->renderForm($form, $this->translate('New User Group'));
-        $this->render('form');
     }
 
     /**
@@ -181,8 +180,7 @@ class GroupController extends AuthBackendController
             $this->httpNotFound(sprintf($this->translate('Group "%s" not found'), $groupName));
         }
 
-        $this->view->form = $form;
+        $this->renderForm($form, $this->translate('Update User Group'));
-        $this->render('form');
     }
 
     /**
@@ -204,8 +202,7 @@ class GroupController extends AuthBackendController
             $this->httpNotFound(sprintf($this->translate('Group "%s" not found'), $groupName));
         }
 
-        $this->view->form = $form;
+        $this->renderForm($form, $this->translate('Remove User Group'));
-        $this->render('form');
     }
 
     /**
@@ -232,8 +229,7 @@ class GroupController extends AuthBackendController
             $this->httpNotFound(sprintf($this->translate('Group "%s" not found'), $groupName));
         }
 
-        $this->view->form = $form;
+        $this->renderForm($form, $this->translate('New User Group Member'));
-        $this->render('form');
     }
 
     /**

application/views/scripts/config/resource.phtml

@@ -3,7 +3,7 @@
 </div>
 <div class="content">
     <?= $this->qlink(
-        $this->translate('Create A New Resource') ,
+        $this->translate('Create a New Resource') ,
         'config/createresource',
         null,
         array(

application/views/scripts/config/userbackend/reorder.phtml

@@ -3,7 +3,7 @@
 </div>
 <div class="content">
     <?= $this->qlink(
-        $this->translate('Create A New User Backend') ,
+        $this->translate('Create a New User Backend') ,
         'config/createuserbackend',
         null,
         array(

application/views/scripts/group/list.phtml

@@ -25,12 +25,29 @@ if (! isset($backend)) {
     $extensible = $this->hasPermission('config/authentication/groups/add') && $backend instanceof Extensible;
     $reducible = $this->hasPermission('config/authentication/groups/remove') && $backend instanceof Reducible;
 }
+?>
 
+<?php if ($extensible): ?>
+    <?= $this->qlink(
+        $this->translate('Add a New User Group') ,
+        'group/add',
+        array('backend' => $backend->getName()),
+        array(
+            'class'             => 'button-link',
+            'data-base-target'  => '_next',
+            'icon'              => 'plus',
+            'title'             => $this->translate('Create a new user')
+        )
+    ) ?>
+<?php endif ?>
+
+<?php
+// @TODO(el): Remove $firstRow stuff
 $firstRow = true;
 foreach ($groups as $group): ?>
   <?php if ($firstRow): ?>
     <?php $firstRow = false; ?>
-<table data-base-target="_next" class="action group-list alternating">
+<table data-base-target="_next" class="action-table listing-table">
   <thead>
     <tr>
       <th class="group-name"><?= $this->translate('Group'); ?></th>
@@ -72,11 +89,4 @@ foreach ($groups as $group): ?>
 <?php else: ?>
 <p><?= $this->translate('No groups found matching the filter'); ?></p>
 <?php endif ?>
-<?php if ($extensible): ?>
-<?= $this->qlink($this->translate('Add a new group'), 'group/add', array('backend' => $backend->getName()), array(
-  'icon'                => 'plus',
-  'data-base-target'    => '_next',
-  'class'               => 'group-add'
-)); ?>
-<?php endif ?>
 </div>

application/views/scripts/group/show.phtml

@@ -50,6 +50,20 @@ if ($this->hasPermission('config/authentication/groups/edit') && $backend instan
 <?php endif ?>
 </div>
 <div class="content members" data-base-target="_next">
+<?php if ($extensible): ?>
+    <?= $this->qlink(
+        $this->translate('Add User to Group') ,
+        'group/addmember',
+        null,
+        array(
+            'class'             => 'button-link',
+            'data-base-target'  => '_next',
+            'icon'              => 'plus',
+            'title'             => $this->translate('Add user to group')
+        )
+    ) ?>
+<?php endif ?>
+
 <?php
 
 $firstRow = true;
@@ -97,14 +111,4 @@ foreach ($members as $member): ?>
 <?php else: ?>
   <p><?= $this->translate('No group member found matching the filter'); ?></p>
 <?php endif ?>
-<?php if ($extensible): ?>
-  <?= $this->qlink($this->translate('Add a new member'), 'group/addmember', array(
-    'backend'           => $backend->getName(),
-    'group'             => $group->group_name
-  ), array(
-    'icon'              => 'plus',
-    'data-base-target'  => '_next',
-    'class'             => 'member-add'
-  )); ?>
- <?php endif ?>
 </div>

application/views/scripts/navigation/index.phtml

@@ -9,7 +9,7 @@
 <?php endif ?>
 <div class="content">
     <?= $this->qlink(
-        $this->translate('Create A New Navigation Item') ,
+        $this->translate('Create a New Navigation Item') ,
         'navigation/add',
         null,
         array(

application/views/scripts/role/list.phtml

@@ -3,7 +3,7 @@
 </div>
 <div class="content">
     <?= $this->qlink(
-        $this->translate('Create A New Role') ,
+        $this->translate('Create a New Role') ,
         'role/add',
         null,
         array(

application/views/scripts/usergroupbackend/list.phtml

@@ -3,7 +3,7 @@
 </div>
 <div class="content">
     <?= $this->qlink(
-        $this->translate('Create A New User Group Backend') ,
+        $this->translate('Create a New User Group Backend') ,
         'usergroupbackend/create',
         null,
         array(

modules/monitoring/application/views/scripts/config/index.phtml

@@ -6,7 +6,7 @@
     <div>
     <h2><?= $this->translate('Monitoring Backends') ?></h2>
     <?= $this->qlink(
-        $this->translate('Create A New Monitoring Backend') ,
+        $this->translate('Create a New Monitoring Backend') ,
         'monitoring/config/createbackend',
         null,
         array(
@@ -61,7 +61,7 @@
     <div>
     <h2><?= $this->translate('Command Transports') ?></h2>
     <?= $this->qlink(
-        $this->translate('Create A New Command Transport') ,
+        $this->translate('Create a New Command Transport') ,
         'monitoring/config/createtransport',
         null,
         array(