diff options
author | David Heinemeier Hansson <david@loudthinking.com> | 2005-09-20 18:36:14 +0000 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2005-09-20 18:36:14 +0000 |
commit | 66ecf31ffe751d2eeec124b081e80a38d2d47d87 (patch) | |
tree | 9fccd3fc7a9fee4af33d60f630a833fe6d38be0d | |
parent | 5213a1f73332995ae98d2a163454dff2b288e650 (diff) | |
download | rails-66ecf31ffe751d2eeec124b081e80a38d2d47d87.tar.gz rails-66ecf31ffe751d2eeec124b081e80a38d2d47d87.tar.bz2 rails-66ecf31ffe751d2eeec124b081e80a38d2d47d87.zip |
Fixed that the create_x method from belongs_to wouldn't save the association properly #2042 [Florian Weber]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@2279 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
-rw-r--r-- | activerecord/CHANGELOG | 2 | ||||
-rw-r--r-- | activerecord/lib/active_record/associations/belongs_to_association.rb | 4 | ||||
-rwxr-xr-x | activerecord/test/associations_test.rb | 3 |
3 files changed, 7 insertions, 2 deletions
diff --git a/activerecord/CHANGELOG b/activerecord/CHANGELOG index 6e69f3cd6a..5b8657fa89 100644 --- a/activerecord/CHANGELOG +++ b/activerecord/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Fixed that the create_x method from belongs_to wouldn't save the association properly #2042 [Florian Weber] + * Fixed saving a record with two unsaved belongs_to associations pointing to the same object #2023 [Tobias Luetke] * Improved migrations' behavior when the schema_info table is empty. [Nicholas Seckar] diff --git a/activerecord/lib/active_record/associations/belongs_to_association.rb b/activerecord/lib/active_record/associations/belongs_to_association.rb index fd36c2d8ae..681bd1b88d 100644 --- a/activerecord/lib/active_record/associations/belongs_to_association.rb +++ b/activerecord/lib/active_record/associations/belongs_to_association.rb @@ -13,8 +13,8 @@ module ActiveRecord end def create(attributes = {}) - record = build(attributes) - record.save + record = @association_class.create(attributes) + replace(record, true) record end diff --git a/activerecord/test/associations_test.rb b/activerecord/test/associations_test.rb index 763b66da9b..2bbccb5c9e 100755 --- a/activerecord/test/associations_test.rb +++ b/activerecord/test/associations_test.rb @@ -664,6 +664,9 @@ class BelongsToAssociationsTest < Test::Unit::TestCase citibank = Account.create("credit_limit" => 10) apple = citibank.create_firm("name" => "Apple") assert_equal apple, citibank.firm + citibank.save + citibank.reload + assert_equal apple, citibank.firm end def test_building_the_belonging_object |