Commit 3778a136 authored by Jasper Maes's avatar Jasper Maes

Rails5 fix passing Group objects array into for_projects_and_groups milestone scope

parent 4791f771
...@@ -123,9 +123,9 @@ class Projects::MilestonesController < Projects::ApplicationController ...@@ -123,9 +123,9 @@ class Projects::MilestonesController < Projects::ApplicationController
def search_params def search_params
if request.format.json? && @project.group && can?(current_user, :read_group, @project.group) if request.format.json? && @project.group && can?(current_user, :read_group, @project.group)
groups = @project.group.self_and_ancestors groups = @project.group.self_and_ancestors_ids
end end
params.permit(:state).merge(project_ids: @project.id, group_ids: groups&.select(:id)) params.permit(:state).merge(project_ids: @project.id, group_ids: groups)
end end
end end
---
title: Rails5 fix passing Group objects array into for_projects_and_groups milestone
scope
merge_request: 19863
author: Jasper Maes
type: fixed
...@@ -65,7 +65,7 @@ module Banzai ...@@ -65,7 +65,7 @@ module Banzai
# We don't support IID lookups for group milestones, because IIDs can # We don't support IID lookups for group milestones, because IIDs can
# clash between group and project milestones. # clash between group and project milestones.
if project.group && !params[:iid] if project.group && !params[:iid]
finder_params[:group_ids] = project.group.self_and_ancestors.select(:id) finder_params[:group_ids] = project.group.self_and_ancestors_ids
end end
MilestonesFinder.new(finder_params).find_by(params) MilestonesFinder.new(finder_params).find_by(params)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment