wrap-nested-params doesn't support repeated parameters about ring HOT 3 CLOSED

I believe the URL syntax should be http://someurl?param[]=value1&param[]=value2.

from ring.

@asmala You're sorta correct. The syntax for PHP does require the [] suffix. However, according to this StackOverflow post, the suffix is just a PHP-ism.

The way I understand it:
If there's a GET request at http://someurl?argument=original, then the result of calling (.getParameterValues req "argument") would be ["original"] and the result of calling (.getParameterValue req "argument") would be "original".
If there's a GET request at http://someurl?argument=shadowing&argument=original, then the result of calling (.getParameterValues req "argument") would be ["shadowing" "original"], but the result of (.getParameterValue req "argument") would be "shadowing".

@weavejester, do you have an opinion on how to handle this?

from ring.

From looking at the current tests, it seems like Ring supports the PHP-style nested parameters. So I guess the main question is whether it should support both.

...time goes by in which I get distracted trying to implement this...

OH okay, if there is a problem, it doesn't (solely?) reside in ring.middleware.nested-params. I discovered this when writing a test for (array-map "foo" "bar" "foo" "baz"); if wrap-nested-params receives the parameters already as a map, then the fact that one input "key" had two values has been lost.

...goes off to explore the Ring codebase again...

from ring.