Permalink
Browse files

Don't tokenize string when counting characters

  • Loading branch information...
iangreenleaf committed Dec 5, 2011
1 parent cf6ccf0 commit 5ff71ac9f844f2c8bcde8cdd5de767c9bdd4cc9e
Showing with 1 addition and 2 deletions.
  1. +1 −2 activemodel/lib/active_model/validations/length.rb
@@ -6,7 +6,6 @@ class LengthValidator < EachValidator
MESSAGES = { :is => :wrong_length, :minimum => :too_short, :maximum => :too_long }.freeze
CHECKS = { :is => :==, :minimum => :>=, :maximum => :<= }.freeze
- DEFAULT_TOKENIZER = lambda { |value| value.split(//) }
RESERVED_OPTIONS = [:minimum, :maximum, :within, :is, :tokenizer, :too_short, :too_long]
def initialize(options)
@@ -36,7 +35,7 @@ def check_validity!
end
def validate_each(record, attribute, value)
- value = (options[:tokenizer] || DEFAULT_TOKENIZER).call(value) if value.kind_of?(String)
+ value = options[:tokenizer].call(value) if value.kind_of?(String) && options[:tokenizer].present?
CHECKS.each do |key, validity_check|
next unless check_value = options[key]

0 comments on commit 5ff71ac

Please sign in to comment.