Merge branch 'fix_templates_panel_issues' of github.com:kezabelle/django-debug-toolbar

* 'fix_templates_panel_issues' of github.com:kezabelle/django-debug-toolbar:
  Fix issue where evaluating `value` might cause side-effects above-and-beyond triggering a DB query. This is most prevalent with objects like Forms, whose default __str__ renders the form, and causes things like full_clean to take place when they otherwise wouldn't. In an ideal world we'd use repr() directly, as it is SUPER fast by comparison. saferepr offers a happier middleground between repr() and pformat().

Author: matthiask

debug_toolbar/panels/templates/panel.py

@@ -3,7 +3,7 @@
 from collections import OrderedDict
 from contextlib import contextmanager
 from os.path import normpath
-from pprint import pformat
+from pprint import pformat, saferepr
 
 from django import http
 from django.conf.urls import url
@@ -126,7 +126,7 @@ def _store_template_info(self, sender, **kwargs):
                         else:
                             try:
                                 recording(False)
-                                force_text(value)  # this MAY trigger a db query
+                                saferepr(value)  # this MAY trigger a db query
                             except SQLQueryTriggered:
                                 temp_layer[key] = '<<triggers database query>>'
                             except UnicodeEncodeError: