about summary refs log tree commit diff
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
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
-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)