diff options
| author | Santiago Pastorino <santiago@wyeworks.com> | 2011-03-12 21:10:19 -0200 |
|---|---|---|
| committer | Santiago Pastorino <santiago@wyeworks.com> | 2011-03-12 21:12:44 -0200 |
| commit | 157c37f55854a17fcb50e77b7636fab01773df3c (patch) | |
| tree | 2a09e280afd4ff8a31b95e37c0198ec041a787fc /activemodel/lib/active_model/validations/length.rb | |
| parent | f48d3d4df6c8ff6ab544f5fe384af2a0552984d7 (diff) | |
| download | rails-157c37f55854a17fcb50e77b7636fab01773df3c.tar.gz rails-157c37f55854a17fcb50e77b7636fab01773df3c.tar.bz2 rails-157c37f55854a17fcb50e77b7636fab01773df3c.zip | |
Refactor length validation
Diffstat (limited to 'activemodel/lib/active_model/validations/length.rb')
| -rw-r--r-- | activemodel/lib/active_model/validations/length.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/activemodel/lib/active_model/validations/length.rb b/activemodel/lib/active_model/validations/length.rb index 3e6f6e62e6..72735cfb89 100644 --- a/activemodel/lib/active_model/validations/length.rb +++ b/activemodel/lib/active_model/validations/length.rb @@ -42,9 +42,9 @@ module ActiveModel next unless check_value = options[key] value ||= [] if key == :maximum - - next if value.kind_of?(Fixnum) && value.to_s.size.send(validity_check, check_value) - next if value && value.size.send(validity_check, check_value) + + value_length = value.respond_to?(:length) ? value.length : value.to_s.length + next if value_length.send(validity_check, check_value) errors_options = options.except(*RESERVED_OPTIONS) errors_options[:count] = check_value |