about summary refs log tree commit diff
path: root/app/models/notification.rb
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/models/notification.rb
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/models/notification.rb')
-rw-r--r--app/models/notification.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/notification.rb b/app/models/notification.rb
index b6db37d6d..5e9ea62a0 100644
--- a/app/models/notification.rb
+++ b/app/models/notification.rb
@@ -96,7 +96,7 @@ class Notification < ApplicationRecord
 
       return if account_ids.empty?
 
-      accounts = Account.where(id: account_ids).includes(:account_stat).each_with_object({}) { |a, h| h[a.id] = a }
+      accounts = Account.where(id: account_ids).includes(:account_stat).index_by(&:id)
 
       cached_items.each do |item|
         item.from_account = accounts[item.from_account_id]