Revert "Modiy HTTP::Params.[] to fetch all params"

This reverts commit c0745cc9e25068e9ce832ca5a79519de516ca4e5.

spec/param_parser_spec.cr

@@ -2,98 +2,101 @@ require "./spec_helper"
 
 describe "ParamParser" do
   it "parses query params" do
+    route = Route.new "POST", "/" do |env|
+      hasan = env.params.query["hasan"]
+      "Hello #{hasan}"
+    end
     request = HTTP::Request.new("POST", "/?hasan=cemal")
     query_params = Kemal::ParamParser.new(request).query
     query_params["hasan"].should eq "cemal"
   end
 
-  context "#query" do
+  it "parses multiple values for query params" do
-    it "parses multiple values for query params" do
+    route = Route.new "POST", "/" do |env|
-      request = HTTP::Request.new("POST", "/?hasan=cemal&hasan=lamec")
+      hasan = env.params.query["hasan"]
-      query_params = Kemal::ParamParser.new(request).query
+      "Hello #{hasan}"
-      query_params["hasan"].should eq ["cemal", "lamec"]
+    end
+    request = HTTP::Request.new("POST", "/?hasan=cemal&hasan=lamec")
+    query_params = Kemal::ParamParser.new(request).query
+    query_params.fetch_all("hasan").should eq ["cemal", "lamec"]
+  end
+
+  it "parses url params" do
+    kemal = Kemal::RouteHandler::INSTANCE
+    kemal.add_route "POST", "/hello/:hasan" do |env|
+      "hello #{env.params.url["hasan"]}"
+    end
+    request = HTTP::Request.new("POST", "/hello/cemal")
+    # Radix tree MUST be run to parse url params.
+    io_with_context = create_request_and_return_io(kemal, request)
+    url_params = Kemal::ParamParser.new(request).url
+    url_params["hasan"].should eq "cemal"
+  end
+
+  it "decodes url params" do
+    kemal = Kemal::RouteHandler::INSTANCE
+    kemal.add_route "POST", "/hello/:email/:money/:spanish" do |env|
+      email = env.params.url["email"]
+      money = env.params.url["money"]
+      spanish = env.params.url["spanish"]
+      "Hello, #{email}. You have #{money}. The spanish word of the day is #{spanish}."
+    end
+    request = HTTP::Request.new("POST", "/hello/sam%2Bspec%40gmail.com/%2419.99/a%C3%B1o")
+    # Radix tree MUST be run to parse url params.
+    io_with_context = create_request_and_return_io(kemal, request)
+    url_params = Kemal::ParamParser.new(request).url
+    url_params["email"].should eq "sam+spec@gmail.com"
+    url_params["money"].should eq "$19.99"
+    url_params["spanish"].should eq "año"
+  end
+
+  it "parses request body" do
+    route = Route.new "POST", "/" do |env|
+      name = env.params.query["name"]
+      age = env.params.query["age"]
+      hasan = env.params.body["hasan"]
+      "Hello #{name} #{hasan} #{age}"
+    end
+
+    request = HTTP::Request.new(
+      "POST",
+      "/?hasan=cemal",
+      body: "name=serdar&age=99",
+      headers: HTTP::Headers{"Content-Type" => "application/x-www-form-urlencoded"},
+    )
+
+    query_params = Kemal::ParamParser.new(request).query
+    {"hasan" => "cemal"}.each do |k, v|
+      query_params[k].should eq(v)
+    end
+
+    body_params = Kemal::ParamParser.new(request).body
+    {"name" => "serdar", "age" => "99"}.each do |k, v|
+      body_params[k].should eq(v)
     end
   end
 
-  context "#url" do
+  it "parses multiple values in request body" do
-    it "parses url params" do
+    route = Route.new "POST", "/" do |env|
-      kemal = Kemal::RouteHandler::INSTANCE
+      hasan = env.params.body["hasan"]
-      kemal.add_route "POST", "/hello/:hasan" do |env|
+      "Hello #{hasan}"
-        "hello #{env.params.url["hasan"]}"
-      end
-      request = HTTP::Request.new("POST", "/hello/cemal")
-      # Radix tree MUST be run to parse url params.
-      io_with_context = create_request_and_return_io(kemal, request)
-      url_params = Kemal::ParamParser.new(request).url
-      url_params["hasan"].should eq "cemal"
     end
 
