Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add multiple Query String Param, same Key #68

Closed
ruudvh opened this issue Oct 20, 2014 · 3 comments
Closed

Add multiple Query String Param, same Key #68

ruudvh opened this issue Oct 20, 2014 · 3 comments
Milestone

Comments

@ruudvh
Copy link

ruudvh commented Oct 20, 2014

Hi,

I am trying to add multiple QueryStringParam through AddQueryStringParam, this all works great unless I start adding params with the same Key. When adding multiple params with the same Key it only adds the latest one. I assume this is because AddQueryStringParam uses a dictionary and only unique Keys are allowed?

I have no problem editing the code but I was wondering if somebody reported this already or if you are adding it to 4.0?

Eg. Assume I send a GET request where I want to filter and only pull three fields, the current server I use accepts the following:

'Specify unique item
Request.AddQuerystringParam "Number", "123"
Request.AddQuerystringParam "Name", "Test Name"

'Specify which columns to pull
Request.AddQuerystringParam "Field", "Number"
Request.AddQuerystringParam "Field", "Name"
Request.AddQuerystringParam "Field", "OriginalAmount"

`Output should be something like:
`url/products?Number=123&Name=Test+Name&Field=Number&Field=Name&Field=OriginalAmount

`Output is:
`url/products?Number=123&Name=Test+Name&Field=OriginalAmount

BTW, again great REST script - very excited for 4.0.
Thanks!

@timhall
Copy link
Member

timhall commented Oct 20, 2014

Hi @ruudvh, hadn't thought of needing duplicate keys in querystring params so I'll change that to a Collection underneath (like Headers). I'll add it to the v4.0 tasks

@timhall timhall mentioned this issue Oct 20, 2014
13 tasks
@timhall timhall added this to the 4.0 milestone Oct 20, 2014
@ruudvh
Copy link
Author

ruudvh commented Oct 20, 2014

Thanks @timhall, that is what I was thinking as well.

@timhall
Copy link
Member

timhall commented Dec 17, 2014

Merged as part of v4.0.0 RC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants