EconomistDigitalSolutions / ramlapi

Automatic API generation for Go projects
MIT License
83 stars 5 forks source link

Problems with multiple query parameters #9

Closed sjn88 closed 8 years ago

sjn88 commented 8 years ago

When more than one query parameter is in a raml file, the library panics, causing the host application to fail unexpectedly.

buddhamagnet commented 8 years ago

Thanks @Seanseviltwin I think the panic was caused by a missig or empty pattern value in the RAML file, as it takes a string pointer. Looking at what we have now, I can see that we need to add support for multiple query parameters, as it only processes one right now so you'll end up only having support for the last one it finds as far as I can see. I'll try and make some time to add this, driven by tests.

burrbd commented 8 years ago

This issue should be resolved by PR https://github.com/EconomistDigitalSolutions/ramlapi/pull/12