Ticket #9935: json_request_parsing.2.diff

test/controller/request_test.rb

@@ -327 +327 @@
-    assert_equal Mime::TEXT, @request.format
-  end
-
+  def test_json_format
+    @request.instance_eval { @parameters = { :format => 'json' } }
+    assert_equal Mime::JSON, @request.format
+  end
+
-  def test_nil_format
-    @request.instance_eval { @parameters = { :format => nil } }
-    @request.env["HTTP_ACCEPT"] = "text/javascript"
@@ -352 +357 @@
-    assert_equal Mime::XML, @request.content_type
-  end
-  
+  def test_content_type_json
+    @request.env["CONTENT_TYPE"] = "application/json"
+    assert_equal Mime::JSON, @request.content_type
+  end
+
-  def test_user_agent
-    assert_not_nil @request.user_agent
-  end
@@ -782 +792 @@
-      ActionController::CgiRequest.new(cgi).request_parameters
-    end
-end
+
+
+class JsonParamsParsingTest < Test::Unit::TestCase
+  def test_json_hash
+    params = parse_body(<<-JSON)
+      { person: {
+          name: "David",
+          url: "http://rubyonrails.org/",
+          count: 1
+        } }
+    JSON
+
+    assert_equal 'David', params['json']['person']['name']
+    assert_equal 'http://rubyonrails.org/', params['json']['person']['url']
+    assert_equal 1, params[:json][:person][:count]
+  end
+
+  def test_json_fail_unless_hash
+    assert_raise Exception do
+      parse_body(<<-JSON)
+        [ { name: "David" },
+          { name: "Allan" } ]
+      JSON
+    end
+
+    assert_raise Exception do
+      parse_body(%{"value"})
+    end
+  end
+
+  private
+    def parse_body(body)
+      env = { 'CONTENT_TYPE'   => 'application/json',
+              'CONTENT_LENGTH' => body.size.to_s }
+      cgi = ActionController::Integration::Session::MockCGI.new(env, body)
+      ActionController::CgiRequest.new(cgi).request_parameters
+    end
+end

lib/action_controller/request.rb

@@ -373 +373 @@
-            self.class.parse_multipart_form_parameters(body, boundary, content_length, env)
-          when :xml_simple, :xml_node
-            body.blank? ? {} : Hash.from_xml(body).with_indifferent_access
+          when :json
+            json = ActiveSupport::JSON.decode(body)
+            raise ActionControllerError, "Can only accept JSON objects, not arrays" unless Hash === json
+            { :json => json }.with_indifferent_access
-          when :yaml
-            YAML.load(body)
-          else

lib/action_controller/base.rb

@@ -306 +306 @@
-    #   ActionController::Base.param_parsers[Mime::YAML] = :yaml
-    @@param_parsers = { Mime::MULTIPART_FORM => :multipart_form,
-                        Mime::URL_ENCODED_FORM => :url_encoded_form,
-
+
+
-    cattr_accessor :param_parsers
-
-    # Controls the default charset for all renders.