Bug #23573
Le changement de workflow peut abimer l'historique des demandes
0%
Description
Cas typique, workflow de départ :
/-- Statut 1 (terminal) Arrivée -> aguillage selon form_var_foo \-- Statut 2 (terminal)
Workflow d'arrivée :
/-- Statut X (terminal) Arrivée -> aguillage selon form_var_foo \-- Statut Y (terminal)
Lors du passage du workflow de départ vers celui d'arrivée, on propose de changer le statut "aguillage selon form_var_foo" en Statut X ou Statut Y, car on ne propose que les statuts terminaux.
Problème : les historiques de toutes demandes contiennent un aiguillage... qui va devenir faux (Statut X, ou Statut Y, selon le choix fait lors du changement).
Il faudrait donc afficher tous les statuts comme cible, et ne pas se limiter aux status terminaux.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 6 ans
- afficher d'abord la liste des statuts terminaux du workflows d'arrivée, puis la liste des statuts intermédiaires (quand même moins problématique), en expliquant éventuellement que la gestion de ces derniers n'est utile que pour la mise à jour des historiques.
- idem pour la liste des statuts cibles, avec un select à deux sections (terminaux/intermédiaires)
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier Capture d’écran de 2018-05-03 18-13-06.png Capture d’écran de 2018-05-03 18-13-06.png ajouté
- Fichier 0001-DRAFT-remap-all-status-23573.patch 0001-DRAFT-remap-all-status-23573.patch ajouté
Un truc dans le genre, sauf que ça marche pas du tout parce que je pense n'avoir rien compris à l'usage de options/options_with_attributes dans SingleSelectHintWidget (on se retrouve avec des value="{disabled:True}" dans le HTML, bravo Thomas).
Copie d'écran du rendu, pour l'idée. On voit pas mais chaque liste propose l'ensemble des status, groupés par finaux/intermédiaires avec un séparateur, c'est très joli.
Mis à jour par Frédéric Péters il y a presque 6 ans
Pour options_with_attributes il faut un tuple avec 4 éléments; le (value:any, description:any, key:string) complet de Quixote + un dictionnaire.
Mis à jour par Thomas Noël il y a presque 6 ans
- Fichier 0001-DRAFT-remap-all-status-23573.patch 0001-DRAFT-remap-all-status-23573.patch ajouté
- Fichier Capture d’écran de 2018-05-28 16-46-52.png Capture d’écran de 2018-05-28 16-46-52.png ajouté
- Patch proposed changé de Non à Oui
Merci. Bon... c'est toujours pas ça, le formulaire semble correct au niveau du HTML, mais quixote ne valide pas... C'est pas mon jour.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier 0001-admin-expose-all-workflow-status-when-remapping-2357.patch 0001-admin-expose-all-workflow-status-when-remapping-2357.patch ajouté
- Statut changé de Nouveau à Solution proposée
Je reviens parce que j'ai rencontré ça (sur une situation un peu différente); le patch parle de distinction entre "terminal" et "intermediate" mais ce n'est pas "terminal", c'est "waitpoint", et je ne pense pas nécessaire de venir exposer cette notion. ("statuts de passage", "statuts de transition"...?)
Au plus simple, tout exposer.
Mis à jour par Thomas Noël il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Au plus simple, tout exposer.
J'eusse du rester sur ça qui était ma proposition initiale. (je pensais que ça passerait pas test_form_workflow_remapping, mais si, hop)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9d04ffd77901d9fbf315b29d612af94a613b08d3 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Feb 11 10:15:09 2019 +0100 admin: expose all workflow status when remapping (#23573)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
admin: expose all workflow status when remapping (#23573)