Development #34807
Field, WorkflowStatus, WorkflowStatusItem: implémenter __repr__
Début:
12 juillet 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Pour débugger ça peut être pratique d'avoir le label et l'id.
Fichiers
Révisions associées
tests: add checks for repr methods (#34807)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0001-add-__repr__-to-Field-WorkflowStatus-and-WorkflowSta.patch 0001-add-__repr__-to-Field-WorkflowStatus-and-WorkflowSta.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
J'en avais un petit peu plus alors je l'ai mis quand même.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Sujet changé de Field: implémenter __repr__ à Field, WorkflowStatus, WorkflowStatusItem: implémenter __repr__
Mis à jour par Frédéric Péters il y a presque 5 ans
Je mettrais l'id avant le texte libre.
Le label de Field à est None au début, ça va faire sauter le repr, ce qui est pire que tout.
Sur WorkflowStatusItem, je n'incluerais pas description, qui n'apporte pas d'info particulière.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0001-add-__repr__-to-Field-WorkflowStatus-and-WorkflowSta.patch 0001-add-__repr__-to-Field-WorkflowStatus-and-WorkflowSta.patch ajouté
Remarques prises en compte.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Poussé, ainsi que des tests basiques,
commit 75601f9cecbe3c87cd7272c260145b6a40a21625
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Jul 14 12:41:28 2019 +0200
tests: add checks for __repr__ methods (#34807)
commit 4af6edad53efae56a015ebfdefdd4db23efa6b13
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Jul 12 19:36:40 2019 +0200
misc: add __repr__ to Field, WorkflowStatus and WorkflowStatusItem (#34807)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: add repr to Field, WorkflowStatus and WorkflowStatusItem (#34807)