]> cat aescling's git repositories - mastodon.git/commitdiff
Do not re-query mentions from serializers (#6858)
authorEugen Rochko <eugen@zeonfederated.com>
Thu, 22 Mar 2018 10:31:17 +0000 (11:31 +0100)
committerGitHub <noreply@github.com>
Thu, 22 Mar 2018 10:31:17 +0000 (11:31 +0100)
Fix performance regression from #6836

app/serializers/activitypub/note_serializer.rb
app/serializers/rest/status_serializer.rb

index abaf29047ad390d3805a3025e343bdaee9bca5ff..ddafb540d0a2d5c0fbbae97e7d5201448f67a8ae 100644 (file)
@@ -57,7 +57,7 @@ class ActivityPub::NoteSerializer < ActiveModel::Serializer
   end
 
   def virtual_tags
-    object.mentions.order(:id) + object.tags + object.emojis
+    object.mentions.to_a.sort_by(&:id) + object.tags + object.emojis
   end
 
   def atom_uri
index 67da92cd5ce40846faeee1c55a21e05e8269851d..fe3dc9bfcfb2838d9bd1a75437f13d2cbe57e374 100644 (file)
@@ -87,7 +87,7 @@ class REST::StatusSerializer < ActiveModel::Serializer
   end
 
   def ordered_mentions
-    object.mentions.order(:id)
+    object.mentions.to_a.sort_by(&:id)
   end
 
   class ApplicationSerializer < ActiveModel::Serializer