Development #74132
Interdire dans un même statut deux sauts avec le même identifiant. (ValueError: form already has 'button-action-st-annuler' widget)
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/103534/
ValueError: form already has 'button-action-st-annuler' widget (10 additional frame(s) were not displayed) ... File "quixote/directory.py", line 92, in _q_traverse return obj() File "wcs/backoffice/management.py", line 2169, in _q_index multi_form.add_submit( File "wcs/qommon/form.py", line 287, in add_submit return self.add(SubmitWidget, name, value, **kwargs) File "wcs/qommon/form.py", line 299, in add QuixoteForm.add(self, widget_class, name, *args, **kwargs) File "quixote/form/form.py", line 230, in add raise ValueError("form already has '%s' widget" % name)
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ un an
- Sujet changé de ValueError: form already has 'button-action-st-annuler' widget à Interdire dans un même statut deux sauts avec le même identifiant. (ValueError: form already has 'button-action-st-annuler' widget)
Mis à jour par Thomas Noël il y a environ un an
Frédéric Péters a écrit :
Contourné silencieusement dans #75321.
Je pense qu'au final on ne veut pas interdire dans un même statut deux sauts avec le même identifiants, car il peut s'agir de deux sauts qui s'excluent l'un l'autre au travers de conditions. (cas vu par ailleurs)
Mis à jour par Benjamin Dauvergne il y a environ un an
Thomas Noël a écrit :
Je pense qu'au final on ne veut pas interdire dans un même statut deux sauts avec le même identifiants, car il peut s'agir de deux sauts qui s'excluent l'un l'autre au travers de conditions. (cas vu par ailleurs)
C'était mon analyse aussi, aussi je me disais que les gens ne donnaient pas le même juste par paresse.
Mis à jour par Frédéric Péters il y a environ un an
C'était mon analyse aussi, aussi je me disais que les gens ne donnaient pas le même juste par paresse.
Cette analyse rate le fait que l'identifiant n'est actuellement pas exploité/exploitable par les gens, uniquement utilisé en interne; ça pourrait changer avec #74723 mais mon analyse est que les gens n'ont pas répétés les identifiants en anticipation de ce ticket.
Mis à jour par Emmanuel Cazenave il y a 12 mois
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel Cazenave
Mis à jour par Robot Gitea il y a 12 mois
- Tracker changé de Bug à Development
Emmanuel Cazenave (ecazenave) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/303
- Titre : WIP: workflow: guarantee jump identifiers unicity on new workflows (#74132)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/303/files
Mis à jour par Emmanuel Cazenave il y a 12 mois
- Lié à Development #74723: Donner accès à l'identifiant d'un saut manuel dans les conditions d'exécutions d'action ajouté
Mis à jour par Emmanuel Cazenave il y a 12 mois
Une proposition qui interdit les duplications d'identifiant sur les nouveaux workflows ou sur les workflows existants dans lesquels il n'y a pas de duplication.
Les workflow existants sont laissés tranquilles, dans l'idée qu'on pourrait passer dessus à la main pour corriger et qu'on pourra ensuite dire officiellement que l'unicité est garantie.
Mis à jour par Robot Gitea il y a 11 mois
- Statut changé de Solution proposée à Solution validée
Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 11 mois
- Statut changé de Solution validée à Résolu (à déployer)
Emmanuel Cazenave (ecazenave) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/303
- Titre : workflow: guarantee jump identifiers unicity on new workflows (#74132)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/303/files
Mis à jour par Transition automatique il y a 11 mois
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Anaïs Ecuvillon → en congés, retour le 30/04 il y a 8 mois
- Lié à Bug #80843: L'unicité d'identifiant d'un saut manuel n'est pas respecté lorsque l'on utilise la copie d'une action ajouté
workflow: guarantee jump identifiers unicity on new workflows (#74132)
And on existing workflow with no existing identifier duplication.