Merge pull request #2109 from tlconnor/fixArrayParams

Fix bug with handling array params.

Author: dblock

lib/grape/dsl/inside_route.rb

@@ -75,8 +75,7 @@ def declared_hash(passed_params, options, declared_params, params_nested_path)
 
               params_nested_path_dup = params_nested_path.dup
               params_nested_path_dup << declared_param.to_s
-
-              memo[memo_key] = handle_passed_param(params_nested_path_dup) do
+              memo[memo_key] = passed_param || handle_passed_param(params_nested_path_dup) do
                 passed_param
               end
             end

spec/grape/endpoint/declared_spec.rb

@@ -135,6 +135,20 @@ def app
       expect(JSON.parse(last_response.body)['nested'].keys.size).to eq 9
     end
 
+    it 'builds arrays correctly' do
+      subject.params do
+        requires :first
+        optional :second, type: Array
+      end
+      subject.post('/declared') { declared(params) }
+
+      post '/declared', first: 'present', second: ['present']
+      expect(last_response.status).to eq(201)
+
+      body = JSON.parse(last_response.body)
+      expect(body['second']).to eq(['present'])
+    end
+
     it 'builds nested params when given array' do
       subject.get '/dummy' do
       end