about summary refs log tree commit diff
path: root/app/services
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-10-06 21:33:33 +0200
committerEugen Rochko <eugen@zeonfederated.com>2016-10-06 21:33:33 +0200
commitd772db43441ebc28655b76f3662f02fa7fd48839 (patch)
tree1537747e9340dc0f13ea070f1a621e29c63a0973 /app/services
parent3554d638b31aa5ace2aa524db0165932c15d9a71 (diff)
Fix #83 - if user cannot be found (or is self) throw error, don't return empty 200
This prevents the undefined profile from being opened in the first place on such an error
Diffstat (limited to 'app/services')
-rw-r--r--app/services/follow_service.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/app/services/follow_service.rb b/app/services/follow_service.rb
index 4caf55078..2a9dc82c3 100644
--- a/app/services/follow_service.rb
+++ b/app/services/follow_service.rb
@@ -5,7 +5,7 @@ class FollowService < BaseService
   def call(source_account, uri)
     target_account = follow_remote_account_service.call(uri)
 
-    return nil if target_account.nil? || target_account.id == source_account.id
+    raise ActiveRecord::RecordNotFound if target_account.nil? || target_account.id == source_account.id
 
     follow = source_account.follow!(target_account)