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).
Files
Related issues
Associated revisions
multitenant: drop 1.8 compatibility from template loader (#33238)
History
Updated by Frédéric Péters about 4 years ago
- Blocked by Development #33236: mise à jour {% extends … %} pour 1.11 added
Updated by Frédéric Péters about 4 years ago
- File 0002-multitenant-drop-1.8-compatibility-from-template-loa.patch 0002-multitenant-drop-1.8-compatibility-from-template-loa.patch added
- File 0001-misc-require-django-1.11-33238.patch 0001-misc-require-django-1.11-33238.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Benjamin Dauvergne about 4 years ago
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é.
Updated by Frédéric Péters about 4 years ago
Updated by Emmanuel Cazenave about 4 years ago
- Status changed from Solution proposée to Solution validée
Updated by Frédéric Péters about 4 years ago
- Status changed from Solution validée to 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)
>Updated by Frédéric Péters about 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: require django 1.11 (#33238)