Projet

Général

Profil

Development #41597

django-debug-toolbar se retrouve dans les templates distribués aux autres briques

Ajouté par Benjamin Dauvergne il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
10 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Vu aujourd'hui sur mon installation locale, le menu django-debug-toolbar est visible sur fargo (alors qu'il n'est pas activé), tout simplement parce que le squelette transmis par combo à fargo contient les déclarations des scripts de dj-dbg-tlbr.

Il faudrait trouver comme désactiver django-debug-toolbar sur la vue des squelettes.


Fichiers

Révisions associées

Révision c81470ef (diff)
Ajouté par Benjamin Dauvergne il y a environ 4 ans

misc: hide debug toolbar in skeleton view (#41597)

Historique

#1

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

  • Tracker changé de Development à Bug
#2

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

  • Assigné à mis à Benjamin Dauvergne
#3

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

Testé localement avec publik-devinst ça marche très bien.

#4

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

Je me dis que ces trois lignes auraient peut-être toutes pu être dans le settings.py; ma préoccupation étant ici surtout une question d'affichage, ça va faire dans le coverage un fichier pas couvert, il y aura nécessairement quelqu'un à un moment pour trouver ça moche et vouloir arranger ça.

Ma préférence irait donc plutôt à déplacer le code pour être

def debug_show_toolbar(request):
    from debug_toolbar.middleware import show_toolbar as dt_show_toolbar
    return dt_show_toolbar(request) and not request.path.startswith('/__skeleton__/')

DEBUG_TOOLBAR_CONFIG = {
    'SHOW_TOOLBAR_CALLBACK': debug_show_toolbar,
}

(j'ai vérifié on peut passer ainsi la fonction)

#5

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

Ok, branche à jour.

#6

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

  • Statut changé de Solution proposée à Solution validée
#7

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit c81470efcf4db1ba7b55d43b6e0a9c38e30078e8
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Apr 20 21:06:43 2020 +0200

    misc: hide debug toolbar in skeleton view (#41597)
#8

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF