From c6487fb305304c38b43cd15f39a8e081a5f6f9b2 Mon Sep 17 00:00:00 2001 From: Regis Date: Tue, 8 Aug 2017 14:48:31 -0600 Subject: use pluck to grab all ids instead of mapping --- app/controllers/dashboard/milestones_controller.rb | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/app/controllers/dashboard/milestones_controller.rb b/app/controllers/dashboard/milestones_controller.rb index ca96435b435..78bbc8725d7 100644 --- a/app/controllers/dashboard/milestones_controller.rb +++ b/app/controllers/dashboard/milestones_controller.rb @@ -22,14 +22,9 @@ class Dashboard::MilestonesController < Dashboard::ApplicationController private def milestones - group_milestones = current_user - .groups - .map do |group_id| - group_params = params.merge(group_ids: group_id) - - MilestonesFinder.new(group_params).execute - end - .flatten + group_milestones = MilestonesFinder.new( + group_ids: current_user.groups.pluck(:id) + ).execute project_milestones = DashboardMilestone.build_collection(@projects, params) -- cgit v1.2.1