Multiple query parameters with same key aren't handled properly

[bradvogel]

Specify this route in your code:

Router.route('/testroute', function() {
  console.log(this.params.query)
}, {
  where: 'server'
});

Now load: localhost:3000/testroute?param=1&param=2

Expected result in console:

{
   "param": ["1", "2"]
}

Actual result in console:

{
   "param": "2"
}

[dburles]

Hey @bradvogel this is expected behavior, you need to pass the query params like param[]=1&param[]=2 for them to be treated as an array

[bradvogel]

That isn't how the browser works though. The following form:

    <form action="/" method="get">
        <input name="param">
        <input name="param">
        <input type="submit">
    </form>

will submit to the url: /?param=one&param=two

[dburles]

Just rename the field names to param[]. IR will then know to treat them as an array. Not sure if there's any documentation on that now I think about it.

[bradvogel]

Yes, I see that's a workaround, but this is still a bug. The router should support arbitrarily named multiple parameters, as the spec allows.

[dburles]

I wouldn't say it's a bug. The router/whatever can choose to handle them however they wish and the convention of automatically coercing field names with square brackets into arrays is quite common and it makes handling things much less complicated. I know that Rails (edit: not sure about Django) and PHP do the same thing.

[dburles]

Here's the issue @tmeasday opened with some reasoning behind the choice iron-meteor/iron-url#1

[bradvogel]

Ok, thanks for the help. I filed iron-meteor/iron-url#8 as a feature request. Closing this. Thanks!