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

Already on GitHub? Sign in to your account

Incorrect handling of optional named operator when all values are null #10

Closed
Paul-Martin opened this Issue Mar 14, 2013 · 1 comment

Comments

Projects
None yet
2 participants

UriTemplate.parse('/foo{?a,b,c}').expand({});
produces
"/foo?c="
should be
"/foo"

same for operators '&' and ';'

It seems to me that 'if (isFirstVarspec)' block starting at line 668 is unnecessary and removing it will resolve this issue without unwanted side effects.
.

Owner

fxa commented Mar 14, 2013

Of course, you are right.
But I had some side effects, when removing this block.
I will add your test case and fix the bug

@fxa fxa closed this in 819db32 Mar 17, 2013

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