Skip to content
This repository

Add options flag to include empty values #48

Closed
wants to merge 1 commit into from
+6,485 5,046

3 participants

David Murdoch Jan Buschtöns Josh Dague
David Murdoch

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.

David Murdoch 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
David Murdoch

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

Jan Buschtöns
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.

Josh Dague

@silvinci, it is still an issue.

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

Showing 1 unique commit by 1 author.

Dec 07, 2012
David Murdoch 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
Something went wrong with that request. Please try again.