Fix record lookup when id is given as string

Author: nhasselmeyer

lib/active_type/nested_attributes/association.rb

@@ -97,7 +97,7 @@ def derive_class_name
       end
 
       def fetch_child(parent, id)
-        assigned = assigned_children(parent).detect { |r| r.id == id }
+        assigned = assigned_children(parent).detect { |r| r.id.to_s == id.to_s }
         return assigned if assigned
 
         if child = find_scope(parent).find_by_id(id)

spec/active_type/nested_attributes_spec.rb

@@ -235,6 +235,19 @@ def should_assign_and_persist(assign, persist = assign)
         should_assign_and_persist(["existing 1", "updated", "existing 3"])
       end
 
+      it 'allows the id to be given as a string' do
+        subject.records = [
+          NestedAttributesSpec::Record.new(:persisted_string => "existing 1"),
+          NestedAttributesSpec::Record.new(:persisted_string => "existing 2"),
+        ]
+        subject.records[0].id = 100
+        subject.records[1].id = 101
+
+        subject.records_attributes = { '1' => { 'id' => '101', 'persisted_string' => 'updated' } }
+
+        should_assign_and_persist(["existing 1", "updated"])
+      end
+
       it 'does not update records matching a reject_if proc' do
         extra_options.merge!(:reject_if => :bad)
         subject.records = [