diff options
| author | Douwe Maan <douwe@gitlab.com> | 2015-07-31 15:15:49 +0300 |
|---|---|---|
| committer | Douwe Maan <douwe@gitlab.com> | 2015-07-31 15:15:49 +0300 |
| commit | 0736f348a6953e6417d3857b78ea5c7eb5954e51 (patch) | |
| tree | b4fdb1879a5e5cc8ceefd5a219a01142ea6baecc | |
| parent | d953f6927cfa605d0bc336b09a25bde2a483b3ba (diff) | |
Use before_actions
| -rw-r--r-- | app/controllers/groups/application_controller.rb | 6 | ||||
| -rw-r--r-- | app/controllers/groups/group_members_controller.rb | 5 |
2 files changed, 7 insertions, 4 deletions
diff --git a/app/controllers/groups/application_controller.rb b/app/controllers/groups/application_controller.rb index 4df9d1b7533..6878d4bc07e 100644 --- a/app/controllers/groups/application_controller.rb +++ b/app/controllers/groups/application_controller.rb @@ -18,4 +18,10 @@ class Groups::ApplicationController < ApplicationController return render_404 end end + + def authorize_admin_group_member! + unless can?(current_user, :admin_group_member, group) + return render_403 + end + end end diff --git a/app/controllers/groups/group_members_controller.rb b/app/controllers/groups/group_members_controller.rb index b9c428a964f..91518c44a98 100644 --- a/app/controllers/groups/group_members_controller.rb +++ b/app/controllers/groups/group_members_controller.rb @@ -5,6 +5,7 @@ class Groups::GroupMembersController < Groups::ApplicationController # Authorize before_action :authorize_read_group! before_action :authorize_admin_group!, except: [:index, :leave] + before_action :authorize_admin_group_member!, only: [:create, :resend_invite] def index @project = @group.projects.find(params[:project_id]) if params[:project_id] @@ -21,8 +22,6 @@ class Groups::GroupMembersController < Groups::ApplicationController end def create - return render_403 unless can?(current_user, :admin_group_member, @group) - @group.add_users(params[:user_ids].split(','), params[:access_level], current_user) redirect_to group_group_members_path(@group), notice: 'Users were successfully added.' @@ -51,8 +50,6 @@ class Groups::GroupMembersController < Groups::ApplicationController end def resend_invite - return render_403 unless can?(current_user, :admin_group_member, @group) - redirect_path = group_group_members_path(@group) @group_member = @group.group_members.find(params[:id]) |