aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMikhail Korobov2011-09-26 00:15:25 +0200
committerTomasz Buchert2011-09-26 00:15:25 +0200
commit65394257a31226a4c2a34aa8a358f0651c5f90f8 (patch)
tree0b313efe1516ced73413cc2ffc708efca1d7e755
parent01b5d65a1da80f3cc6120e53bba46e4da2ff44ca (diff)
downloaddjango-debug-toolbar-65394257a31226a4c2a34aa8a358f0651c5f90f8.tar.bz2
Better substitutions for template context variables triggering SQL queries
-rw-r--r--debug_toolbar/panels/template.py5
-rw-r--r--tests/tests.py4
2 files changed, 5 insertions, 4 deletions
diff --git a/debug_toolbar/panels/template.py b/debug_toolbar/panels/template.py
index ff8cd15..7ff3dc6 100644
--- a/debug_toolbar/panels/template.py
+++ b/debug_toolbar/panels/template.py
@@ -70,13 +70,14 @@ class TemplateDebugPanel(DebugPanel):
temp_layer[key] = '<<languages>>'
# QuerySet would trigger the database: user can run the query from SQL Panel
elif isinstance(value, QuerySet):
- temp_layer[key] = '<<queryset>>'
+ model_name = "%s.%s" % (value.model._meta.app_label, value.model.__name__)
+ temp_layer[key] = '<<queryset of %s>>' % model_name
else:
try:
recording(False)
pformat(value) # this MAY trigger a db query
except SQLQueryTriggered:
- temp_layer[key] = '<<contains queryset>>'
+ temp_layer[key] = '<<triggers database query>>'
else:
temp_layer[key] = value
finally:
diff --git a/tests/tests.py b/tests/tests.py
index 5a52670..c074c23 100644
--- a/tests/tests.py
+++ b/tests/tests.py
@@ -207,8 +207,8 @@ class TemplatePanelTestCase(BaseTestCase):
self.assertEquals(len(sql_panel._queries), 0)
ctx = template_panel.templates[0]['context'][0]
ctx = eval(ctx) # convert back to Python
- self.assertEquals(ctx['queryset'], '<<queryset>>')
- self.assertEquals(ctx['deep_queryset'], '<<contains queryset>>')
+ self.assertEquals(ctx['queryset'], '<<queryset of auth.User>>')
+ self.assertEquals(ctx['deep_queryset'], '<<triggers database query>>')
def module_func(*args, **kwargs):
"""Used by dispatch tests"""