summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorEugenia Grieff <egrieff@gitlab.com>2019-09-23 14:21:11 +0100
committerEugenia Grieff <egrieff@gitlab.com>2019-10-01 15:01:17 +0100
commit6af071dd08df45776859d9a8b94565a92d238f64 (patch)
tree46b7fc5920365d9d88c13517b7715b610c789096 /app
parent5a54a8d9be626ef1a708f3ffbc80b39209c07351 (diff)
downloadgitlab-ce-6af071dd08df45776859d9a8b94565a92d238f64.tar.gz
Backport for CE MR
https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/3409
Diffstat (limited to 'app')
-rw-r--r--app/finders/labels_finder.rb8
-rw-r--r--app/models/project.rb8
2 files changed, 11 insertions, 5 deletions
diff --git a/app/finders/labels_finder.rb b/app/finders/labels_finder.rb
index e523942ea4c..027cdc4fc78 100644
--- a/app/finders/labels_finder.rb
+++ b/app/finders/labels_finder.rb
@@ -51,7 +51,7 @@ class LabelsFinder < UnionFinder
end
label_ids << Label.where(group_id: projects.group_ids)
- label_ids << Label.where(project_id: projects.select(:id)) unless only_group_labels?
+ label_ids << Label.where(project_id: ids_user_can_read_labels(projects)) unless only_group_labels?
end
label_ids
@@ -188,4 +188,10 @@ class LabelsFinder < UnionFinder
groups.select { |group| authorized_to_read_labels?(group) }
end
end
+
+ # rubocop: disable CodeReuse/ActiveRecord
+ def ids_user_can_read_labels(projects)
+ Project.where(id: projects.select(:id)).ids_with_issuables_available_for(current_user)
+ end
+ # rubocop: enable CodeReuse/ActiveRecord
end
diff --git a/app/models/project.rb b/app/models/project.rb
index 7c065db9829..da67e9b273a 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -586,11 +586,11 @@ class Project < ApplicationRecord
joins(:namespace).where(namespaces: { type: 'Group' }).select(:namespace_id)
end
- # Returns ids of projects with milestones available for given user
+ # Returns ids of projects with issuables available for given user
#
- # Used on queries to find milestones which user can see
- # For example: Milestone.where(project_id: ids_with_milestone_available_for(user))
- def ids_with_milestone_available_for(user)
+ # Used on queries to find milestones or labels which user can see
+ # For example: Milestone.where(project_id: ids_with_issuables_available_for(user))
+ def ids_with_issuables_available_for(user)
with_issues_enabled = with_issues_available_for_user(user).select(:id)
with_merge_requests_enabled = with_merge_requests_available_for_user(user).select(:id)