Handle json array

Author: mohammednasser-32

lib/grape/dsl/parameters.rb

@@ -251,7 +251,7 @@ def map_params(params, element, is_array = false)
       # @return hash of parameters relevant for the current scope
       # @api private
       def params(params)
-        params = @parent.params(params) if instance_variable_defined?(:@parent) && @parent
+        params = @parent.params_meeting_dependency.presence || @parent.params(params) if instance_variable_defined?(:@parent) && @parent
         params = map_params(params, @element) if instance_variable_defined?(:@element) && @element
         params
       end

lib/grape/validations/params_scope.rb

@@ -4,7 +4,7 @@ module Grape
   module Validations
     class ParamsScope
       attr_accessor :element, :parent, :index
-      attr_reader :type
+      attr_reader :type, :params_meeting_dependency
 
       include Grape::DSL::Parameters
 
@@ -67,6 +67,7 @@ def initialize(opts, &block)
         @type             = opts[:type]
         @group            = opts[:group]
         @dependent_on     = opts[:dependent_on]
+        @params_meeting_dependency = []
         @declared_params = []
         @index = nil
 
@@ -94,7 +95,11 @@ def should_validate?(parameters)
       def meets_dependency?(params, request_params)
         return true unless @dependent_on
         return false if @parent.present? && !@parent.meets_dependency?(@parent.params(request_params), request_params)
-        return params.any? { |param| meets_dependency?(param, request_params) } if params.is_a?(Array)
+
+        if params.is_a?(Array)
+          @params_meeting_dependency = params.filter { |param| meets_dependency?(param, request_params) }
+          return @params_meeting_dependency.present?
+        end
 
         meets_hash_dependency?(params)
       end

spec/grape/dsl/parameters_spec.rb

@@ -259,6 +259,7 @@ def extract_message_option(attrs)
       parent_params = { foo: 'bar' }
       subject.parent = Object.new
       allow(subject.parent).to receive(:params).and_return(parent_params)
+      allow(subject.parent).to receive(:params_meeting_dependency).and_return(nil)
       expect(subject.params({})).to eq parent_params
     end
 

spec/grape/validations/params_scope_spec.rb

@@ -630,6 +630,59 @@ def initialize(value)
       expect(last_response.body).to eq 'inner3[0][baz][0][baz_category] is missing'
     end
 
+    context 'detect when json array' do
+      before do
+        subject.params do
+          requires :array, type: Array do
+            requires :a, type: String
+            given a: ->(val) { val == 'a' } do
+              requires :json, type: Hash do
+                requires :b, type: String
+              end
+            end
+          end
+        end
+
+        subject.post '/nested_array' do
+          'nested array works!'
+        end
+      end
+
+      it 'succeeds' do
+        params = {
+          array: [
+            {
+              a: 'a',
+              json: { b: 'b' }
+            },
+            {
+              a: 'b'
+            }
+          ]
+        }
+        post '/nested_array', params.to_json, 'CONTENT_TYPE' => 'application/json'
+
+        expect(last_response.status).to eq(201)
+      end
+
+      it 'fails' do
+        params = {
+          array: [
+            {
+              a: 'a',
+              json: { b: 'b' }
+            },
+            {
+              a: 'a'
+            }
+          ]
+        }
+        post '/nested_array', params.to_json, 'CONTENT_TYPE' => 'application/json'
+
+        expect(last_response.status).to eq(400)
+      end
+    end
+
     it 'includes the parameter within #declared(params)' do
       get '/test', a: true, b: true