Development #85206
remplacer CommaSeparatedStringField par StringOrListField ?
Statut:
Nouveau
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
02 janvier 2024
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
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 a deux inconvénients :CommaSeparatedStringField devrait être réservé aux listes passées en paramètre d'URL
- 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.