Fix nested parameter hash parsing bug. #10797 [thomas.lee]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@9010 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

actionpack/CHANGELOG

@@ -1,5 +1,7 @@
 *SVN*
 
+* Fix nested parameter hash parsing bug.  #10797 [thomas.lee]
+
 * Allow using named routes in ActionController::TestCase before any request has been made. Closes #11273 [alloy]
 
 * Fixed that sweepers defined by cache_sweeper will be added regardless of the perform_caching setting. Instead, control whether the sweeper should be run with the perform_caching setting. This makes testing easier when you want to turn perform_caching on/off [DHH]

actionpack/lib/action_controller/request.rb

@@ -674,6 +674,7 @@ def bind(key, value)
             else
               top << {key => value}.with_indifferent_access
               push top.last
+              value = top[key]
             end
           else
             top << value

actionpack/test/controller/request_test.rb

@@ -705,6 +705,12 @@ def test_parse_params_with_nil_key
     expected = { "test2" => "value1" }
     assert_equal expected, ActionController::AbstractRequest.parse_request_parameters(input)
   end
+
+  def test_parse_params_with_array_prefix_and_hashes
+    input = { "a[][b][c]" => %w(d) }
+    expected = {"a" => [{"b" => {"c" => "d"}}]}
+    assert_equal expected, ActionController::AbstractRequest.parse_request_parameters(input)
+  end
 end