-    it "decodes url params" do
+    request = HTTP::Request.new(
-      kemal = Kemal::RouteHandler::INSTANCE
+      "POST",
-      kemal.add_route "POST", "/hello/:email/:money/:spanish" do |env|
+      "/",
-        email = env.params.url["email"]
+      body: "hasan=cemal&hasan=lamec",
-        money = env.params.url["money"]
+      headers: HTTP::Headers{"Content-Type" => "application/x-www-form-urlencoded"},
-        spanish = env.params.url["spanish"]
+    )
-        "Hello, #{email}. You have #{money}. The spanish word of the day is #{spanish}."
+
-      end
+    body_params = Kemal::ParamParser.new(request).body
-      request = HTTP::Request.new("POST", "/hello/sam%2Bspec%40gmail.com/%2419.99/a%C3%B1o")
+    body_params.fetch_all("hasan").should eq(["cemal", "lamec"])
-      # Radix tree MUST be run to parse url params.
-      io_with_context = create_request_and_return_io(kemal, request)
-      url_params = Kemal::ParamParser.new(request).url
-      url_params["email"].should eq "sam+spec@gmail.com"
-      url_params["money"].should eq "$19.99"
-      url_params["spanish"].should eq "año"
-    end
   end
 
-  context "#body" do
+  context "when content type is application/json" do
     it "parses request body" do
-      request = HTTP::Request.new(
+      route = Route.new "POST", "/" { }
-        "POST",
-        "/?hasan=cemal",
-        body: "name=serdar&age=99",
-        headers: HTTP::Headers{"Content-Type" => "application/x-www-form-urlencoded"},
-      )
 
-      query_params = Kemal::ParamParser.new(request).query
-      {"hasan" => "cemal"}.each do |k, v|
-        query_params[k].should eq(v)
-      end
-
-      body_params = Kemal::ParamParser.new(request).body
-      {"name" => "serdar", "age" => "99"}.each do |k, v|
-        body_params[k].should eq(v)
-      end
-    end
-
-    it "parses multiple values in request body" do
-      request = HTTP::Request.new(
-        "POST",
-        "/",
-        body: "hasan=cemal&hasan=lamec",
-        headers: HTTP::Headers{"Content-Type" => "application/x-www-form-urlencoded"},
-      )
-
-      body_params = Kemal::ParamParser.new(request).body
-      body_params["hasan"].should eq(["cemal", "lamec"])
-    end
-
-    it "parses array values in request body" do
-      request = HTTP::Request.new(
-        "POST",
-        "/",
-        body: "framework=kemal&feats[]=fast&feats[]=effective&feats[]=simple",
-        headers: HTTP::Headers{"Content-Type" => "application/x-www-form-urlencoded"},
-      )
-
-      body_params = Kemal::ParamParser.new(request).body
-      body_params["framework"].should eq("kemal")
-      body_params["feats[]"].should eq(["fast", "effective", "simple"])
-    end
-  end
-
-  context "#json" do
-    it "parses request body" do
       request = HTTP::Request.new(
         "POST",
         "/",
@@ -106,6 +109,8 @@ describe "ParamParser" do
     end
 
     it "parses request body when passed charset" do
+      route = Route.new "POST", "/" { }
+
       request = HTTP::Request.new(
         "POST",
         "/",
@@ -118,6 +123,8 @@ describe "ParamParser" do
     end
 
     it "parses request body for array" do
+      route = Route.new "POST", "/" { }
+
       request = HTTP::Request.new(
         "POST",
         "/",
@@ -130,6 +137,8 @@ describe "ParamParser" do
     end
 
     it "parses request body and query params" do
+      route = Route.new "POST", "/" { }
+
       request = HTTP::Request.new(
         "POST",
         "/?foo=bar",
@@ -147,6 +156,8 @@ describe "ParamParser" do
     end
 
     it "handles no request body" do
+      route = Route.new "GET", "/" { }
+
       request = HTTP::Request.new(
         "GET",
         "/",
@@ -169,6 +180,13 @@ describe "ParamParser" do
 
   context "when content type is incorrect" do
     it "does not parse request body" do
+      route = Route.new "POST", "/" do |env|
+        name = env.params.body["name"]
+        age = env.params.body["age"]
+        hasan = env.params.query["hasan"]
+        "Hello #{name} #{hasan} #{age}"
+      end
+
       request = HTTP::Request.new(
         "POST",
         "/?hasan=cemal",

src/kemal/ext/params.cr

@@ -1,8 +0,0 @@
-module HTTP
-  struct Params
-    def [](name)
-      params = raw_params[name]
-      params.size == 1 ? params.first : params
-    end
-  end
-end

src/kemal/ext/request.cr

@@ -1,5 +1,5 @@
 class HTTP::Request
-  property override_method : String?
+  property override_method
   property url_params : Hash(String, String)?
   getter param_parser : Kemal::ParamParser?
 
@@ -21,7 +21,7 @@ class HTTP::Request
       @param_parser = Kemal::ParamParser.new(self)
       params = @param_parser.not_nil!.body
       if params.has_key?("_method") && HTTP::Request.override_method_valid?(params["_method"])
-        @override_method = params["_method"].as(String)
+        @override_method = params["_method"]
       end
     end
     @override_method