Commit 4e9aa7e2 authored by Robert Schilling's avatar Robert Schilling

Correctly search for labels in parent groups

parent 22d2a480
......@@ -57,7 +57,7 @@ module API
delete ':id/labels' do
authorize! :admin_label, user_group
label = find_label(user_group, params[:name])
label = find_label(user_group, params[:name], false)
destroy_conditionally!(label)
end
......@@ -76,7 +76,7 @@ module API
put ':id/labels' do
authorize! :admin_label, user_group
label = find_label(user_group, params[:name])
label = find_label(user_group, params[:name], false)
label = ::Labels::UpdateService.new(declared_params(include_missing: false)).execute(label)
render_validation_error!(label) unless label.valid?
......
......@@ -84,8 +84,8 @@ module API
page || not_found!('Wiki Page')
end
def available_labels_for(label_parent, search_params = {})
search_params.merge(include_ancestor_groups: true)
def available_labels_for(label_parent, include_ancestor_groups = true)
search_params = { include_ancestor_groups: include_ancestor_groups }
if label_parent.is_a?(Project)
search_params[:project_id] = label_parent.id
......@@ -170,8 +170,8 @@ module API
end
end
def find_label(parent, id)
labels = available_labels_for(parent)
def find_label(parent, id, include_ancestor_groups = true)
labels = available_labels_for(parent, include_ancestor_groups)
label = labels.find_by_id(id) || labels.find_by_title(id)
label || not_found!('Label')
......
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