Projet

Général

Profil

Development #37920

Crash de passerelle sur debug tool bar

Ajouté par Emmanuel Cazenave il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
25 novembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 72d6fec4 (diff)
Ajouté par Emmanuel Cazenave il y a plus de 4 ans

use template inclusion to enable debug toolbar (#37920)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Assigné à changé de Emmanuel Cazenave à Benjamin Dauvergne
#2

Mis à jour par Emmanuel Cazenave il y a plus de 4 ans

Ç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.

#3

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

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.

#4

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 ?

#5

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.

#6

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.

#7

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)

Formats disponibles : Atom PDF