Development #20374
Permettre de "mélanger" les demandes de différentes collectivités
0%
Description
Aujourd'hui vu la structure des données passées au template d'une cellule "Demandes en cours", il est très compliqué de lister indifféremment les demandes venant de différentes collectivités (typiquement la métropole et une commune); il faudrait plutôt passer dans le contexte une liste plate et qu'un regroupement ait lieu au niveau du template de rendu, quand on le souhaite.
Fichiers
Révisions associées
wcs: rely on global render method (#20374)
wcs: provide flat lists of formdatas/drafts to templates (#20374)
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Assigné à mis à Frédéric Péters
- Priorité changé de Normal à Haut
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0003-wcs-provide-flat-lists-of-formdatas-drafts-to-templa.patch 0003-wcs-provide-flat-lists-of-formdatas-drafts-to-templa.patch ajouté
- Fichier 0002-wcs-rely-on-global-render-method-20374.patch 0002-wcs-rely-on-global-render-method-20374.patch ajouté
- Fichier 0001-wcs-add-site-slug-attribute-to-all-items-20374.patch 0001-wcs-add-site-slug-attribute-to-all-items-20374.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Dans 0001 l'ajout d'une propriété site_slug à ce qu'on récupère de w.c.s.; ça permettra dans les templates de différencier les démarches venant de tel ou tel site.
Dans 0002 la modification aux cellules de combo.apps.wcs pour fournir get_cell_extra_context et ne plus redéfinir de méthode render(), l'avantage étant qu'on gagne ainsi la prise en charge de templates alternatifs (basés sur le slug de la cellule). (ce n'est pas particulièrement lié au ticket mais je préférais pour la suite faire ça dans la bonne méthode plutôt qu'empiler dans .render(), et c'est mieux pour les tests).
Dans 0003 le point de ce ticket, réunir dans une liste unique les infos des différents sites (variable nommée forms/drafts selon la cellule).
Les anciennes variables et leurs contenus sont conservés, pour conserver la compatibilité avec les templates existants.
Mis à jour par Thomas Noël il y a plus de 6 ans
0001: ok
0002:
ligne 357, je crois qu'il faut appeler super(WcsFormsOfCategoryCell,... au lieu de super(WcsCommonCategoryCell,...
Juste en dessous il y a disparition de context[self.variable_name] = self.get_data(context), qui n'est pas repris dans extra_context ? (j'avoue être un peu perdu avec ce context[self.variable_name] rejoué un peu partout, c'est peut-être une disparition voulue)
0003: et plouf je suis perdu, comprends plus rien, dodo
Mis à jour par Frédéric Péters il y a plus de 6 ans
ligne 357, je crois qu'il faut appeler super(WcsFormsOfCategoryCell,... au lieu de super(WcsCommonCategoryCell,...
yep.
Juste en dessous il y a disparition de context[self.variable_name] = self.get_data(context), qui n'est pas repris dans extra_context ? (j'avoue être un peu perdu avec ce context[self.variable_name] rejoué un peu partout, c'est peut-être une disparition voulue)
C'est fait au niveau de la classe parente WcsBlurpMixin. (et il existe #13490 pour refactoriser et s'éloigner de la structure qui était venue avec l'utilisation de blurps).
0003: et plouf je suis perdu, comprends plus rien, dodo
Zut alors ça me semblait plus net que le 0002, pas de bagage passé à vivre; en gros, à côté des données déjà obtenues, et présentes dans context['user_forms'] pour l'une et dans context['current_drafts'] pour l'autre, on crée une simple liste, dans respectivement context['forms'] et context['drafts'].
Mis à jour par Thomas Noël il y a plus de 6 ans
C'est justement le context['forms'] qui me perdait un peu, à cause du context[self.variable_name] où variable_name peut être forms, etc... Je relis ça.
...
Et c'est pigé, et donc, ack.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 31698ae4a94723b8de6761bdeb848c592abbd199
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 3 17:39:00 2018 +0100
wcs: provide flat lists of formdatas/drafts to templates (#20374)
commit 921ab9f4840cea32dfe05efee91db4474384e674
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 3 17:28:30 2018 +0100
wcs: rely on global render method (#20374)
commit a129bda7c9fdc1e5cb5d92755d788f00c3ddb694
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 3 17:03:24 2018 +0100
wcs: add site slug attribute to all items (#20374)
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: add site slug attribute to all items (#20374)