From ac4bf1180aa0f82616038522bddaf3ff3d5020c8 Mon Sep 17 00:00:00 2001
From: Joshua Peek <josh@joshpeek.com>
Date: Fri, 9 Jan 2009 13:12:39 -0600
Subject: [PATCH] Ensure we override Rack::Request's POST method too

---
 actionpack/lib/action_controller/request.rb             | 5 ++++-
 actionpack/test/controller/query_string_parsing_test.rb | 4 +++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/actionpack/lib/action_controller/request.rb b/actionpack/lib/action_controller/request.rb
index 8371f1bf5c..b4ab1ccda1 100755
--- a/actionpack/lib/action_controller/request.rb
+++ b/actionpack/lib/action_controller/request.rb
@@ -387,6 +387,7 @@ def raw_post
     def parameters
       @parameters ||= request_parameters.merge(query_parameters).update(path_parameters).with_indifferent_access
     end
+    alias_method :params, :parameters
 
     def path_parameters=(parameters) #:nodoc:
       @env["rack.routing_args"] = parameters
@@ -418,9 +419,11 @@ def GET
     end
     alias_method :query_parameters, :GET
 
-    def request_parameters
+    # Override Rack's POST method to support nested query strings
+    def POST
       @parser.request_parameters
     end
+    alias_method :request_parameters, :POST
 
     def body_stream #:nodoc:
       @env['rack.input']
diff --git a/actionpack/test/controller/query_string_parsing_test.rb b/actionpack/test/controller/query_string_parsing_test.rb
index 91f5b2b27a..a31e326ddf 100644
--- a/actionpack/test/controller/query_string_parsing_test.rb
+++ b/actionpack/test/controller/query_string_parsing_test.rb
@@ -1,4 +1,6 @@
-class QueryStringParsingTest <ActionController::IntegrationTest
+require 'abstract_unit'
+
+class QueryStringParsingTest < ActionController::IntegrationTest
   class TestController < ActionController::Base
     class << self
       attr_accessor :last_query_parameters
-- 
GitLab