about summary refs log tree commit diff
path: root/app/controllers/api/v1
diff options
context:
space:
mode:
authorThibG <thib@sitedethib.com>2021-01-12 09:27:38 +0100
committerGitHub <noreply@github.com>2021-01-12 09:27:38 +0100
commit54d4e5252be8cd9fa780df46e06541da8263d368 (patch)
tree36ea8ee8f190ed388b7eb5f789ff1d305230b0fd /app/controllers/api/v1
parent7bed25f3eabecc84268a4b6c254cc3e0738926b6 (diff)
Use Rails' index_by where it makes sense (#15542)
* Use Rails' index_by where it makes sense

* Fix tests

Co-authored-by: Claire <claire.github-309c@sitedethib.com>
Diffstat (limited to 'app/controllers/api/v1')
-rw-r--r--app/controllers/api/v1/markers_controller.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/app/controllers/api/v1/markers_controller.rb b/app/controllers/api/v1/markers_controller.rb
index 28c2ec791..867e6facf 100644
--- a/app/controllers/api/v1/markers_controller.rb
+++ b/app/controllers/api/v1/markers_controller.rb
@@ -7,7 +7,7 @@ class Api::V1::MarkersController < Api::BaseController
   before_action :require_user!
 
   def index
-    @markers = current_user.markers.where(timeline: Array(params[:timeline])).each_with_object({}) { |marker, h| h[marker.timeline] = marker }
+    @markers = current_user.markers.where(timeline: Array(params[:timeline])).index_by(&:timeline)
     render json: serialize_map(@markers)
   end