diff options
author | José Valim <jose.valim@gmail.com> | 2009-12-22 23:12:21 +0100 |
---|---|---|
committer | José Valim <jose.valim@gmail.com> | 2009-12-22 23:12:21 +0100 |
commit | 2476c5312dcbd29f49672f71617a3d34c6a60cc7 (patch) | |
tree | 26394dcc6a26a845269c9a004b3960b01fbe693a /activemodel/lib/active_model | |
parent | 4b8330d2d50ae4de14dd43ffbea4d91804553140 (diff) | |
download | rails-2476c5312dcbd29f49672f71617a3d34c6a60cc7.tar.gz rails-2476c5312dcbd29f49672f71617a3d34c6a60cc7.tar.bz2 rails-2476c5312dcbd29f49672f71617a3d34c6a60cc7.zip |
Validator is simply sent to validate method. However, the API needs to change, so validate accepts a record.
Diffstat (limited to 'activemodel/lib/active_model')
-rw-r--r-- | activemodel/lib/active_model/validations/with.rb | 8 | ||||
-rw-r--r-- | activemodel/lib/active_model/validator.rb | 7 |
2 files changed, 6 insertions, 9 deletions
diff --git a/activemodel/lib/active_model/validations/with.rb b/activemodel/lib/active_model/validations/with.rb index edc2133ddc..8313aded11 100644 --- a/activemodel/lib/active_model/validations/with.rb +++ b/activemodel/lib/active_model/validations/with.rb @@ -49,12 +49,10 @@ module ActiveModel # end # def validates_with(*args) - configuration = args.extract_options! + options = args.extract_options! - validate configuration do |record| - args.each do |klass| - klass.new(record, configuration.except(:on, :if, :unless)).validate - end + args.each do |klass| + validate klass.new(options.except(:on, :if, :unless)), options end end end diff --git a/activemodel/lib/active_model/validator.rb b/activemodel/lib/active_model/validator.rb index 09de72b757..80a538fcb6 100644 --- a/activemodel/lib/active_model/validator.rb +++ b/activemodel/lib/active_model/validator.rb @@ -54,14 +54,13 @@ module ActiveModel #:nodoc: # end # class Validator - attr_reader :record, :options + attr_reader :options - def initialize(record, options) - @record = record + def initialize(options) @options = options end - def validate + def validate(record) raise "You must override this method" end end |