Post-process two query attributes

There is no reason for them to be computed when recording, so move them
to SQLPanel.generate_stats().

Author: living180

debug_toolbar/panels/sql/panel.py

@@ -6,6 +6,7 @@
 from django.urls import path
 from django.utils.translation import gettext_lazy as _, ngettext
 
+from debug_toolbar import settings as dt_settings
 from debug_toolbar.forms import SignedDataForm
 from debug_toolbar.panels import Panel
 from debug_toolbar.panels.sql import views
@@ -204,6 +205,8 @@ def generate_stats(self, request, response):
         duplicate_query_groups = defaultdict(list)
 
         if self._queries:
+            sql_warning_threshold = dt_settings.get_config()["SQL_WARNING_THRESHOLD"]
+
             width_ratio_tally = 0
             factor = int(256.0 / (len(self._databases) * 2.5))
             for n, db in enumerate(self._databases.values()):
@@ -261,6 +264,12 @@ def generate_stats(self, request, response):
 
                 if query["sql"]:
                     query["sql"] = reformat_sql(query["sql"], with_toggle=True)
+
+                query["is_slow"] = query["duration"] > sql_warning_threshold
+                query["is_select"] = (
+                    query["raw_sql"].lower().lstrip().startswith("select")
+                )
+
                 query["rgb_color"] = self._databases[alias]["rgb_color"]
                 try:
                     query["width_ratio"] = (query["duration"] / self._sql_time) * 100

debug_toolbar/panels/sql/tracking.py

@@ -7,7 +7,6 @@
 from django.db.backends.utils import CursorWrapper
 from django.utils.encoding import force_str
 
-from debug_toolbar import settings as dt_settings
 from debug_toolbar.utils import get_stack_trace, get_template_info
 
 try:
@@ -191,10 +190,6 @@ def _record(self, method, sql, params):
                 "params": _params,
                 "raw_params": params,
                 "stacktrace": get_stack_trace(skip=2),
-                "is_slow": (
-                    duration > dt_settings.get_config()["SQL_WARNING_THRESHOLD"]
-                ),
-                "is_select": sql.lower().strip().startswith("select"),
                 "template_info": template_info,
             }