Bug #24081
supprimer "from global_settings import *"
0%
Description
- Pour assurer en amont la double compatibilité Django 1.8 et 1.11 dans une configuration Hobo (qui cherche l'existence de TEMPLATE_DEBUG, et qui même si TEMPLATES est défini, il sera ignoré)
- L'option préférable serait d'importer seulement les variables de global_settings dont on a besoin dans les settings d'Authentic.
Suite à la discussion (x2) sur Hobo (#24071)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Anonyme il y a presque 6 ans
- Lié à Development #21489: Fonctionner avec Django 1.11 (et 1.8) ajouté
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Lié à Development #24071: Support de TEMPLATES dès qu'il sont définis par ailleurs ajouté
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-dont-import-everything-from-global_settings-24081.patch 0001-dont-import-everything-from-global_settings-24081.patch ajouté
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel Cazenave
- Patch proposed changé de Non à Oui
Testé sur mon publik local.
J'ai fait le tour de debian-jessie/debin_config.py
et debian-jessie/multitenant/debian_config.py
pour identifier ce qui pourrait clocher,
d'où le hack sur CACHES
.
Aussi fait le tour des serveurs de prod et test pour voir ce qu'il y a dans les /etc/authentic2-multitenant. Pour les autres déploiements je ne sais pas.
Sans doute à merger et pousser en tout début de fenêtre pour avoir le max de temps pour ajuster.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Ça
STATICFILES_FINDERS = global_settings.STATICFILES_FINDERS + ('gadjo.finders.XStaticFinder',)
Pour que ça passe plusieurs versions de Django notamment les changements tuple() -> list() (je ne sais plus quand ça arrive, ou si c'est déjà complètement arrivé en 1.8), il faudrait pas convertir en concat_seq(s1, s2)
avec
def concat_seq(l1, l1): return l1 + l1.__class__(l2)
Sinon ok.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-dont-import-everything-from-global_settings-24081.patch 0001-dont-import-everything-from-global_settings-24081.patch ajouté
- Statut changé de En cours à Solution proposée
ok, je pousserai celui là.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Je croyais que c'était le sens de ton "ack pour ..." !
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 96d6282d56525d0bc444a9cec98c65a7ef493311 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Wed Jun 6 16:45:50 2018 +0200 dont' import everything from global_settings (#24081)
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution déployée à Fermé
dont' import everything from global_settings (#24081)