Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

check that `changes` param is present

This commit is contained in:
Jason W. May 2014-11-21 10:03:29 -08:00
parent 50de22801f
commit 98404d19c5
1 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
app/controllers/admin/groups_controller.rb
View File

@ -23,15 +23,16 @@ class Admin::GroupsController < Admin::AdminController
def update_patch(group)
raise Discourse::InvalidAccess.new("automatic groups do not permit membership changes") if group.automatic
actions = params[:changes]
Array(actions[:add]).each do |username|
if user = User.find_by_username(username)
group.add(user)
if actions = params[:changes]
Array(actions[:add]).each do |username|
if user = User.find_by_username(username)
group.add(user)
end
end
end
Array(actions[:delete]).each do |username|
if user = User.find_by_username(username)
group.remove(user)
Array(actions[:delete]).each do |username|
if user = User.find_by_username(username)
group.remove(user)
end
end
end