Set query parameter for array of primitive types

CHANGELOG.md

@@ -3,6 +3,7 @@
 #### Features
 
 * [#927](https://github.com/ruby-grape/grape-swagger/pull/927): Set default parameter location based on consumes - [@spaceraccoon](https://github.com/spaceraccoon)
+* [#929](https://github.com/ruby-grape/grape-swagger/pull/929): Set query parameter for array of primitive types - [@spaceraccoon](https://github.com/spaceraccoon)
 * Your contribution here.
 
 #### Fixes

lib/grape-swagger/doc_methods/data_type.rb

@@ -75,6 +75,14 @@ def primitives
           PRIMITIVE_MAPPINGS.keys.map(&:downcase)
         end
 
+        def query_array_primitive?(type)
+          query_array_primitives.include?(type.to_s.downcase)
+        end
+
+        def query_array_primitives
+          primitives << 'string'
+        end
+
         def mapping(value)
           PRIMITIVE_MAPPINGS[value] || 'string'
         end

lib/grape-swagger/doc_methods/parse_params.rb

@@ -21,7 +21,7 @@ def call(param, settings, path, route, definitions, consumes) # rubocop:disable
           # optional properties
           document_description(settings)
           document_type_and_format(settings, data_type)
-          document_array_param(value_type, definitions) if value_type[:is_array]
+          document_array_param(value_type, definitions, consumes) if value_type[:is_array]
           document_default_value(settings) unless value_type[:is_array]
           document_range_values(settings) unless value_type[:is_array]
           document_required(settings)
@@ -68,28 +68,47 @@ def document_add_extensions(settings)
           GrapeSwagger::DocMethods::Extensions.add_extensions_to_root(settings, @parsed_param)
         end
 
-        def document_array_param(value_type, definitions)
+        def document_array_param(value_type, definitions, consumes)
           if value_type[:documentation].present?
-            param_type = value_type[:documentation][:param_type] || value_type[:documentation][:in]
             doc_type = value_type[:documentation][:type]
             type = DataType.mapping(doc_type) if doc_type && !DataType.request_primitive?(doc_type)
             collection_format = value_type[:documentation][:collectionFormat]
           end
 
-          param_type ||= value_type[:param_type]
-
           array_items = parse_array_item(
             definitions,
             type,
             value_type
           )
 
-          @parsed_param[:in] = param_type || 'formData'
+          @parsed_param[:in] = array_param_type(value_type, consumes)
           @parsed_param[:items] = array_items
           @parsed_param[:type] = 'array'
           @parsed_param[:collectionFormat] = collection_format if DataType.collections.include?(collection_format)
         end
 
+        def array_param_type(value_type, consumes)
+          if value_type[:documentation].present?
+            param_type = value_type[:documentation][:param_type] || value_type[:documentation][:in]
+          end
+
+          param_type ||= value_type[:param_type]
+
+          if param_type
+            param_type
+          elsif %w[POST PUT PATCH].include?(value_type[:method])
+            if consumes.include?('application/x-www-form-urlencoded') || consumes.include?('multipart/form-data')
+              'formData'
+            else
+              'body'
+            end
+          elsif DataType.query_array_primitive?(value_type[:data_type])
+            'query'
+          else
+            'formData'
+          end
+        end
+
         def parse_array_item(definitions, type, value_type)
           array_items = {}
           if definitions[value_type[:data_type]]

spec/issues/883_query_array_parameter_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe '#883 Group Params as Array' do
+  let(:app) do
+    Class.new(Grape::API) do
+      namespace :issue_883 do
+        params do
+          requires :array_of_string, type: [String]
+          requires :array_of_integer, type: [Integer]
+        end
+        get '/get_primitive_array_parameters' do
+          'accepts array query parameters of primitive value types'
+        end
+
+        params do
+          requires :array_of, type: Array, documentation: { type: 'link', is_array: true }
+        end
+        get '/get_object_array_parameters' do
+          'does not accept array query parameters of object value types'
+        end
+      end
+      add_swagger_documentation
+    end
+  end
+
+  describe 'retrieves the documentation for typed group range parameters' do
+    subject do
+      get '/swagger_doc'
+      JSON.parse(last_response.body)
+    end
+
+    specify do
+      expect(subject['paths']['/issue_883/get_primitive_array_parameters']['get']['parameters']).to eql(
+        [
+          { 'in' => 'query', 'name' => 'array_of_string', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => true },
+          { 'in' => 'query', 'name' => 'array_of_integer', 'type' => 'array', 'items' => { 'type' => 'integer', 'format' => 'int32' }, 'required' => true }
+        ]
+      )
+      expect(subject['paths']['/issue_883/get_object_array_parameters']['get']['parameters']).to eql(
+        [{ 'in' => 'formData', 'items' => { 'type' => 'string' }, 'name' => 'array_of', 'required' => true, 'type' => 'array' }]
+      )
+    end
+  end
+end

spec/swagger_v2/params_array_collection_format_spec.rb

@@ -52,7 +52,7 @@ def app
     specify do
       expect(subject['paths']['/array_of_strings_without_collection_format']['get']['parameters']).to eql(
         [
-          { 'in' => 'formData', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false, 'description' => 'array in csv collection format' }
+          { 'in' => 'query', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false, 'description' => 'array in csv collection format' }
         ]
       )
     end
@@ -67,7 +67,7 @@ def app
     specify do
       expect(subject['paths']['/array_of_strings_multi_collection_format']['get']['parameters']).to eql(
         [
-          { 'in' => 'formData', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false, 'collectionFormat' => 'multi', 'description' => 'array in multi collection format' }
+          { 'in' => 'query', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false, 'collectionFormat' => 'multi', 'description' => 'array in multi collection format' }
         ]
       )
     end
@@ -82,7 +82,7 @@ def app
     specify do
       expect(subject['paths']['/array_of_strings_brackets_collection_format']['get']['parameters']).to eql(
         [
-          { 'in' => 'formData', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false, 'collectionFormat' => 'brackets', 'description' => 'array in brackets collection format' }
+          { 'in' => 'query', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false, 'collectionFormat' => 'brackets', 'description' => 'array in brackets collection format' }
         ]
       )
     end
@@ -97,7 +97,7 @@ def app
     specify do
       expect(subject['paths']['/array_of_strings_invalid_collection_format']['get']['parameters']).to eql(
         [
-          { 'in' => 'formData', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false }
+          { 'in' => 'query', 'name' => 'array_of_strings', 'type' => 'array', 'items' => { 'type' => 'string' }, 'required' => false }
         ]
       )
     end