Development #37920
Crash de passerelle sur debug tool bar
0%
Description
Suite à #37917 qui fait lui même suite à #37760, dans devinst :
self._wrapped = Settings(settings_module) File "/home/cazino/envs/publik-env/local/lib/python2.7/site-packages/django/conf/__init__.py", line 110, in __init__ mod = importlib.import_module(self.SETTINGS_MODULE) File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module __import__(name) File "/home/cazino/src/passerelle/passerelle/settings.py", line 249, in <module> execfile(local_settings_file) File "/home/cazino/.config/publik/settings/passerelle/settings.py", line 58, in <module> MIDDLEWARE= ['debug_toolbar.middleware.DebugToolbarMiddleware'] + MIDDLEWARE TypeError: can only concatenate list (not "tuple") to list
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Assigné à changé de Emmanuel Cazenave à Benjamin Dauvergne
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
- Fichier 0001-use-template-inclusion-to-enable-debug-toolbar-37920.patch 0001-use-template-inclusion-to-enable-debug-toolbar-37920.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à changé de Benjamin Dauvergne à Emmanuel Cazenave
- Patch proposed changé de Non à Oui
Ça rétablit sur passerelle et combo qui était les deux seuls où ça marchait (a2 est censé marcher aussi mais en fait non, je n'ai jamais crée le ticket parce que je me sers pas de la toolbar).
Et je change de façon de faire, pas compris qu'est-ce que je devais rajouter comme clause dans le if vars globals truc machin, pas envie de savoir, je trouve ça bien plus clair en étant déclaratif.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-MIDDLEWARE-is-a-tuple-with-Publik-37920.patch 0001-MIDDLEWARE-is-a-tuple-with-Publik-37920.patch ajouté
- Tracker changé de Bug à Development
Au début j'ai voulu mettre list(MIDDLEWARE) puis je me suis dit que c'était un peu tôt pour ce genre de changements audacieux. J'ai fait dans l'orthodoxie.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Assigné à changé de Emmanuel Cazenave à Benjamin Dauvergne
J'ai raté que tu t'étais assigné le ticket, néanmoins pourquoi autant de changements qui n'ont rien à voir ? Ça marche déjà non ?
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
- Assigné à changé de Benjamin Dauvergne à Emmanuel Cazenave
Ça a à voir parce que maintenant ça ne crashe plus, et oui j'ai pas fait un patch minimaliste : pour tout dire je n'aime vraiment pas la mini programmation dans les fichiers de configuration, je trouve que ça rend les choses confuses, ça me démangeait de revenir là dessus, j'ai trouvé l'occasion.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à changé de Emmanuel Cazenave à Benjamin Dauvergne
Je ne comprends pas la nouvelle façon de faire mais bon.
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
- Statut changé de Solution validée à Solution déployée
Pour la petite explication, j'utilise ici de la composition de template, c'était déjà utilisé par ailleurs sur ces fichiers de conf.
commit 72d6fec46160dff8138baacc1b603e70d74d7558 (HEAD -> master, origin/master) Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Mon Nov 25 14:50:00 2019 +0100 use template inclusion to enable debug toolbar (#37920)
use template inclusion to enable debug toolbar (#37920)