about summary refs log tree commit diff
path: root/app/serializers
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2017-12-14 21:35:30 +0100
committerGitHub <noreply@github.com>2017-12-14 21:35:30 +0100
commita3b2ea599df58b1d9944acba8b59f9a465f5adbd (patch)
tree7312c54e6723dda246501160738e88d6e82c6b4d /app/serializers
parent573414f728b5406e2f402f3c21e8ffa38ffc6d8e (diff)
Fix #6022 - Prevent nested migrated accounts, or migrations to self (#6026)
Diffstat (limited to 'app/serializers')
-rw-r--r--app/serializers/rest/account_serializer.rb8
1 files changed, 5 insertions, 3 deletions
diff --git a/app/serializers/rest/account_serializer.rb b/app/serializers/rest/account_serializer.rb
index bab944c5a..19b746520 100644
--- a/app/serializers/rest/account_serializer.rb
+++ b/app/serializers/rest/account_serializer.rb
@@ -7,9 +7,7 @@ class REST::AccountSerializer < ActiveModel::Serializer
              :note, :url, :avatar, :avatar_static, :header, :header_static,
              :followers_count, :following_count, :statuses_count
 
-  has_one :moved_to_account, key: :moved, serializer: REST::AccountSerializer, if: :moved?
-
-  delegate :moved?, to: :object
+  has_one :moved_to_account, key: :moved, serializer: REST::AccountSerializer, if: :moved_and_not_nested?
 
   def id
     object.id.to_s
@@ -38,4 +36,8 @@ class REST::AccountSerializer < ActiveModel::Serializer
   def header_static
     full_asset_url(object.header_static_url)
   end
+
+  def moved_and_not_nested?
+    object.moved? && object.moved_to_account.moved_to_account_id.nil?
+  end
 end