Projet

Général

Profil

Bug #43243

changement de workflow d'un formulaire : appliquer le mapping des status aux marqueurs

Ajouté par Thomas Noël il y a presque 4 ans. Mis à jour il y a 6 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
24 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Lors d'un changement de workflow dans workflow_status_remapping_submit (admin/forms.py) on adapte les statuts des formdata et de leur évolution :

     def workflow_status_remapping_submit(self, form):
            ...
            # if there are status changes, update all formdatas
            ...
            for item in self.formdef.data_class().select():
                item.status = status_mapping.get(item.status)
                if item.evolution:
                    for evo in item.evolution:
                        evo.status = status_mapping.get(evo.status)
                item.store()

On a oublié ici d'adapter aussi formdata.workflow_data['_markers_stack']

Révisions associées

Révision 3ee39ffd (diff)
Ajouté par Frédéric Péters il y a 6 mois

misc: update jump markers stack on workflow changes (#43243)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 2 ans

C'est toujours le cas et la résolution clashe désormais avec le remapping qui se fait maintenant en SQL :/

#2

Mis à jour par Thomas Noël il y a plus de 2 ans

Frédéric Péters a écrit :

C'est toujours le cas et la résolution clashe désormais avec le remapping qui se fait maintenant en SQL :/

On va sérieusement devoir envisager de passer ça en asynchrone ? Ou autre idée ?

#3

Mis à jour par Frédéric Péters il y a plus de 2 ans

(non pas d'idée particulière, j'ai juste noté ça en passant)

#4

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Je serais plutôt pour déplacer _markers_stacks dans une colonne dédiée de type text[] une fois pour toute, pour pouvoir continuer à traiter ça efficacement en SQL. C'est sûr que c'est plus de boulot dans l'immédiat, mais je pense que déprécier workflow_data serait une bonne chose à long terme, en commençant par les usages un peu bloquant comme ici (et ensuite les trucs moches comme les triggers qui écrasent tout le contenu de workflow_data un peu violemment).

#5

Mis à jour par Robot Gitea il y a 6 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Robot Gitea il y a 6 mois

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

Mis à jour par Robot Gitea il y a 6 mois

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

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Robot Gitea il y a 6 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#9

Mis à jour par Transition automatique il y a 6 mois

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

Mis à jour par Transition automatique il y a 4 mois

Automatic expiration

Formats disponibles : Atom PDF