Project

General

Profile

Development #85206

remplacer CommaSeparatedStringField par StringOrListField ?

Added by Valentin Deniaud 4 months ago. Updated 4 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
02 January 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Ça concernerait aussi chrono mais j'ouvre ici parce que l'usage est plus réduit et que ça m'a embêté pour un debug.

J'ai fait un peu d'archéologie et je suis d'accord avec mon moi du passé, #60351-2 :

CommaSeparatedStringField devrait être réservé aux listes passées en paramètre d'URL

CommaSeparatedStringField a deux inconvénients :
  • Quand on a une vraie liste on doit ajouter |join:',' alors que c'est plus joli sans.
  • Quand on oublie cet ajout l'erreur est potentiellement difficile à interpréter.

Typiquement j'ai dû dépatouiller l'erreur

{'err': 1, 'err_class': 'invalid payload', 'err_desc': 'contenu de requête invalide', 'errors': {'slots': ["Agendas inconnus\xa0: 'accueil-periscolaire-alexandre-dumas-2023-09-04-accueil-periscolaire-alexandre-dumas, ['accueil-periscolaire-alexandre-dumas-2023-09-04-accueil-periscolaire-alexandre-dumas"]}

et en lisant trop vite on part sur la fausse piste d'aller vérifier que les slugs sont bons dans les différents endroits.

Also available in: Atom PDF