Do not consider moved accounts as non-functional

This might be somewhat controversial, but putting a redirection up
makes the account unusable, which works against use cases such as
using the moved account as backup in case the new one temporarily fails
(sure, the redirection can be temporarily removed, but it is a hassle)

diff --git a/app/models/user.rb b/app/models/user.rb
index e33610d549260eb3ab4e97f47860f35bc1b2c76c..49cfc25ca2e47873e34f44305fcfceff950c9f8f 100644 (file)
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -169,7 +169,7 @@ class User < ApplicationRecord
   end
 
   def functional?
-    confirmed? && approved? && !disabled? && !account.suspended? && account.moved_to_account_id.nil?
+    confirmed? && approved? && !disabled? && !account.suspended?
   end
 
   def unconfirmed_or_pending?