When parameters where in the path and in a validation, they were twice in the params hash (once with symbol key, once with string key)

Author: tim-vandecasteele

lib/grape/validations.rb

@@ -137,8 +137,8 @@ def document_attribute(names, opts)
           @last_description[:params] ||= {}
 
           Array(names).each do |name|
-            @last_description[:params][name.to_sym] ||= {}
-            @last_description[:params][name.to_sym].merge!(opts)
+            @last_description[:params][name.to_s] ||= {}
+            @last_description[:params][name.to_s].merge!(opts)
           end
         end
       end

spec/grape/api_spec.rb

@@ -938,7 +938,7 @@ def three
       subject.routes.map { |route|
         { :description => route.route_description, :params => route.route_params }
       }.should eq [
-        { :description => "method", :params => { :ns_param => { :required => true, :desc => "namespace parameter" }, :method_param => { :required => false, :desc => "method parameter" } } }
+        { :description => "method", :params => { "ns_param" => { :required => true, :desc => "namespace parameter" }, "method_param" => { :required => false, :desc => "method parameter" } } }
       ]
     end
     it "should merge the parameters of nested namespaces" do
@@ -964,7 +964,7 @@ def three
       subject.routes.map { |route|
         { :description => route.route_description, :params => route.route_params }
       }.should eq [
-        { :description => "method", :params => { :ns_param => { :required => true, :desc => "ns param 2" }, :ns1_param => { :required => true, :desc => "ns1 param" }, :ns2_param => { :required => true, :desc => "ns2 param" }, :method_param => { :required => false, :desc => "method param" } } }
+        { :description => "method", :params => { "ns_param" => { :required => true, :desc => "ns param 2" }, "ns1_param" => { :required => true, :desc => "ns1 param" }, "ns2_param" => { :required => true, :desc => "ns2 param" }, "method_param" => { :required => false, :desc => "method param" } } }
       ]
     end
     it "should not symbolize params" do