about summary refs log tree commit diff
path: root/app/validators/existing_username_validator.rb
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2022-10-21 14:07:02 +0200
committerGitHub <noreply@github.com>2022-10-21 14:07:02 +0200
commitabf6c87ee8b57e09dca5f5b1fe1839a314e1aa46 (patch)
treeb36275028768d23ba1f1513e54fe826363ad6149 /app/validators/existing_username_validator.rb
parentc60f9cb86568f65d8e743b41c482163ef1ceaf99 (diff)
Fix remote account in contact account setting not being used (#19351)
Diffstat (limited to 'app/validators/existing_username_validator.rb')
-rw-r--r--app/validators/existing_username_validator.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/app/validators/existing_username_validator.rb b/app/validators/existing_username_validator.rb
index 8f7d96b8e..1c5596821 100644
--- a/app/validators/existing_username_validator.rb
+++ b/app/validators/existing_username_validator.rb
@@ -6,7 +6,7 @@ class ExistingUsernameValidator < ActiveModel::EachValidator
 
     usernames_and_domains = begin
       value.split(',').map do |str|
-        username, domain = str.strip.gsub(/\A@/, '').split('@')
+        username, domain = str.strip.gsub(/\A@/, '').split('@', 2)
         domain = nil if TagManager.instance.local_domain?(domain)
 
         next if username.blank?
@@ -21,8 +21,8 @@ class ExistingUsernameValidator < ActiveModel::EachValidator
 
     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
+    elsif usernames_with_no_accounts.any? || usernames_and_domains.size > 1
+      record.errors.add(attribute, I18n.t('existing_username_validator.not_found'))
     end
   end
 end