diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2019-01-05 07:16:46 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-05 07:16:46 +0100 |
commit | 45899cfa691b1e4f43da98c456ae8faa584eb437 (patch) | |
tree | b3bb6355c4cbe8f74b96d463ac5905c5c898cdfa /app/validators/note_length_validator.rb | |
parent | 90398b9d007d10716cb891d341f64018bb60c822 (diff) |
Change note length validation to ignore mention domains and URLs (#9717)
Fix #4419
Diffstat (limited to 'app/validators/note_length_validator.rb')
-rw-r--r-- | app/validators/note_length_validator.rb | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/app/validators/note_length_validator.rb b/app/validators/note_length_validator.rb new file mode 100644 index 000000000..5ff6df6df --- /dev/null +++ b/app/validators/note_length_validator.rb @@ -0,0 +1,22 @@ +# frozen_string_literal: true + +class NoteLengthValidator < ActiveModel::EachValidator + def validate_each(record, attribute, value) + record.errors.add(attribute, I18n.t('statuses.over_character_limit', max: options[:maximum])) if too_long?(value) + end + + private + + def too_long?(value) + countable_text(value).mb_chars.grapheme_length > options[:maximum] + end + + def countable_text(value) + return '' if value.nil? + + value.dup.tap do |new_text| + new_text.gsub!(FetchLinkCardService::URL_PATTERN, 'x' * 23) + new_text.gsub!(Account::MENTION_RE, '@\2') + end + end +end |