]> cat aescling's git repositories - mastodon.git/commitdiff
Fix N+1 on AtomSerializer (#7669)
authorabcang <abcang1015@gmail.com>
Tue, 29 May 2018 11:34:02 +0000 (20:34 +0900)
committerEugen Rochko <eugen@zeonfederated.com>
Tue, 29 May 2018 11:34:02 +0000 (13:34 +0200)
app/lib/ostatus/atom_serializer.rb

index 698f2ee229ea26956b7ad469b7d99904e7e6cc40..5c6ff4f9b9fd04b5d4bf42e30b79a3de3ea9d013 100644 (file)
@@ -354,7 +354,7 @@ class OStatus::AtomSerializer
     append_element(entry, 'summary', status.spoiler_text, 'xml:lang': status.language) if status.spoiler_text?
     append_element(entry, 'content', Formatter.instance.format(status).to_str, type: 'html', 'xml:lang': status.language)
 
-    status.mentions.order(:id).each do |mentioned|
+    status.mentions.sort_by(&:id).each do |mentioned|
       append_element(entry, 'link', nil, rel: :mentioned, 'ostatus:object-type': OStatus::TagManager::TYPES[:person], href: OStatus::TagManager.instance.uri_for(mentioned.account))
     end