ljharb/qs

Cannot parse array of objects using arrayFormat = 'brackets'

Open

#215 ouverte le 13 juin 2017

Voir sur GitHub
 (13 commentaires) (11 réactions) (0 assignés)JavaScript (8 015 stars) (744 forks)batch import
bughelp wantedparse

Description

The README states:

You can also create arrays of objects:

var arraysOfObjects = qs.parse('a[][b]=c');
assert.deepEqual(arraysOfObjects, { a: [{ b: 'c' }] });

However when you try to parse an array with 2 objects, an object with 2 keys is created instead:

var arraysOfObjects = qs.parse('a[][b]=c&a[][d]=e');
assert.deepEqual(arraysOfObjects, { a: [{ b: 'c' , d: 'e'}] });

This also happens with arrayFormat = 'repeat'.

If arrays without indices cannot be supported, may the allowDots option could affect arrays instead: (a.0.b=c&a.1.d=e)

Guide contributeur

Stack technique
javascriptnodejs
Domaine
backend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScriptBasic knowledge of query string formats
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55
Direction de recherche
The issue requires modifying the parsing logic in the qs library, likely in 'lib/parse.js', to correctly handle arrays of objects when using 'brackets' or 'repeat' arrayFormat. The current behavior combines multiple objects into one; it should create separate objects. Consider the proposed use of allowDots as an alternative. Review the issue comments for additional context and any suggested fixes.