diff options
author | Ahmad Sherif <me@ahmadsherif.com> | 2016-09-21 00:05:49 +0300 |
---|---|---|
committer | Ahmad Sherif <me@ahmadsherif.com> | 2016-09-23 14:19:16 +0300 |
commit | b8bfe50a5080909c7ba2b517d99d3c5f79d2a32f (patch) | |
tree | e1954df2cc9fb02e397ea8684dbe897fe5926343 | |
parent | ac3470280dec5c9af43d9fa0da2ca47295c28d9a (diff) |
Reduce number of queries when calling GlobalMilestone#{labels,participants}
-rw-r--r-- | CHANGELOG | 1 | ||||
-rw-r--r-- | app/models/global_milestone.rb | 14 |
2 files changed, 13 insertions, 2 deletions
diff --git a/CHANGELOG b/CHANGELOG index 5fbc8830d7b..90092c3008e 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -180,6 +180,7 @@ v 8.12.0 (unreleased) - Clean environment variables when running git hooks - Fix Import/Export issues importing protected branches and some specific models - Fix non-master branch readme display in tree view + - Speed-up group milestones show page - Add UX improvements for merge request version diffs v 8.11.7 diff --git a/app/models/global_milestone.rb b/app/models/global_milestone.rb index 81b7343d7ad..8f38a2b6254 100644 --- a/app/models/global_milestone.rb +++ b/app/models/global_milestone.rb @@ -61,11 +61,11 @@ class GlobalMilestone end def participants - @participants ||= milestones.map(&:participants).flatten.compact.uniq + @participants ||= milestones_relation.includes(:participants).map(&:participants).flatten.compact.uniq end def labels - @labels ||= GlobalLabel.build_collection(milestones.map(&:labels).flatten) + @labels ||= GlobalLabel.build_collection(milestones_relation.includes(:labels).map(&:labels).flatten) .sort_by!(&:title) end @@ -89,4 +89,14 @@ class GlobalMilestone end end end + + private + + def milestones_relation + @milestones_relation ||= if milestones.is_a?(ActiveRecord::Relation) + milestones + else + Milestone.where(id: milestones.map(&:id)) + end + end end |