Fix server error when failing to follow back followers from `/relatio…

…nships` (mastodon#23787)
fedibird · Apr 22, 2023 · 465c1b3 · 465c1b3
1 parent 7119444
commit 465c1b3
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/app/controllers/relationships_controller.rb b/app/controllers/relationships_controller.rb
@@ -19,6 +19,8 @@ def update
     @form.save
   rescue ActionController::ParameterMissing
     # Do nothing
+  rescue Mastodon::NotPermittedError, ActiveRecord::RecordNotFound
+    flash[:alert] = I18n.t('relationships.follow_failure') if action_from_button == 'follow'
   ensure
     redirect_to relationships_path(filter_params)
   end

diff --git a/app/models/form/account_batch.rb b/app/models/form/account_batch.rb
@@ -31,9 +31,15 @@ def save
   private
 
   def follow!
-    accounts.find_each do |target_account|
+    error = nil
+
+    accounts.each do |target_account|
       FollowService.new.call(current_account, target_account)
+    rescue Mastodon::NotPermittedError, ActiveRecord::RecordNotFound => e
+      error ||= e
     end
+
+    raise error if error.present?
   end
 
   def unfollow!

diff --git a/config/locales/en.yml b/config/locales/en.yml
@@ -1290,6 +1290,7 @@ en:
   relationships:
     activity: Account activity
     dormant: Dormant
+    follow_failure: Could not follow some of the selected accounts.
     follow_selected_followers: Follow selected followers
     followers: Followers
     following: Following