Fix missing query-actions buttons for queries with date/datetime params.

The param `strings_only=True` caused params with type `time`, `date` and
`datetime` to not be converted to string by `NormalCursorWrapper._decode()`,
and in turn causes the json dumping to fail.

This change keeps `strings_only=True` for every type but datetime, date and
time, enabling conversion to string in those cases.

Fixes #882

Author: jieter

debug_toolbar/panels/sql/tracking.py

@@ -1,5 +1,6 @@
 from __future__ import absolute_import, unicode_literals
 
+import datetime
 import json
 from threading import local
 from time import time
@@ -95,8 +96,10 @@ def _quote_params(self, params):
         return [self._quote_expr(p) for p in params]
 
     def _decode(self, param):
+        # make sure datetime, date and time are converted to string by force_text
+        CONVERT_TYPES = (datetime.datetime, datetime.date, datetime.time)
         try:
-            return force_text(param, strings_only=True)
+            return force_text(param, strings_only=not isinstance(param, CONVERT_TYPES))
         except UnicodeDecodeError:
             return '(encoded string)'
 
@@ -114,7 +117,7 @@ def _record(self, method, sql, params):
             _params = ''
             try:
                 _params = json.dumps([self._decode(p) for p in params])
-            except Exception:
+            except TypeError as e:
                 pass  # object not JSON serializable
 
             template_info = get_template_info()

tests/panels/test_sql.py

@@ -2,6 +2,7 @@
 
 from __future__ import absolute_import, unicode_literals
 
+import datetime
 import unittest
 
 from django.contrib.auth.models import User
@@ -69,6 +70,31 @@ def test_non_ascii_query(self):
         # ensure the panel renders correctly
         self.assertIn('café', self.panel.content)
 
+    def test_param_conversion(self):
+        self.assertEqual(len(self.panel._queries), 0)
+
+        list(
+            User.objects
+                .filter(first_name='Foo')
+                .filter(is_staff=True)
+                .filter(is_superuser=False)
+                .filter(date_joined__hour=13)
+                .filter(date_joined__gte=datetime.datetime(2017, 12, 22, 16, 7, 1))
+                .filter(date_joined__date__gte=datetime.date(2017, 12, 23))
+                .filter(date_joined__time__gte=datetime.time(11, 11, 11))
+        )
+        self.panel.process_response(self.request, self.response)
+        self.panel.generate_stats(self.request, self.response)
+
+        # ensure query was logged
+        self.assertEqual(len(self.panel._queries), 1)
+        query = self.panel._queries[0]
+
+        self.assertEqual(
+            query[1]['params'],
+            '["Foo", true, false, 13, "2017-12-22 16:07:01", "2017-12-23", "11:11:11"]'
+        )
+
     def test_insert_content(self):
         """
         Test that the panel only inserts content after generate_stats and