pep8

Author: dcramer

debug_toolbar/middleware.py

@@ -5,7 +5,6 @@
 import thread
 
 from django.conf import settings
-from django.conf.urls.defaults import include, patterns
 from django.http import HttpResponseRedirect
 from django.shortcuts import render_to_response
 from django.utils.encoding import smart_unicode
@@ -16,6 +15,7 @@
 
 _HTML_TYPES = ('text/html', 'application/xhtml+xml')
 
+
 def replace_insensitive(string, target, replacement):
     """
     Similar to string.replace() but is case insensitive
@@ -25,9 +25,10 @@ def replace_insensitive(string, target, replacement):
     index = no_case.rfind(target.lower())
     if index >= 0:
         return string[:index] + replacement + string[index + len(target):]
-    else: # no results so return the original string
+    else:  # no results so return the original string
         return string
 
+
 class DebugToolbarMiddleware(object):
     """
     Middleware to set up Debug Toolbar on incoming request and render toolbar
@@ -43,10 +44,10 @@ def __init__(self):
         self._urlconfs = {}
 
         # Set method to use to decide to show toolbar
-        self.show_toolbar = self._show_toolbar # default
+        self.show_toolbar = self._show_toolbar  # default
 
         # The tag to attach the toolbar to
-        self.tag= u'</body>'
+        self.tag = u'</body>'
 
         if hasattr(settings, 'DEBUG_TOOLBAR_CONFIG'):
             show_toolbar_callback = settings.DEBUG_TOOLBAR_CONFIG.get(

debug_toolbar/panels/__init__.py

@@ -9,59 +9,59 @@ class DebugPanel(object):
     """
     # name = 'Base'
     # template = 'debug_toolbar/panels/base.html'
-    has_content = False # If content returns something, set to true in subclass
-    
+    has_content = False  # If content returns something, set to true in subclass
+
     # We'll maintain a local context instance so we can expose our template
     # context variables to panels which need them:
     context = {}
-    
+
     # Panel methods
     def __init__(self, context={}):
         self.context.update(context)
         self.slug = slugify(self.name)
-    
+
     def dom_id(self):
         return 'djDebug%sPanel' % (self.name.replace(' ', ''))
-    
+
     def nav_title(self):
         """Title showing in toolbar"""
         raise NotImplementedError
-    
+
     def nav_subtitle(self):
         """Subtitle showing until title in toolbar"""
         return ''
-    
+
     def title(self):
         """Title showing in panel"""
         raise NotImplementedError
-    
+
     def url(self):
         raise NotImplementedError
-    
+
     def content(self):
         if self.has_content:
             context = self.context.copy()
             context.update(self.get_stats())
             return render_to_string(self.template, context)
-    
+
     def record_stats(self, stats):
         toolbar = DebugToolbarMiddleware.get_current()
         panel_stats = toolbar.stats.get(self.slug)
         if panel_stats:
             panel_stats.update(stats)
         else:
             toolbar.stats[self.slug] = stats
-    
+
     def get_stats(self):
         toolbar = DebugToolbarMiddleware.get_current()
         return toolbar.stats.get(self.slug, {})
-    
+
     # Standard middleware methods
     def process_request(self, request):
         pass
-    
+
     def process_view(self, request, view_func, view_args, view_kwargs):
         pass
-    
+
     def process_response(self, request, response):
         pass

debug_toolbar/panels/cache.py

@@ -6,12 +6,13 @@
 from django.utils.translation import ugettext_lazy as _
 from debug_toolbar.panels import DebugPanel
 
+
 class CacheStatTracker(BaseCache):
     """A small class used to track cache calls."""
     def __init__(self, cache):
         self.cache = cache
         self.reset()
-    
+
     def reset(self):
         self.calls = []
         self.hits = 0
@@ -21,11 +22,11 @@ def reset(self):
         self.get_many = 0
         self.deletes = 0
         self.total_time = 0
-    
+
     def _get_func_info(self):
         stack = inspect.stack()[2]
         return (stack[1], stack[2], stack[3], stack[4])
-    
+
     def get(self, key, default=None):
         t = time.time()
         value = self.cache.get(key, default)
@@ -38,23 +39,23 @@ def get(self, key, default=None):
         self.gets += 1
         self.calls.append((this_time, 'get', (key,), self._get_func_info()))
         return value
-    
+
     def set(self, key, value, timeout=None):
         t = time.time()
         self.cache.set(key, value, timeout)
         this_time = time.time() - t
         self.total_time += this_time * 1000
         self.sets += 1
         self.calls.append((this_time, 'set', (key, value, timeout), self._get_func_info()))
-    
+
     def delete(self, key):
         t = time.time()
         self.cache.delete(key)
         this_time = time.time() - t
         self.total_time += this_time * 1000
         self.deletes += 1
         self.calls.append((this_time, 'delete', (key,), self._get_func_info()))
-    
+
     def get_many(self, keys):
         t = time.time()
         results = self.cache.get_many(keys)
@@ -68,14 +69,15 @@ def get_many(self, keys):
                 self.hits += 1
         self.calls.append((this_time, 'get_many', (keys,), self._get_func_info()))
 
+
 class CacheDebugPanel(DebugPanel):
     """
     Panel that displays the cache statistics.
     """
     name = 'Cache'
     template = 'debug_toolbar/panels/cache.html'
     has_content = True
-    
+
     def __init__(self, *args, **kwargs):
         super(CacheDebugPanel, self).__init__(*args, **kwargs)
         # This is hackish but to prevent threading issues is somewhat needed
@@ -85,16 +87,16 @@ def __init__(self, *args, **kwargs):
         else:
             self.cache = CacheStatTracker(cache.cache)
             cache.cache = self.cache
-    
+
     def nav_title(self):
         return _('Cache: %.2fms') % self.cache.total_time
-    
+
     def title(self):
         return _('Cache Usage')
-    
+
     def url(self):
         return ''
-    
+
     def process_response(self, request, response):
         self.record_stats({
             'cache_calls': len(self.cache.calls),

debug_toolbar/panels/headers.py

@@ -32,21 +32,21 @@ class HeaderDebugPanel(DebugPanel):
         'SERVER_PROTOCOL',
         'SERVER_SOFTWARE',
     )
-    
+
     def nav_title(self):
         return _('HTTP Headers')
-    
+
     def title(self):
         return _('HTTP Headers')
-    
+
     def url(self):
         return ''
-    
+
     def process_request(self, request):
         self.headers = dict(
             [(k, request.META[k]) for k in self.header_filter if k in request.META]
         )
-    
+
     def process_response(self, request, response):
         self.record_stats({
             'headers': self.headers

debug_toolbar/panels/logger.py

@@ -13,16 +13,16 @@ def __init__(self):
         if threading is None:
             raise NotImplementedError("threading module is not available, \
                 the logging panel cannot be used without it")
-        self.records = {} # a dictionary that maps threads to log records
-    
+        self.records = {}  # a dictionary that maps threads to log records
+
     def add_record(self, record, thread=None):
         # Avoid logging SQL queries since they are already in the SQL panel
         # TODO: Make this check whether SQL panel is enabled
         if record.get('channel', '') == 'django.db.backends':
             return
-        
+
         self.get_records(thread).append(record)
-    
+
     def get_records(self, thread=None):
         """
         Returns a list of records for the provided thread, of if none is provided,
@@ -33,7 +33,7 @@ def get_records(self, thread=None):
         if thread not in self.records:
             self.records[thread] = []
         return self.records[thread]
-    
+
     def clear_records(self, thread=None):
         if thread is None:
             thread = threading.currentThread()
@@ -45,7 +45,7 @@ class ThreadTrackingHandler(logging.Handler):
     def __init__(self, collector):
         logging.Handler.__init__(self)
         self.collector = collector
-    
+
     def emit(self, record):
         record = {
             'message': record.getMessage(),
@@ -75,7 +75,7 @@ class LogbookThreadTrackingHandler(logbook.handlers.Handler):
         def __init__(self, collector):
             logbook.handlers.Handler.__init__(self, bubble=True)
             self.collector = collector
-        
+
         def emit(self, record):
             record = {
                 'message': record.message,
@@ -86,37 +86,37 @@ def emit(self, record):
                 'channel': record.channel,
             }
             self.collector.add_record(record)
-    
-    
+
     logbook_handler = LogbookThreadTrackingHandler(collector)
-    logbook_handler.push_application()        # register with logbook
+    logbook_handler.push_application()  # register with logbook
+
 
 class LoggingPanel(DebugPanel):
     name = 'Logging'
     template = 'debug_toolbar/panels/logger.html'
     has_content = True
-    
+
     def process_request(self, request):
         collector.clear_records()
-    
+
     def process_response(self, request, response):
         records = self.get_and_delete()
         self.record_stats({'records': records})
-    
+
     def get_and_delete(self):
         records = collector.get_records()
         collector.clear_records()
         return records
-    
+
     def nav_title(self):
         return _("Logging")
-    
+
     def nav_subtitle(self):
         # FIXME l10n: use ngettext
         return "%s message%s" % (len(collector.get_records()), (len(collector.get_records()) == 1) and '' or 's')
-    
+
     def title(self):
         return _('Log Messages')
-    
+
     def url(self):
         return ''