Development #33238
template loader, simplifier pour 1.11
0%
Description
Il y a des hacks dans template_loader.py pour les @extend '../../' que fai(sai)t publik-base-theme (#33236), ça simplifierait le code de les retirer, ça permettrait ensuite également au cache de fonctionner (cf #20933#note-11).
Fichiers
Demandes liées
Révisions associées
multitenant: drop 1.8 compatibility from template loader (#33238)
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Bloqué par Development #33236: mise à jour {% extends … %} pour 1.11 ajouté
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0002-multitenant-drop-1.8-compatibility-from-template-loa.patch 0002-multitenant-drop-1.8-compatibility-from-template-loa.patch ajouté
- Fichier 0001-misc-require-django-1.11-33238.patch 0001-misc-require-django-1.11-33238.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001 pour forcer 1.11 (où l'on découvre qu'étaient publiés dans jenkins le même rapport de coverage pour les différentes séries de tests).
0002 pour le taf sur template loader.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Frédéric Péters a écrit :
0001 pour forcer 1.11 (où l'on découvre qu'étaient publiés dans jenkins le même rapport de coverage pour les différentes séries de tests).
Pour coverage je préférerai qu'on standardise ma façon de faire depuis quelques mois (utilisée maintenant sur a2, a2-auth-kerberos, django-kerberos et django-mellon), en faisant du --cov-append sur un fichier de coverage unique, ça donne ces options :
authentic2/tox.ini: coverage: COVERAGE=--cov=src --cov-branch --cov-append --cov-report xml --cov-report html
Ce qui est important pour nous c'est la couverture apportée par l'ensemble des tests (sachant que des bouts de code sont spécifiques à tel ou tel environnement)
La version html native est régénérée à chaque fois de manière inutile mais la dernière est la bonne.
0002 pour le taf sur template loader.
Le code me va très bien, je m'aperçois par contre qu'on a aucun test sur ces aspects d'accès aux templates par tenant, templates du thème, templates de variante, templates de portail agent et maintenant d'usage de la récursivité.
Mis à jour par Frédéric Péters il y a presque 5 ans
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a8670a2197f4d7138436f57cfe2ebc6dee4c2804 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun May 19 14:15:38 2019 +0200 multitenant: drop 1.8 compatibility from template loader (#33238) commit f0d10f92a928fc388d22288902a012947bf2a259 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun May 19 14:18:30 2019 +0200 misc: require django 1.11 (#33238)>
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: require django 1.11 (#33238)