Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Changeset 7362

Show
Ignore:
Timestamp:
08/28/07 23:15:47 (1 year ago)
Author:
nzkoz
Message:

Remove unnecessary loop in validates_presence_of. [sur] Closes #9392

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/activerecord/lib/active_record/validations.rb

    r7361 r7362  
    491491        # can't use validates_each here, because it cannot cope with nonexistent attributes, 
    492492        # while errors.add_on_empty can 
    493         attr_names.each do |attr_name| 
    494           send(validation_method(configuration[:on])) do |record| 
    495             unless (configuration[:if] && !evaluate_condition(configuration[:if], record)) || (configuration[:unless] && evaluate_condition(configuration[:unless], record)) 
    496               record.errors.add_on_blank(attr_name,configuration[:message]) 
    497             end 
    498           end 
    499         end 
     493        send(validation_method(configuration[:on])) do |record| 
     494          unless (configuration[:if] && !evaluate_condition(configuration[:if], record)) || (configuration[:unless] && evaluate_condition(configuration[:unless], record)) 
     495            record.errors.add_on_blank(attr_names, configuration[:message]) 
     496          end 
     497        end 
    500498      end 
    501499