Produits Entr'ouvert » w.c.s.

À titre personnel je trouve bien plus pratique le rendu actuel dans w.c.s., qui me fournit directement l'information utile, vs le rendu django, qui commence par un écran de contexte pas utile, puis une trace "most recent call last" qui oblige à scroller de manière précise pour arriver en bas de celle-ci (puisqu'ensuite la page continue avec la partie "request information"), et où ensuite il faut cliquer pour voir les variables locales.

Sans doute si on arrivait à quelque chose de plus utile (à mes yeux) côté django je trouverais une utilité à avoir pareil.

Sur "ça ferait moins de code à maintenir" le code de mise en forme des traces est exploité pour les "logged errors", serait de toute façon à conserver.

Je serais pour fermer ce ticket et plutôt améliorer les choses ailleurs. (surtout que les améliorations générales réalisées bénéficieront aussi aux vues "pure django" dans wcs)

Mais, si certaines personnes veulent les pages django, je peux faire ce patch :

--- a/wcs/compat.py
+++ b/wcs/compat.py
@@ -78,6 +78,8 @@ class TemplateWithFallbackView(TemplateView):
             # so it can be caught/stopped at when running tests.
             if settings.DEBUG_PROPAGATE_EXCEPTIONS:
                 raise
+            if settings.USE_DJANGO_EXCEPTION_PAGES:
+                raise
             context = {'body': get_publisher().finish_failed_request()}
             self.quixote_response = get_request().response

@@ -215,6 +217,8 @@ class CompatWcsPublisher(WcsPublisher):
             # so it can be caught/stopped at when running tests.
             if settings.DEBUG_PROPAGATE_EXCEPTIONS:
                 raise
+            if settings.USE_DJANGO_EXCEPTION_PAGES:
+                raise
             output = self.finish_failed_request()
         response = request.response

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

→ /backoffice/settings/debug_options et cocher "Activer le mode de debug" et ça donne des traces affichées dans le navigateur.

Super je ne connaissais pas, ça résout pas mal le problème effectivement (idéalement ça serait posé par devinst, ou alors se trouver activé automatiquement dès lors que DEBUG=True (j'ai créé ce ticket après avoir convenu avec Manu du constat de pénibilité, on est sûrement une majorité à ne pas connaître l'option)).

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

À titre personnel je trouve bien plus pratique le rendu actuel dans w.c.s., qui me fournit directement l'information utile, vs le rendu django, qui commence par un écran de contexte pas utile, puis une trace "most recent call last" qui oblige à scroller de manière précise pour arriver en bas de celle-ci (puisqu'ensuite la page continue avec la partie "request information"), et où ensuite il faut cliquer pour voir les variables locales.

Oui ce qui est gênant dans un terminal est inversement pratique par mail/dans un navigateur.

Sans doute si on arrivait à quelque chose de plus utile (à mes yeux) côté django je trouverais une utilité à avoir pareil.

Oui on pourrait certainement avoir un rendu custom dans hobo et l'utiliser partout, mais il y a sûrement beaucoup de travail plus urgent qui passerait avant ça.

Sur "ça ferait moins de code à maintenir" le code de mise en forme des traces est exploité pour les "logged errors", serait de toute façon à conserver.

(rien n'empêcherait d'appeler le rendu Django à cet endroit là aussi)

Je serais pour fermer ce ticket

Yep

Mais, si certaines personnes veulent les pages django, je peux faire ce patch :

Pas la peine puisque tu donnes déjà la solution, mettre DEBUG_PROPAGATE_EXCEPTIONS = True dans ses settings.