Skip to content

Commit

Permalink
Merge pull request #7253 from acapilleri/lenght_validator_errors_options
Browse files Browse the repository at this point in the history
calculate errors_options one time in validate_each
  • Loading branch information
rafaelfranca committed Aug 3, 2012
2 parents c8a613b + 93b6faf commit 9895f6c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions activemodel/lib/active_model/validations/length.rb
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,12 @@ def check_validity!
def validate_each(record, attribute, value)
value = tokenize(value)
value_length = value.respond_to?(:length) ? value.length : value.to_s.length

errors_options = options.except(*RESERVED_OPTIONS)

CHECKS.each do |key, validity_check|
next unless check_value = options[key]
next if value_length.send(validity_check, check_value)

errors_options = options.except(*RESERVED_OPTIONS)
errors_options[:count] = check_value

default_message = options[MESSAGES[key]]
Expand Down

0 comments on commit 9895f6c

Please sign in to comment.