Bug #13390
wcs: "more_forms" dans le contexte passé d'une cellule à l'autre
0%
Description
Si la page assure le rendu de plusieurs cellules "formulaires d'une catégorie" et que le premier d'entre eux contient plus de formulaires que la limite d'affichage, on trouve ces formulaires dans context['more_forms'], mais les cellules qui suivent, qui ne dépassent pas le nombre limite de formulaires, ne remettent pas context['more_forms'] à [], et on se trouve donc avec les formulaires de la première cellule répétés dans les suivantes.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-wcs-be-sure-to-reset-context-more_forms-when-renderi.patch 0001-wcs-be-sure-to-reset-context-more_forms-when-renderi.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
Il me semble qu'on devrait utiliser context.push()/pop() pour éviter que les contextes de différentes cellules ne se rencontrent (à prendre en compte les cellules qui communiquent entre celle comme ParameterCell/CubeCell mais il me semble que ça passe par une variable commune posée bien avant), notamment dans CellBase.render(). Aussi les cellules wcs modifient context directement dans render() alors que c'est get_cell_extra_context() qui devrait définir le contenu du context. En fait l'utilisation du BlurpMixin rend le code assez difficile à lire.
Mis à jour par Thomas Noël il y a plus de 7 ans
Benjamin Dauvergne a écrit :
En fait l'utilisation du BlurpMixin rend le code assez difficile à lire.
Ah bon ? Vraiment ? ;)
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
Je le pousse mais je crée #13490 pour appeler à du refactoring.
commit f7678c42d86d44503732836496cc027b4a91ff52
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Oct 3 16:30:47 2016 +0200
wcs: be sure to reset context['more_forms'] when rendering cell (#13390)
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
wcs: be sure to reset context['more_forms'] when rendering cell (#13390)