diff options
author | Jon Leighton <j@jonathanleighton.com> | 2011-01-06 19:54:23 +0000 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2011-01-07 15:03:16 -0800 |
commit | 5b28e5254267c581ce6934408aaf458616e44e3f (patch) | |
tree | 2c6e793119db841e22443848fa03e8527dc6c1b4 /activerecord/lib | |
parent | d23c332e02932a6a06853193c79a21d70dbe139e (diff) | |
download | rails-5b28e5254267c581ce6934408aaf458616e44e3f.tar.gz rails-5b28e5254267c581ce6934408aaf458616e44e3f.tar.bz2 rails-5b28e5254267c581ce6934408aaf458616e44e3f.zip |
Don't not remove double negatives
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/associations/has_one_association.rb | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/activerecord/lib/active_record/associations/has_one_association.rb b/activerecord/lib/active_record/associations/has_one_association.rb index 158ae376e1..7503130e8c 100644 --- a/activerecord/lib/active_record/associations/has_one_association.rb +++ b/activerecord/lib/active_record/associations/has_one_association.rb @@ -14,11 +14,11 @@ module ActiveRecord new_record(:build_association, attributes) end - def replace(obj, dont_save = false) + def replace(obj, save = true) load_target unless @target.nil? || @target == obj - if @reflection.options[:dependent] && !dont_save + if @reflection.options[:dependent] && save case @reflection.options[:dependent] when :delete @target.delete if @target.persisted? @@ -45,7 +45,7 @@ module ActiveRecord set_inverse_instance(obj) loaded - unless !@owner.persisted? || obj.nil? || dont_save + unless !@owner.persisted? || obj.nil? || !save return (obj.save ? self : false) else return (obj.nil? ? nil : self) @@ -65,7 +65,7 @@ module ActiveRecord def new_record(method, attributes) record = scoped.scoping { @reflection.send(method, attributes) } - replace(record, true) + replace(record, false) record end end |