Projet

Général

Profil

Development #69659

Duplication de workflow - les ids des triggers ne sont pas conservés

Ajouté par Lauréline Guérin il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
28 septembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

cf ticket client #69589

Un workflow a été dupliqué, quelques modifications ont été apportées (sans que les actions globales soient modifiées), le nouveau workflow a été appliqué au formulaire correspondant.
Une action globale (avec un trigger automatique avec un délai de 5 jours) a été rejouée sur d'anciennes demandes.

Quand on regarde si on doit jouer un trigger ou pas, on se base sur son id. L'id n'étant pas le même, le trigger n'est pas marqué comme étant joué, donc il est rejoué.

                seen_triggers = []
                for part in formdata.iter_evolution_parts():
                    if not isinstance(part, WorkflowGlobalActionTimeoutTriggerMarker):
                        continue
                    seen_triggers.append(part.timeout_id)

                for action, trigger in triggers:
                    if trigger.id in seen_triggers:
                        continue  # already triggered  <-- raté, c'est pas le même id

Reproduit en local: créer un workflow, avec une action globale, un trigger, le dupliquer. L'id du trigger dans le nouveau workflow est modifié.

En pièce jointe un test pour reproduire.


Fichiers

0001-wip.patch (1,26 ko) 0001-wip.patch Lauréline Guérin, 28 septembre 2022 10:28
0001-workflow-keep-trigger-id-on-workflow-duplication-696.patch (1,96 ko) 0001-workflow-keep-trigger-id-on-workflow-duplication-696.patch Lauréline Guérin, 03 octobre 2022 16:27

Révisions associées

Révision b0bcc766 (diff)
Ajouté par Lauréline Guérin il y a plus d'un an

workflow: keep trigger id on workflow duplication (#69659)

Historique

#2

Mis à jour par Lauréline Guérin il y a plus d'un an

#3

Mis à jour par Frédéric Péters il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée
#4

Mis à jour par Lauréline Guérin il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit b0bcc7667209ec69796c24fda3418ef568d011e4
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Oct 3 15:42:25 2022 +0200

    workflow: keep trigger id on workflow duplication (#69659)
#5

Mis à jour par Transition automatique il y a plus d'un an

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

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF