Prevent group altering from trolling douchebags

This commit is contained in:
snipe
2015-09-15 02:46:40 -07:00
parent 06c9076c2f
commit 911e9092f9
3 changed files with 18 additions and 14 deletions
+15 -11
View File
@@ -163,7 +163,7 @@ class GroupsController extends AdminController
if (!Config::get('app.lock_passwords')) {
try {
try {
// Update the group data
$group->name = Input::get('name');
$group->permissions = Input::get('permissions');
@@ -196,18 +196,22 @@ class GroupsController extends AdminController
*/
public function getDelete($id = null)
{
try {
// Get group information
$group = Sentry::getGroupProvider()->findById($id);
if (!Config::get('app.lock_passwords')) {
try {
// Get group information
$group = Sentry::getGroupProvider()->findById($id);
// Delete the group
$group->delete();
// Delete the group
$group->delete();
// Redirect to the group management page
return Redirect::route('groups')->with('success', Lang::get('admin/groups/message.success.delete'));
} catch (GroupNotFoundException $e) {
// Redirect to the group management page
return Redirect::route('groups')->with('error', Lang::get('admin/groups/message.group_not_found', compact('id')));
// Redirect to the group management page
return Redirect::route('groups')->with('success', Lang::get('admin/groups/message.success.delete'));
} catch (GroupNotFoundException $e) {
// Redirect to the group management page
return Redirect::route('groups')->with('error', Lang::get('admin/groups/message.group_not_found', compact('id')));
}
} else {
return Redirect::route('groups')->with('error', Lang::get('general.feature_disabled'));
}
}