about summary refs log tree commit diff
path: root/app/validators
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2018-03-21 10:26:53 +0100
committerGitHub <noreply@github.com>2018-03-21 10:26:53 +0100
commit93897134caf42f1b70620282cef04865af7026b1 (patch)
tree6868a3d7b6eedc44170ee87bb54564cddf5f133a /app/validators
parenta6b59cd1a32ce2d9ac54fa7b5e04672a63692fdf (diff)
Permit dots in usernames with conditions (#6844)
* Permit dots in usernames with conditions

- Dot cannot be the start or end of username
- a.lice and al.ice are considered the same during sign-up

* Fix regex mixin flags
Diffstat (limited to 'app/validators')
-rw-r--r--app/validators/unique_username_validator.rb14
1 files changed, 14 insertions, 0 deletions
diff --git a/app/validators/unique_username_validator.rb b/app/validators/unique_username_validator.rb
new file mode 100644
index 000000000..c76407b16
--- /dev/null
+++ b/app/validators/unique_username_validator.rb
@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+class UniqueUsernameValidator < ActiveModel::Validator
+  def validate(account)
+    return if account.username.nil?
+
+    normalized_username = account.username.downcase.delete('.')
+
+    scope = Account.where(domain: nil, username: normalized_username)
+    scope = scope.where.not(id: account.id) if account.persisted?
+
+    account.errors.add(:username, :taken) if scope.exists?
+  end
+end