lukechilds/node - node - Gitea: Git with a cup of tea

16095 Commits

131 Branches

446 Tags

325 MiB

Tree: c0c1a4c029

Commit Graph

Author	SHA1	Message	Date
Timothy Gu	021338dc6d	url: export URLSearchParams Fixes: #10761	8 years ago
Michaël Zasso	6977224059	url: fix URL query update if searchParams changes If searchParams becomes empty, query must be set to null. Add missing update of context flags. Fixes: https://github.com/nodejs/node/issues/10480 PR-URL: https://github.com/nodejs/node/pull/10486 Reviewed-By: James M Snell <jasnell@gmail.com>	8 years ago
Timothy Gu	f08d8a6c6f	url: improve URLSearchParams spec compliance - Make URLSearchParams constructor spec-compliant - Strip leading `?` in URL#search's setter - Spec-compliant iterable interface - More precise handling of update steps as mandated by the spec - Add class strings to URLSearchParams objects and their prototype - Make sure `this instanceof URLSearchParams` in methods Also included are relevant tests from W3C's Web Platform Tests (https://github.com/w3c/web-platform-tests/tree/master/url). Fixes: https://github.com/nodejs/node/issues/9302 PR-URL: https://github.com/nodejs/node/pull/9484 Reviewed-By: James M Snell <jasnell@gmail.com>	8 years ago

Author

SHA1

Message

Date

Timothy Gu

021338dc6d

url: export URLSearchParams

Fixes: #10761

8 years ago

Michaël Zasso

6977224059

url: fix URL query update if searchParams changes

If searchParams becomes empty, query must be set to null.
Add missing update of context flags.

Fixes: https://github.com/nodejs/node/issues/10480
PR-URL: https://github.com/nodejs/node/pull/10486
Reviewed-By: James M Snell <jasnell@gmail.com>

8 years ago

Timothy Gu

f08d8a6c6f

url: improve URLSearchParams spec compliance

- Make URLSearchParams constructor spec-compliant
- Strip leading `?` in URL#search's setter
- Spec-compliant iterable interface
- More precise handling of update steps as mandated by the spec
- Add class strings to URLSearchParams objects and their prototype
- Make sure `this instanceof URLSearchParams` in methods

Also included are relevant tests from W3C's Web Platform Tests
(https://github.com/w3c/web-platform-tests/tree/master/url).

Fixes: https://github.com/nodejs/node/issues/9302
PR-URL: https://github.com/nodejs/node/pull/9484
Reviewed-By: James M Snell <jasnell@gmail.com>

8 years ago

3 Commits (c0c1a4c029ebf4c8c067574c7bab1af22767ac7d)