0
@@ -1,6 +1,8 @@
0
+*1.14.1* (April 6th, 2005)
0
+
0
 * Fix type_name_with_module to handle type names that begin with '::'. Closes #4614. [Nicholas Seckar]
0
 
0
-*1.14.1* (April 6th, 2005)
0
+* Fixed that that multiparameter assignment doesn't work with aggregations (closes #4620) [Lars Pind]
0
 
0
 * Enable Limit/Offset in Calculations (closes #4558) [lmarlow@yahoo.com]
0
 

0
@@ -1988,7 +1988,7 @@ module ActiveRecord #:nodoc:
0
       def execute_callstack_for_multiparameter_attributes(callstack)
0
         errors = []
0
         callstack.each do |name, values|
0
-          klass = (self.class.reflect_on_aggregation(name) || column_for_attribute(name)).klass
0
+          klass = (self.class.reflect_on_aggregation(name.to_sym) || column_for_attribute(name)).klass
0
           if values.empty?
0
             send(name + "=", nil)
0
           else

0
@@ -2,6 +2,7 @@ require 'abstract_unit'
0
 require 'fixtures/topic'
0
 require 'fixtures/reply'
0
 require 'fixtures/company'
0
+require 'fixtures/customer'
0
 require 'fixtures/developer'
0
 require 'fixtures/project'
0
 require 'fixtures/default'
0
@@ -721,6 +722,14 @@ class BasicsTest < Test::Unit::TestCase
0
     task.attributes = attributes
0
     assert_equal time, task.starting
0
   end
0
+  
0
+  def test_multiparameter_assignment_of_aggregation
0
+    customer = Customer.new
0
+    address = Address.new("The Street", "The City", "The Country")
0
+    attributes = { "address(1)" => address.street, "address(2)" => address.city, "address(3)" => address.country }
0
+    customer.attributes = attributes
0
+    assert_equal address, customer.address
0
+  end
0
 
0
   def test_attributes_on_dummy_time
0
     # Oracle and SQL Server do not have a TIME datatype.

0
@@ -14,6 +14,10 @@ class Address
0
   def close_to?(other_address)
0
     city == other_address.city && country == other_address.country
0
   end
0
+
0
+  def ==(other)
0
+    other.is_a?(self.class) && other.street == street && other.city == city && other.country == country
0
+  end  
0
 end
0
 
0
 class Money