Add options flag to include empty values

tj / node-querystring

querystring parser for node and the browser - supporting nesting (used by Express, Connect, etc)

MIT License

455 stars 66 forks source link

Add options flag to include empty values #48

Closed davidmurdoch closed 11 years ago

davidmurdoch commented 11 years ago

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 commented 11 years ago

Fixes issue https://github.com/visionmedia/node-querystring/issues/39: Empty fields removed when parsing array variables

buschtoens commented 11 years ago

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.

daguej commented 10 years ago

@silvinci, it is still an issue.