diff options
author | Phil Hughes <me@iamphill.com> | 2016-09-06 17:11:58 +0100 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2016-09-13 08:44:59 +0100 |
commit | 401b797671b9b67ef40c4afa75acdeca83b6a6de (patch) | |
tree | c08a022114532b5391f88c4b22e9d8104e6e5297 /app | |
parent | 2b41db9215f322ba61113a7bef2f49da157bbd53 (diff) | |
download | gitlab-ce-401b797671b9b67ef40c4afa75acdeca83b6a6de.tar.gz |
Fixed bug when group_ids not present when creating
Diffstat (limited to 'app')
-rw-r--r-- | app/controllers/projects/project_members_controller.rb | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/app/controllers/projects/project_members_controller.rb b/app/controllers/projects/project_members_controller.rb index ac83377148a..d49598d2786 100644 --- a/app/controllers/projects/project_members_controller.rb +++ b/app/controllers/projects/project_members_controller.rb @@ -55,15 +55,17 @@ class Projects::ProjectMembersController < Projects::ApplicationController current_user: current_user ) - group_ids = params[:group_ids].split(',') - groups = Group.where(id: group_ids) - - groups.each do |group| - project.project_group_links.create( - group: group, - group_access: params[:access_level], - expires_at: params[:expires_at] - ) + if params[:group_ids].present? + group_ids = params[:group_ids].split(',') + groups = Group.where(id: group_ids) + + groups.each do |group| + project.project_group_links.create( + group: group, + group_access: params[:access_level], + expires_at: params[:expires_at] + ) + end end redirect_to namespace_project_project_members_path(@project.namespace, @project) |