ljharb/qs

Stringify with `comma` format should add square brackets to parameter name to identity array with single value

Open

#315 aperta il 4 giu 2019

Vedi su GitHub
 (17 commenti) (5 reazioni) (0 assegnatari)JavaScript (744 fork)batch import
enhancementhelp wantedstringify

Metriche repository

Star
 (8015 star)
Metriche merge PR
 (Merge medio 6g 22h) (2 PR mergiate in 30 g)

Descrizione

Hi. With new comma array format, stringify of array with single value will produce following result:

const options = {
    arrayFormat: 'comma'
};
qs.stringify({foo: [1]}, options); // foo=1
qs.stringify({foo: [1, 2]}, options); // foo=1,2

So, on parse stage, parsing will produce single value.

qs.parse('foo=1); // {foo: 1}
qs.parse('foo=1,2'); // {foo: [1, 2]}

Maybe add square brackets to parameter name, when comma array format is used?

qs.stringify({foo: [1]}, options); // foo[]=1
qs.stringify({foo: [1, 2]}, options); // foo[]=1,2

qs.parse('foo[]=1); // {foo: [1]}
qs.parse('foo[]=1,2'); // {foo: [1, 2]}

Currently, parsing of the following query string foo[]=1,2 working as expected

Guida contributor

Direzione di ricerca
Esamina la gestione attuale degli array con singolo elemento nel formato virgola e modifica la logica di stringify per aggiungere parentesi quadre solo quando l'array ha un singolo elemento. Assicurati che la logica di parse interpreti correttamente la notazione con parentesi.
Tech stack
javascript
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaScriptTesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60