Development #47492
pouvoir marquer une vue personnalisée comme la vue par défaut
0%
Description
Il ne s'agirait pas de rediriger vers celle-ci (/management/plop/ → /management/plop/custom-view/) mais d'appliquer son paramétrage à la vue par défaut. (histoire de ne pas apporter de confusion dans les boutons d'enregistrement de vue qui du coup inviteraient à écraser la vue plutôt qu'à en créer une nouvelle).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
Lisant la branche rapidement, je me demande si le from_default_view porté partout ne peut pas être contourné, genre en posant un flag type is_default_view en attribut, dans le constructeur ou le _q_lookup de FormPage). Moins marquant mais un peu dans le même ordre idée, passer l'attribut view en propriété, pour ne pas avoir à modifier partout s/self.view/self.get_view()/.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0003-backoffice-set-default-view-as-default-47492.patch 0003-backoffice-set-default-view-as-default-47492.patch ajouté
- Fichier 0002-misc-split-backoffice-pages-tests.patch 0002-misc-split-backoffice-pages-tests.patch ajouté
- Fichier 0001-misc-move-backoffice-pages-tests.patch 0001-misc-move-backoffice-pages-tests.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
remarques prises en compte:
- flag use_default_view setté dans _q_index
- property view qui tient compte de ce flag
Du coup ça fait moins de modif, mais au moins ça m'a permis de comprendre comment ça fonctionne :)
en option (0001 et 0002):
split des tests backoffice (j'ai choisi de les ranger dans un dossier - finalement je trouve ça plus propre - si c'est ok à l'occasion je ferai pareil pour les tests admin_pages que j'ai commencé à splitté)
j'en ai aussi profité pour simplifier la création des data de test pour tests/backoffice_pages/test_custom_view.py (en local je passe de 45s à 15s, avant que j'ajoute des tests dans 0003)
Mis à jour par Lauréline Guérin il y a plus de 3 ans
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
(il y a peut-être un rebase un peu pénible à faire pour les déplacements de test)
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f0890d999cd1a7d489c9cc759bfb24adb50f4e1e
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Oct 13 15:10:49 2020 +0200
backoffice: set default view as default (#47492)
commit 5bb8b785126c1b07cbcb6cc6e3e08ac42cc939e5
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Oct 13 15:37:02 2020 +0200
misc: split backoffice pages tests
commit 111569ff59d8e355f4768447dd4ada231653bbc4
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Oct 13 15:20:49 2020 +0200
misc: move backoffice pages tests
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: set default view as default (#47492)