Projet

Général

Profil

Development #23918

Faire la mise à jour des données sur un changement de workflow hors de la requête

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Avec un grand nombre de demandes ça peut prendre un temps considérable.


Fichiers

Révisions associées

Révision e7dac159 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

misc: use an afterjob to update formdatas when a workflow is changed (#23918)

Révision 31156f6c (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

sql: always rebuild security attributes of formdata in the same order (#23918)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

Sous forme d'afterjob, que je préfère quand même au cron tournant toutes les minutes.

#2

Mis à jour par Thomas Noël il y a presque 6 ans

Est-ce qu'on pourrait afficher dans la barre latérale un "Recalcul des permissions sur les demandes liées…" avec un petit spinner ?

Et donc, dit en jabber : sur un formulaire avec des milliers de demandes, et la modification rapide de deux ou trois étapes du workflow, on a un petit risque avoir deux ou trois afterjobs qui se race-conditionnent. Il faudrait minimiser le risque en parcourant les formdef puis les formdata dans un ordre déterminé.

#3

Mis à jour par Frédéric Péters il y a presque 6 ans

Est-ce qu'on pourrait afficher dans la barre latérale un "Recalcul des permissions sur les demandes liées…" avec un petit spinner ?

Pas vraiment évident, non. Le côté persistant en moins, c'est aussi un peu ce que je notais dans #23920.

Et donc, dit en jabber : sur un formulaire avec des milliers de demandes, et la modification rapide de deux ou trois étapes du workflow, on a un petit risque avoir deux ou trois afterjobs qui se race-conditionnent. Il faudrait minimiser le risque en parcourant les formdef puis les formdata dans un ordre déterminé.

J'y crois quand même peu, mais patch indépendant garantissant un ordre.

#4

Mis à jour par Thomas Noël il y a presque 6 ans

Comme je suis têtu, posons aussi un order_by='id' dans le FormDef.select(lambda x: x.workflow_id == self.id, ignore_migration=True)

#6

Mis à jour par Thomas Noël il y a presque 6 ans

Ack pour les deux.

(et promis, je me soigne)

#7

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 31156f6ca19c9fda242b99ec3be2eb1b638bbf31
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri May 18 12:26:55 2018 +0200

    sql: always rebuild security attributes of formdata in the same order (#23918)

commit e7dac1590c391e90b9fae67d844bfcff7ce96bf8
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri May 18 10:34:18 2018 +0200

    misc: use an afterjob to update formdatas when a workflow is changed (#23918)
#8

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

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

Formats disponibles : Atom PDF