Skip to content

Fixes #6412 - More details about template files. #947

Closed
wants to merge 1 commit into from

5 participants

@MarkusH
Django member
MarkusH commented Mar 23, 2013

Report more details about template files in loader postmortem.

https://code.djangoproject.com/ticket/6412

@jezdez jezdez commented on an outdated diff Apr 7, 2013
django/views/debug.py
@@ -230,8 +239,9 @@ def get_traceback_data(self):
source_list_func = loader.get_template_sources
# NOTE: This assumes exc_value is the name of the template that
# the loader attempted to load.
- template_list = [{'name': t, 'exists': os.path.exists(t)} \
- for t in source_list_func(str(self.exc_value))]
+ template_list = [{
+ 'name': t, 'status': self.format_path_status(t)
@jezdez
Django member
jezdez added a note Apr 7, 2013

Mind wrapping the line after t, here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@zerok zerok commented on an outdated diff May 15, 2013
tests/view_tests/tests/debug.py
@@ -5,8 +5,11 @@
import inspect
import os
+import shutil
import sys
@zerok
zerok added a note May 15, 2013

Since also tempfile is also part of the stdlib, I'd add it to the same block as sys, shutil etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@dokterbob dokterbob Fixed #6412 -- More details if a template file cannot be loaded
Report more details about template files in loader postmortem.
42ba8f9
@timgraham
Django member

merged in 61a8de6

@timgraham timgraham closed this May 31, 2013
@MarkusH MarkusH deleted the MarkusH:ticket6412 branch Jun 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.