ljharb/qs

Array of objects not parsed correctly

Open

#252 geöffnet am 1. März 2018

Auf GitHub ansehen
 (14 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)JavaScript (8.015 Stars) (744 Forks)batch import
help wantedparsequestionstringify

Beschreibung

When parsing the URL ?param[][id]=5a53336d48ad1c6cba0e6ae8&param[][id]=5a53336d48ad1c6cba0e6ae7 I get

param: [{ id: ['5a53336d48ad1c6cba0e6ae8', '5a53336d48ad1c6cba0e6ae7'] }]

instead of

param: [ { id: '5a53336d48ad1c6cba0e6ae8' }, { id: '5a53336d48ad1c6cba0e6ae7' } ]

I am parsing as qs.parse(queryString, { arrayLimit: 50, ignoreQueryPrefix: true }).

Am I missing something?

Contributor Guide

Tech Stack
javascript
Domain
api
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic JavaScriptunderstanding of URL query stringsfamiliarity with qs library
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55
Research-Richtung
Examine the `qs.parse` function in `lib/parse.js` to understand how it handles array notation with empty brackets. Check the option handling for `arrayLimit` and `ignoreQueryPrefix`. Review the 14 comments for any maintainer input or proposed solutions. Consider if a new option or fix is needed to parse `param[][id]` as an array of objects with single keys.