lib/reform/form/active_model.rb in reform-0.2.1 vs lib/reform/form/active_model.rb in reform-0.2.2

@@ -22,17 +22,24 @@
     # Modify the incoming Rails params hash to be representable compliant.
     def validate(params)
       # DISCUSS: #validate should actually expect the complete params hash and then pick the right key as it knows the form name.
       # however, this would cause confusion?
       mapper.new(self).nested_forms do |attr, model| # FIXME: make this simpler.
-        if attr.options[:form_collection] # FIXME: why no array?
-          params[attr.name] = params["#{attr.name}_attributes"].values
-        else
-          params[attr.name] = params["#{attr.name}_attributes"]# DISCUSS: delete old key? override existing?
-        end
+        rename_nested_param_for!(params, attr)
       end
 
       super
+    end
+
+  private
+    def rename_nested_param_for!(params, attr)
+      nested_name = "#{attr.name}_attributes"
+      return unless params.has_key?(nested_name)
+
+      value = params["#{attr.name}_attributes"]
+      value = value.values if attr.options[:form_collection]
+
+      params[attr.name] = value
     end
   end
 
 
   def self.included(base)