]> cat aescling's git repositories - mastodon.git/commitdiff
Fix existing username validator not allowing multiple accounts (#16153)
authorEugen Rochko <eugen@zeonfederated.com>
Tue, 4 May 2021 12:22:04 +0000 (14:22 +0200)
committerGitHub <noreply@github.com>
Tue, 4 May 2021 12:22:04 +0000 (14:22 +0200)
Fix #16107

app/validators/existing_username_validator.rb

index afbe0c635cff9e385c3cbb84555b27c7571a0e13..8f7d96b8e94c0ecc9c1e02a028d507a0d8641531 100644 (file)
@@ -19,10 +19,10 @@ class ExistingUsernameValidator < ActiveModel::EachValidator
       str unless Account.find_remote(username, domain)
     end
 
-    if usernames_with_no_accounts.any? && options[:multiple]
-      record.errors.add(attribute, I18n.t('existing_username_validator.not_found_multiple', usernames: usernames_with_no_accounts.join(', ')))
-    elsif usernames_with_no_accounts.any? || usernames_and_domains.size > 1
-      record.errors.add(attribute, I18n.t('existing_username_validator.not_found'))
+    if options[:multiple]
+      record.errors.add(attribute, I18n.t('existing_username_validator.not_found_multiple', usernames: usernames_with_no_accounts.join(', '))) if usernames_with_no_accounts.any?
+    else
+      record.errors.add(attribute, I18n.t('existing_username_validator.not_found')) if usernames_with_no_accounts.any? || usernames_and_domains.size > 1
     end
   end
 end