sindresorhus/query-string

Lack of symmetry between parse and stringify

Offen

#295 geöffnet am 10.11.2020

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (459 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (6.538 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

stringify(parse('?%20&')) yields only &.

Bit of an edge-case I admit, but do we want to be losing this information?

parse('?%20&') yields {"":null," ":null}.

Contributor Guide

Research-Richtung
Das Problem zeigt, dass `stringify(parse('?%20&'))` `&` zurückgibt, anstatt den kodierten Leerraum beizubehalten. Zur Behebung muss die Kodierungslogik in der `stringify` Funktion (wahrscheinlich in `index.js`) untersucht werden. Die `parse` Funktion gibt korrekt `{"":null," ":null}` zurück, daher tritt der Verlust während der Stringifizierung auf. Prüfen Sie, ob die Bibliothek absichtlich leere Werte verwirft oder ob ein Fehler bei der Behandlung von Schlüsseln mit Leerzeichen vorliegt. Ein Kommentar eines Maintainers (falls vorhanden) könnte das erwartete Verhalten leiten.
Tech Stack
javascript
Domain
full stack
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50