Projet

Général

Profil

Bug #75321

sauts manuels comme actions en masse : crash en cas de doublon d'identifiant

Ajouté par Thomas Noël il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Version cible:
-
Début:
10 mars 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

Lié à w.c.s. - Development #65898: Pouvoir sélectionner des statuts particuliers sur un déclencheur d'action globaleFermé23 mars 2023

Actions

Révisions associées

Révision 7e4375e2 (diff)
Ajouté par Benjamin Dauvergne il y a environ un an

misc: remove unused field ManualActin.action_id (#75321)

Révision cb6fe33c (diff)
Ajouté par Benjamin Dauvergne il y a environ un an

misc: make identifier of status manual actions unique (#75321)

Historique

#2

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 »

#4

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.

#6

Mis à jour par Benjamin Dauvergne il y a environ un an

  • Assigné à mis à Benjamin Dauvergne
  • Priorité changé de Normal à Haut
#7

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é
#8

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 :

#10

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de En cours à Solution proposée
#11

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 :

#12

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 :

#13

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#14

Mis à jour par Transition automatique il y a 11 mois

Automatic expiration

Formats disponibles : Atom PDF