Title: refactor(http): rename URLSearchParams to something else · Issue #3513 · angular/angular · GitHub
Open Graph Title: refactor(http): rename URLSearchParams to something else · Issue #3513 · angular/angular
X Title: refactor(http): rename URLSearchParams to something else · Issue #3513 · angular/angular
Description: TL;DR: Rename URLSearchParams to QueryParams. I've discussed with @jeffbcross that it would be wise to rename the class, as we are considering having a behaviour which differs from the WHATWG's version of the interface --- naming this di...
Open Graph Description: TL;DR: Rename URLSearchParams to QueryParams. I've discussed with @jeffbcross that it would be wise to rename the class, as we are considering having a behaviour which differs from the WHATWG's ver...
X Description: TL;DR: Rename URLSearchParams to QueryParams. I've discussed with @jeffbcross that it would be wise to rename the class, as we are considering having a behaviour which differs from the WHATWG...
Opengraph URL: https://github.com/angular/angular/issues/3513
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"refactor(http): rename URLSearchParams to something else","articleBody":"TL;DR: Rename `URLSearchParams` to `QueryParams`. \n\n---\n\nI've discussed with @jeffbcross that it would be wise to rename the class, as we are considering having a behaviour which differs from the WHATWG's version of the interface --- naming this differently could help minimize confusion when using the API.\n\nSome ideas:\n\n-`SearchQuery`\n-`SearchParameters`\n-`SearchParams`\n\nAll missing the leading `URL` and maybe standing out from the family of interfaces derived from https://url.spec.whatwg.org.\n\n---\n\nThe behaviours discussed, which some may agree or disagree with, include the following:\n- An explicit `merge` method (which I think would be valuable to propose to the editors of [whatwg/url](https://github.com/whatwg/url). If the WHATWG did in the future accept a proposal like this, it may be possible to deprecate and remove the in-angular URLSearchParams and rely on the native implementation instead, in the future. So that's something!\n\nSemantic issue: Tab Atkins Jr isn't keen on the fact that sometimes, you want to merge by appending to existing values (a MultiMap behaviour) and sometimes you want to replace existing values (a Map behaviour) --- but doesn't believe there's value in providing a way to specify a schema. Having two methods would certainly require being explicit, though.\n\nOR\n- Implicitly merging during RequestOptions.merge(), rather than the WHATWG's strategy of replacing the original query entirely (currently, but possibly not in the future, for all URLs which are associated with the URLSearchParams object). It should be possible to implicitly merge this way even if the WHATWG's version of the object were used, but may not be.\n\nAny thoughts or suggestions are welcome\n","author":{"url":"https://github.com/caitp","@type":"Person","name":"caitp"},"datePublished":"2015-08-06T20:09:41.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":8},"url":"https://github.com/3513/angular/issues/3513"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:b7867a60-0cd7-1c0e-be3c-a2b44d45c612 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | E44A:18C6F1:2AAD73:39EB78:6A5FEF87 |
| html-safe-nonce | 323ff30e87dfff47e1e31fb34cbcc2252c712ab04b69d4ea4377ec8dd5a02454 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNDRBOjE4QzZGMToyQUFENzM6MzlFQjc4OjZBNUZFRjg3IiwidmlzaXRvcl9pZCI6IjYwMDIyODcyNzQwNjMwMzIxOTkiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | afc4f3eb7f588ac8a31f4011e07354d1213929c2c09ceb6af0d5468e4e21d2b3 |
| hovercard-subject-tag | issue:99514272 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/angular/angular/3513/issue_layout |
| twitter:image | https://opengraph.githubassets.com/5d85846fe2323d037aaf0d66497bb796e780c7a65dd149aa8019b9c3b19638c3/angular/angular/issues/3513 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/5d85846fe2323d037aaf0d66497bb796e780c7a65dd149aa8019b9c3b19638c3/angular/angular/issues/3513 |
| og:image:alt | TL;DR: Rename URLSearchParams to QueryParams. I've discussed with @jeffbcross that it would be wise to rename the class, as we are considering having a behaviour which differs from the WHATWG's ver... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | caitp |
| hostname | github.com |
| expected-hostname | github.com |
| None | ba296ecd3a3521238fe43b3b3b08cc09207ddc21173e16ee656db03803de8e2b |
| turbo-cache-control | no-preview |
| go-import | github.com/angular/angular git https://github.com/angular/angular.git |
| octolytics-dimension-user_id | 139426 |
| octolytics-dimension-user_login | angular |
| octolytics-dimension-repository_id | 24195339 |
| octolytics-dimension-repository_nwo | angular/angular |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 24195339 |
| octolytics-dimension-repository_network_root_nwo | angular/angular |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 6e50fb37e29e4aa0df6386cb485e58b0e3ca97ab |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width