diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2019-03-23 14:07:04 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-03-23 14:07:04 +0100 |
commit | 555c4e11baf58401c1bdd915e4ecef679e6ae514 (patch) | |
tree | db7f30a6828f4df45c7dae6c387aef8e25b2eb5d /app/validators | |
parent | 55a9658ad8552804c7c585df12d46c391b84dd94 (diff) |
Add validations to admin settings (#10348)
* Add validations to admin settings - Validate correct HTML markup - Validate presence of contact username & e-mail - Validate that all usernames are valid - Validate that enums have expected values * Fix code style issue * Fix tests
Diffstat (limited to 'app/validators')
-rw-r--r-- | app/validators/existing_username_validator.rb | 20 | ||||
-rw-r--r-- | app/validators/html_validator.rb | 14 |
2 files changed, 34 insertions, 0 deletions
diff --git a/app/validators/existing_username_validator.rb b/app/validators/existing_username_validator.rb new file mode 100644 index 000000000..4388a0c98 --- /dev/null +++ b/app/validators/existing_username_validator.rb @@ -0,0 +1,20 @@ +# frozen_string_literal: true + +class ExistingUsernameValidator < ActiveModel::EachValidator + def validate_each(record, attribute, value) + return if value.blank? + + if options[:multiple] + missing_usernames = value.split(',').map { |username| username unless Account.find_local(username) }.compact + record.errors.add(attribute, I18n.t('existing_username_validator.not_found_multiple', usernames: missing_usernames.join(', '))) if missing_usernames.any? + else + record.errors.add(attribute, I18n.t('existing_username_validator.not_found')) unless Account.find_local(value) + end + end + + private + + def valid_html?(str) + Nokogiri::HTML.fragment(str).to_s == str + end +end diff --git a/app/validators/html_validator.rb b/app/validators/html_validator.rb new file mode 100644 index 000000000..882c35d41 --- /dev/null +++ b/app/validators/html_validator.rb @@ -0,0 +1,14 @@ +# frozen_string_literal: true + +class HtmlValidator < ActiveModel::EachValidator + def validate_each(record, attribute, value) + return if value.blank? + record.errors.add(attribute, I18n.t('html_validator.invalid_markup')) unless valid_html?(value) + end + + private + + def valid_html?(str) + Nokogiri::HTML.fragment(str).to_s == str + end +end |