Skip to content

Commit 7a7c400

Browse files
committed
Remove "unset $args['id']" warning in view when creating a user group
1 parent 7337768 commit 7a7c400

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

featherbb/Controller/Admin/Groups.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,15 +107,16 @@ public function addedit($req, $res, $args)
107107

108108
AdminUtils::generateAdminMenu('groups');
109109

110-
$group = $this->model->info_add_group($groups, $args['id']);
110+
$id = isset($args['id']) ? intval($args['id']) : intval(Input::post('base_group'));
111+
$group = $this->model->info_add_group($groups, $id);
111112

112113
View::setPageInfo(array(
113114
'title' => array(Utils::escape(ForumSettings::get('o_board_title')), __('Admin'), __('User groups')),
114115
'active_page' => 'admin',
115116
'admin_console' => true,
116117
'group' => $group,
117118
'groups' => $groups,
118-
'id' => $args['id'],
119+
'id' => $id,
119120
'group_list' => $this->model->get_group_list($groups, $group),
120121
)
121122
)->addTemplate('admin/groups/add_edit_group.php')->display();

featherbb/Model/Admin/Groups.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,9 @@ public function info_add_group($groups, $id)
3636
$group = array();
3737

3838
if (Input::post('add_group')) {
39-
$group['base_group'] = intval(Input::post('base_group'));
39+
$group['base_group'] = $id; // Equals intval(Input::post('base_group'))
4040
$group['base_group'] = Container::get('hooks')->fire('model.admin.groups.add_user_group', $group['base_group']);
41-
$group['info'] = $groups[$group['base_group']];
41+
$group['info'] = $groups[$id];
4242

4343
$group['mode'] = 'add';
4444
} else {

0 commit comments

Comments
 (0)