'optional' Array with nested 'requires' params

[ipkes]

Grape v 0.8.0

Headers:
Content-Type: application/json
Accept: /
Accept-Encoding: gzip,deflate,sdch
Accept-Language: en-US,en;q=0.8,ru;q=0.6

Validation:

params do
  requires :docs do
    requires :type, type: String
    optional :files do 
      requires :href, type: String
    end
  end
end

Package json(with xml same problem):

{
  "docs": [
       {"type": "aaa"}, 
       {"type": "bbb", "files": [{"href": "some_href"}] }
  ]
}

Result:

<error>
  <message>docs[files][href] is missing</message>
</error>

This

{
  "docs": [
      {"type": "aaa"}
  ]
}

and this:

{
  "docs": [
     {"type": "bbb", "files": [{"href": "som_href"}] }
  ]
}

Works fine!
Help plz! What I'm doing wrong? Sorry if subject duplicate.

[dblock]

First, add explicit types to docs and files, type: Hash and type: Array, see if it changes anything. Then can you please try against HEAD? If it's still a problem, try to add to a spec to Grape just like this.

[ipkes]

Explicit types doesn't help.
"Then can you please try against HEAD" - I'm not sure what you mean.. so, I removed grape and then install it again. Doesn't help too.
I will try add spec at the weekend. Thanks!

[dblock]

Against head I mean change the reference to Grape like this in Gemfile:

gem 'grape', github: 'intridea/grape'

Then bundle update grape.

[ipkes]

Updating git://github.com/intridea/grape.git
...
Your bundle is complete!

nothing changed. still:

{"error":"docs[files][href] is missing"}

[taratatach]

I have a related problem. With the following parameters, at least date_string_validator is called if I don't pass incidents in the request.
Using debugger, I can see that incidentDate is marked as required when incidents isn't present.
Everything works fine if I pass an empty array for incidents.

...
optional :incidents, type: Array do
  requires :incidentType, type: String,
    values: incident_types
  requires :incidentDetails, type: String,
    incident_mappings: true
  requires :incidentDate, type: String,
    date_string: true
end
...

[dblock]

@ipkes, @taratatach - looks like bugs without digging deeper, it would be helpful to make pull requests with failing specs added to Grape, thanks

[strobejb]

I was having this same issue in Grape 0.7.0, with a Hash param rather than Array
Seems like it has been fixed in 0.9.0

[dblock]

I want to add a test to match exactly what's above in the reported issue before closing this, @strobejb if you would like to help, that'd be great!

[dblock]

This is definitely fixed, closing. Please reopen if you still see something like this.

[justfalter]

I think this needs to be re-opened. Optional Array and Hash groups do not validate properly when the required parameter is missing. See pull request #883 for specs.

[dblock]

I think you're right @justfalter, reopening.