Validator fails if number is passed instead of object

[marvinthepa]

undefined method `empty?' for 1:Fixnum: grape-1.0.2/lib/grape/validations/validators/base.rb:41:in `block in validate!'

This happens for something like this:

        params do
          optional :foos, :type => Array do
            optional :bar, :type => String
          end
        end

and then POSTing something like this:

{ "foos": [1] }

Setting requires :bar (instead of optional) fixes this.

While it is arguable that allowing empty objects does not make too much sense, who knows what users might want?

The problem is here:

grape/lib/grape/validations/validators/base.rb

Line 41 in 2dcef9a

next if !@scope.required? && resource_params.empty?

It probably makes sense to check if resource_params is actually a hash in this case.

[dblock]

Looks like a legit bug, please PR a fix.

[marvinthepa]

I will try. Might take me some time to get around it, though.

[dblock]

Closed via #1947 (comment)