Projet

Général

Profil

Development #76051

api, ne pas charger toutes les évolutions quand on sait qu'elles ne serviront pas

Ajouté par Frédéric Péters il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
30 mars 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Suite à #76003,

Mais pour l'appel list, par défaut on désactive tout et on ne renvoie que le minimum. Or malgré le patch je vois une requête pour récupérer les evolutions.

--- a/wcs/backoffice/management.py
+++ b/wcs/backoffice/management.py
@@ -2493,7 +2493,6 @@ class FormPage(FormdefDirectoryBase):
             offset=offset,
             limit=limit,
         )[0]
-        self.formdef.data_class().load_all_evolutions(items)
         digest_key = 'default'
         if self.view and isinstance(self.formdef, CardDef):
             view_digest_key = 'custom-view:%s' % self.view.get_url_slug()
@@ -2506,6 +2505,8 @@ class FormPage(FormdefDirectoryBase):
         include_submission = get_query_flag('include-submission') or full
         include_workflow = get_query_flag('include-workflow') or full
         include_workflow_data = get_query_flag('include-workflow-data') or full
+        if include_evolution or include_workflow:
+            self.formdef.data_class().load_all_evolutions(items)
         # noqa pylint: disable=too-many-boolean-expressions
         if (
             include_fields

Révisions associées

Révision bd2118ed (diff)
Ajouté par Frédéric Péters il y a environ un an

api: do not load all evolutions if they are not required (#76051)

Historique

#1

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à En cours

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

#2

Mis à jour par Robot Gitea il y a environ un an

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

Mis à jour par Robot Gitea il y a environ un an

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

Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a environ un an

  • 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 :

#5

Mis à jour par Transition automatique il y a environ un an

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

Mis à jour par Transition automatique il y a 11 mois

Automatic expiration

Formats disponibles : Atom PDF