Development #76051
api, ne pas charger toutes les évolutions quand on sait qu'elles ne serviront pas
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
Historique
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 :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/214
- Titre : WIP: api: do not load all evolutions if they are not required (#76051)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/214/files
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 :
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 :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/214
- Titre : api: do not load all evolutions if they are not required (#76051)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/214/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
api: do not load all evolutions if they are not required (#76051)