Fix codestyle in `ScreenNameValidator`

This commit is contained in:
Andreas Nedbal 2022-01-22 21:32:45 +01:00 committed by Andreas Nedbal
parent 77c6dc7120
commit 1933aaac7f
1 changed files with 8 additions and 10 deletions

View File

@ -1,16 +1,14 @@
# frozen_string_literal: true
class ScreenNameValidator < ActiveModel::EachValidator class ScreenNameValidator < ActiveModel::EachValidator
FORBIDDEN_SCREEN_NAMES = %w(justask_admin retrospring_admin admin justask retrospring about public FORBIDDEN_SCREEN_NAMES = %w[justask_admin retrospring_admin admin justask retrospring about public
notifications inbox sign_in sign_up sidekiq moderation moderator mod administrator notifications inbox sign_in sign_up sidekiq moderation moderator mod administrator
siteadmin site_admin help retro_spring retroospring retrosprlng niisding nllsding siteadmin site_admin help retro_spring retroospring retrosprlng niisding nllsding
pixeidesu plxeldesu plxeidesu terms privacy linkfilter feedback) pixeidesu plxeldesu plxeidesu terms privacy linkfilter feedback].freeze
FORBIDDEN_SCREEN_NAME_REGEXPS = [/wreciap\z/i] FORBIDDEN_SCREEN_NAME_REGEXPS = [/wreciap\z/i].freeze
def validate_each(record, attribute, value) def validate_each(record, attribute, value)
if FORBIDDEN_SCREEN_NAMES.include? value.downcase record.errors[attribute] << "Thou shalt not use this username! Please choose another one." if FORBIDDEN_SCREEN_NAMES.include? value.downcase
record.errors[attribute] << "Thou shalt not use this username! Please choose another one." record.errors[attribute] << "Registration is tempoarily disabled for new users." if FORBIDDEN_SCREEN_NAME_REGEXPS.any? { |regexp| value.downcase =~ regexp }
end
if FORBIDDEN_SCREEN_NAME_REGEXPS.any? { |regexp| value.downcase =~ regexp }
record.errors[attribute] << "Registration is tempoarily disabled for new users."
end
end end
end end