Bug #75321
sauts manuels comme actions en masse : crash en cas de doublon d'identifiant
0%
Description
Si deux sauts manuels ont le même identifiant, alors on a un crash 500 sur le listing des demandes :
Exception: type = '<class 'ValueError'>', value = 'form already has 'button-action-st-12-relance' widget' Stack trace (most recent call first): File "/usr/lib/python3/dist-packages/quixote/form/form.py", line 230, in add 228 def add(self, widget_class, name, *args, **kwargs): 229 if name in self._names: > 230 raise ValueError("form already has '%s' widget" % name) 231 # add 'id' attribute if not already present 232 if 'id' not in kwargs: ...
Il faudrait éviter de crasher, peut-être juste afficher uniquement le premier saut manuel (ou ajouter l'id technique de l'action et pas juste son identifiant)
Demandes liées
Révisions associées
misc: make identifier of status manual actions unique (#75321)
Historique
Mis à jour par Thomas Noël il y a environ un an
Note : ça vient de #65898 où get_status_manual_actions ne vérifie pas l'unicité des « choice.identifier »
Mis à jour par Thomas Noël il y a environ un an
Note : pour l'instant le contournement c'est de (faire) modifier les identifiants dans le workflow.
Mis à jour par Benjamin Dauvergne il y a environ un an
- Assigné à mis à Benjamin Dauvergne
- Priorité changé de Normal à Haut
Mis à jour par Lauréline Guérin il y a environ un an
- Lié à Development #65898: Pouvoir sélectionner des statuts particuliers sur un déclencheur d'action globale ajouté
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Nouveau à En cours
Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/167
- Titre : WIP: crash lorsque des actions manuelles de statut sont collectées pour application massive
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/167/files
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à Résolu (à déployer)
Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/167
- Titre : crash lorsque des actions manuelles de statut sont collectées pour application massive
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/167/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: remove unused field ManualActin.action_id (#75321)