Changeset 7940

Timestamp:

10/16/07 06:56:07 (11 months ago)

Author:

bitsweat

Message:

Fix regression where the association would not construct new finder SQL on save causing bogus queries for "WHERE owner_id = NULL" even after owner was saved. References #8713.

Files:

• branches/1-2-stable/activerecord/CHANGELOG (modified) (1 diff)
• branches/1-2-stable/activerecord/lib/active_record/associations.rb (modified) (1 diff)
• branches/1-2-stable/activerecord/test/associations_test.rb (modified) (2 diffs)

branches/1-2-stable/activerecord/CHANGELOG

@@ +1 @@
+*SVN*
+
+* Fix regression where the association would not construct new finder SQL on save causing bogus queries for "WHERE owner_id = NULL" even after owner was saved.  #8713 [Bryan Helmkamp]
+
+
-*1.15.5* (October 12th, 2007)
-

branches/1-2-stable/activerecord/lib/active_record/associations.rb

@@ -1003 +1003 @@
-            end
-
-if !
-  records_to_save.each { |record| association.send(:insert_record, record) }
-  association.send(:construct_sql)   
-end
+records_to_save.each { |record| association.send(:insert_record, record) } unless 
+
+
+association.send(:construct_sql) if association.respond_to?(:construct_sql)
-          end_eval
-

branches/1-2-stable/activerecord/test/associations_test.rb

@@ -11 +11 @@
-require 'fixtures/post'
-require 'fixtures/author'
+require 'fixtures/person'
+require 'fixtures/reader'
-
-
@@ -21 +23 @@
-      Class.new(ActiveRecord::Base).has_many(:wheels, :name => 'wheels')
-    end
+  end
+  
+  def test_should_construct_new_finder_sql_after_create
+    person = Person.new
+    assert_equal [], person.readers.find(:all)
+    person.save!
+    reader = Reader.create! :person => person, :post => Post.new(:title => "foo", :body => "bar")
+    assert_equal [reader], person.readers.find(:all)
-  end
-