diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2018-03-21 10:26:53 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-03-21 10:26:53 +0100 |
commit | 93897134caf42f1b70620282cef04865af7026b1 (patch) | |
tree | 6868a3d7b6eedc44170ee87bb54564cddf5f133a /app/validators | |
parent | a6b59cd1a32ce2d9ac54fa7b5e04672a63692fdf (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.rb | 14 |
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 |