Skip to content
Snippets Groups Projects
Commit 25ea69eb authored by Sean McGivern's avatar Sean McGivern
Browse files

Merge branch 'followup-7899' into 'master'

Use a single query in Projects::ProjectMembersController to fetch members

See merge request !7997

Former-commit-id: b546fd7b
parents 97223c75 057de5e1
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -35,13 +35,12 @@ class Projects::ProjectMembersController < Projects::ApplicationController
@group_links = @project.project_group_links.where(group_id: @project.invited_groups.search(params[:search]).select(:id))
end
 
member_ids = @project_members.pluck(:id)
wheres = ["id IN (#{@project_members.select(:id).to_sql})"]
wheres << "id IN (#{group_members.select(:id).to_sql})" if group_members
 
if group_members
member_ids += group_members.pluck(:id)
end
@project_members = Member.where(id: member_ids).order(access_level: :desc).page(params[:page])
@project_members = Member.
where(wheres.join(' OR ')).
order(access_level: :desc).page(params[:page])
 
@requesters = AccessRequestsFinder.new(@project).execute(current_user)
 
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment