Add options flag to include empty values #48

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@davidmurdoch
Contributor

Currently, when parsing a querystring, empty values are not included in
the result. This adds a flag, includeEmptyValues, to the (new)
options object (express/connect already sends its options object
here) allowing for the result to include empty values.

2 new tests added.
README.md updated to include an example.
./querystring.js has been rebuilt and all ./test/browser scripts and
styles have also been updated.

@davidmurdoch davidmurdoch Add options flag to include empty values
Currently, when parsing a querystring, empty values are not included in
the result. This adds a flag, `includeEmptyValues`, to the (new)
`options` object (express/connect already sends its `options` object
here) allowing for the result to include empty values.

2 new tests added.
README.md updated to include an example.
./querystring.js has been rebuilt and all `./test/browser` scripts and
styles have also been updated.
ac1049d
@davidmurdoch
Contributor

Fixes issue #39: Empty fields removed when parsing array variables

@buschtoens
Collaborator

Sadly, this PR is really outdated by now and can't be merged cleanly anymore. But AFAIK issue #39 still remains, though I can't test it right now.

@buschtoens buschtoens closed this Sep 9, 2013
@daguej
daguej commented Nov 7, 2013

@silvinci, it is still an issue.

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