Bug #52629
lors de l'édition d'une demande, une évolution "vide" est créée, et last_update_time est mis à jour
0%
Description
Si l'édition est annulée, cette evolution finale avec status=None reste en place et cela plante notamment le calcul de last_update_time
Et alors, par exemple, les expirations ne sont plus calculées correctement.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 3 ans
- Fichier 0001-formdata-do-not-consider-empty-evolution-in-last_upd.patch 0001-formdata-do-not-consider-empty-evolution-in-last_upd.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'est lors de l'entrée en mode wfedit (edit_mode=True) qu'une evolution est ajoutée, sans statut. Elle reste là si l'édition est annulée ou si l'édition est validée mais que la demande ne change pas de statut (action Edition sans statut cible)
Je n'ai pas cerné où cette évolution est ajoutée... ni si elle est bien pertinente...
Alors pour quand même voir le pépin j'ai écrit un test... et de là, ce patch qui se propose juste de calculer différement last_update_time, en ignorant les évolutions sans status.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à En cours
C'est rouge.
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de En cours à Solution proposée
(oui mais sans rapport), je viens d'aller cliquer sur le lien pour relancer un build.
~~
cela étant je préférerais prendre un peu de temps sur l'origine du soucis, plutôt que contourner.
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à En cours
Frédéric Péters a écrit :
cela étant je préférerais prendre un peu de temps sur l'origine du soucis, plutôt que contourner.
So do I.
Mis à jour par Frédéric Péters il y a environ 3 ans
- Assigné à changé de Thomas Noël à Frédéric Péters
- Patch proposed changé de Oui à Non
Mis à jour par Frédéric Péters il y a environ 3 ans
- Fichier 0001-workflows-add-history-entry-after-successful-edit-ac.patch 0001-workflows-add-history-entry-after-successful-edit-ac.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 29ac5ab0d3670f1ef9c434b5d0f8ce32ae1b5019
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat Apr 3 20:11:15 2021 +0200
workflows: add history entry after successful edit action, not before (#52629)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: add history entry after successful edit action, not before (#52629)