Removed unneeded Panel.slug attribute.

Author: aaugustin

debug_toolbar/panels/__init__.py

@@ -25,7 +25,6 @@ class Panel(object):
     def __init__(self, toolbar, context={}):
         self.toolbar = toolbar
         self.context.update(context)
-        self.slug = slugify(self.name)
 
     def content(self):
         if self.has_content:
@@ -69,17 +68,13 @@ def enable_instrumentation(self):
     def disable_instrumentation(self):
         pass
 
-    # Store and retrieve stats (shared between panels)
+    # Store and retrieve stats (shared between panels for no good reason)
 
     def record_stats(self, stats):
-        panel_stats = self.toolbar.stats.get(self.slug)
-        if panel_stats:
-            panel_stats.update(stats)
-        else:
-            self.toolbar.stats[self.slug] = stats
+        self.toolbar.stats.setdefault(self.panel_id, {}).update(stats)
 
     def get_stats(self):
-        return self.toolbar.stats.get(self.slug, {})
+        return self.toolbar.stats.get(self.panel_id, {})
 
     # Standard middleware methods
 

tests/test_integration.py

@@ -45,7 +45,7 @@ def _resolve_stats(self, path):
         panel = self.toolbar.get_panel_by_id('RequestPanel')
         panel.process_request(self.request)
         panel.process_response(self.request, self.response)
-        return self.toolbar.stats['request']
+        return panel.get_stats()
 
     def test_url_resolving_positional(self):
         stats = self._resolve_stats('/resolving1/a/b/')