Project

General

Profile

Development #33238

template loader, simplifier pour 1.11

Added by Frédéric Péters (de retour le 27 mai) about 5 years ago. Updated almost 5 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
19 May 2019
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

0002-multitenant-drop-1.8-compatibility-from-template-loa.patch (5.45 KB) 0002-multitenant-drop-1.8-compatibility-from-template-loa.patch Frédéric Péters (de retour le 27 mai), 19 May 2019 03:59 PM
0001-misc-require-django-1.11-33238.patch (5.2 KB) 0001-misc-require-django-1.11-33238.patch Frédéric Péters (de retour le 27 mai), 19 May 2019 03:59 PM

Related issues

Blocked by Intégrations graphiques Publik - Development #33236: mise à jour {% extends … %} pour 1.11Fermé19 May 201927 May 2019

Actions

Associated revisions

Revision a8670a21 (diff)
Added by Frédéric Péters (de retour le 27 mai) almost 5 years ago

multitenant: drop 1.8 compatibility from template loader (#33238)

History

#1

Updated by Frédéric Péters (de retour le 27 mai) about 5 years ago

#2

Updated by Frédéric Péters (de retour le 27 mai) about 5 years ago

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.

#3

Updated by Benjamin Dauvergne about 5 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é.

#4

Updated by Frédéric Péters (de retour le 27 mai) about 5 years ago

coverage je préférerai qu'on standardise ma façon de faire depuis quelques mois

J'ai créé #33244.

je m'aperçois par contre qu'on a aucun test sur ces aspects d'accès aux templates

Oui, avec #32886 il y aura le début d'une fixture de thèmes, qui permettra d'avancer là-dessus.

#5

Updated by Emmanuel Cazenave almost 5 years ago

  • Status changed from Solution proposée to Solution validée
#6

Updated by Frédéric Péters (de retour le 27 mai) almost 5 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)
>
#7

Updated by Frédéric Péters (de retour le 27 mai) almost 5 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF