Projet

Général

Profil

Development #85206

remplacer CommaSeparatedStringField par StringOrListField ?

Ajouté par Valentin Deniaud il y a 4 mois. Mis à jour il y a 4 mois.

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 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.

Formats disponibles : Atom PDF