Projet

Général

Profil

Bug #23035

Authentic 2 - Development #21489: Fonctionner avec Django 1.11 (et 1.8)

Meilleur support des settings TEMPLATES pour la compatiblité > 1.8

Ajouté par Anonyme il y a environ 6 ans. Mis à jour il y a presque 6 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
09 avril 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Ce fichier utilisé dans le déploiement multitenant (local avec publik-devinst) et sûrement ailleurs.
Il n'est pas conforme avec les défauts en list() et non en tuple(), ni conforme avec la suppression des variables TEMPLATE_*


Fichiers

Révisions associées

Révision f2389cf4 (diff)
Ajouté par Elias Showk il y a environ 6 ans

debian settings: better compatibility switch for > 1.8 TEMPLATES (#23035)

Révision e5f65933 (diff)
Ajouté par Elias Showk il y a environ 6 ans

Revert "debian settings: better compatibility switch for > 1.8 TEMPLATES (#23035)"
This reverts commit f2389cf4dced59fb7a4b40934b4290b99a6cdc6e.

Historique

#1

Mis à jour par Anonyme il y a environ 6 ans

Il suffit peut-être de mettre ligne 160 (?):

if 'TEMPLATE_DEBUG' in globals() and globals()['TEMPLATE_DEBUG']:

#2

Mis à jour par Anonyme il y a environ 6 ans

Évite le mélange difficile à déboguer des formats de settings TEMPLATES et TEMPLATE_ dans debian_config.py

#3

Mis à jour par Anonyme il y a environ 6 ans

  • Statut changé de Nouveau à En cours
#4

Mis à jour par Frédéric Péters il y a environ 6 ans

if not len(TEMPLATES): va planter si TEMPLATES n'est pas défini, ce qui est le cas des modules pas migrés.

#5

Mis à jour par Anonyme il y a environ 6 ans

  • Fichier 0001-debian-settings-better-compatibility-switch-for-1.8-.patch ajouté

Voilà, ça propose de tester l'existence de TEMPLATES dans global_settings
Pour rappel : je cherche une solution ce problème à l'origin : quand on est dans un environnement 1.11, c'est compliqué d'éliminer complètement les anciens settings TEMPLATE_* et ce patch favorise la présence de TEMPLATES pour différencier les version de django dans l'env.

#6

Mis à jour par Anonyme il y a environ 6 ans

  • Fichier 0001-debian-settings-better-compatibility-switch-for-1.8-.patch supprimé
#8

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Ack.

#9

Mis à jour par Anonyme il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit f2389cf4dced59fb7a4b40934b4290b99a6cdc6e (HEAD -> master, origin/master, origin/HEAD)
Author: Elias Showk <eshowk@entrouvert.com>
Date:   Thu Apr 19 10:30:53 2018 +0200

    debian settings: better compatibility switch for > 1.8 TEMPLATES (#23035)

#10

Mis à jour par Frédéric Péters il y a environ 6 ans

  • Statut changé de Résolu (à déployer) à En cours
#11

Mis à jour par Frédéric Péters il y a environ 6 ans

J'aimerais bien avoir #15470 en recette; il faudrait sur ce ticket une résolution rapide, revert ou autre.

#12

Mis à jour par Anonyme il y a environ 6 ans

Frédéric Péters a écrit :

Échec. https://jenkins.entrouvert.org/job/hobo/713/console

Comme dans le master d'authentic authentics2/settings.py, les variables TEMPLATE_ sont utilisées, et la variable TEMPLATES existe mais par défaut elle est vide [].
tox -e coverage-django18-authentic fonctionne bien chez moi désormais.?

#13

Mis à jour par Anonyme il y a environ 6 ans

Revert préférable.

Pour régler la présence à la fois de TEMPLATE_DEBUG et de TEMPLATES (non vide) dans certains environnement, il faudrait étudier

from django.conf import settings

Puis ligne 165, proposer un patch avec:
if 'TEMPLATE_DEBUG' in globals() and len(settings.TEMPLATES) == 0:
#14

Mis à jour par Thomas Noël il y a environ 6 ans

Ok pour le revert.

Pour le reste, il faudrait expliciter "certains environnement", c'est flou pour moi.

#15

Mis à jour par Anonyme il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit e5f65933276fdac004c2b4cb484bb8fae75bf3b0 (HEAD -> master, origin/master, origin/HEAD)
Author: Elias Showk <eshowk@entrouvert.com>
Date:   Fri Apr 20 11:50:19 2018 +0200

    Revert "debian settings: better compatibility switch for > 1.8 TEMPLATES (#23035)" 
    This reverts commit f2389cf4dced59fb7a4b40934b4290b99a6cdc6e.
#16

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

  • Statut changé de Résolu (à déployer) à Rejeté

Comme #24071, ce ticket avait émergé pour faciliter la compatibilité django1.11 pour authentik (#21489),
mais on essaye de s'en sortir autrement : #24081

Je ferme.

Formats disponibles : Atom PDF