Moved view function name extraction into debug_toolbar.utils.get_name_from_obj

2 files changed, 24 insertions, 10 deletions

diff --git a/debug_toolbar/panels/request_vars.py b/debug_toolbar/panels/request_vars.py
index 60214ae..6e7dd03 100644
--- a/debug_toolbar/panels/request_vars.py
+++ b/debug_toolbar/panels/request_vars.py
@@ -1,6 +1,8 @@
 from django.template.loader import render_to_string
 from django.utils.translation import ugettext_lazy as _
+
 from debug_toolbar.panels import DebugPanel
+from debug_toolbar.utils import get_name_from_obj
 
 class RequestVarsDebugPanel(DebugPanel):
     """
@@ -36,17 +38,15 @@ class RequestVarsDebugPanel(DebugPanel):
         context = self.context.copy()
 
         if self.view_func is not None:
-            module = self.view_func.__module__
-            name = getattr(self.view_func, '__name__', None) or getattr(self.view_func.__class__,'__name__','<unknown>')
-            view_func = '%s.%s' % (module, name)
+            name = get_name_from_obj(self.view_func)
         else:
-            view_func = '<no view>'
+            name = '<no view>'
 
         context.update({
             'get': [(k, self.request.GET.getlist(k)) for k in self.request.GET],
             'post': [(k, self.request.POST.getlist(k)) for k in self.request.POST],
             'cookies': [(k, self.request.COOKIES.get(k)) for k in self.request.COOKIES],
-            'view_func': view_func,
+            'view_func': name,
             'view_args': self.view_args,
             'view_kwargs': self.view_kwargs
         })
diff --git a/debug_toolbar/utils/__init__.py b/debug_toolbar/utils/__init__.py
index 2ce38db..c4dc160 100644
--- a/debug_toolbar/utils/__init__.py
+++ b/debug_toolbar/utils/__init__.py
@@ -47,7 +47,7 @@ def get_template_info(source, context_lines=3):
     line = 0
     upto = 0
     source_lines = []
-    before = during = after = ""
+    # before = during = after = ""
 
     origin, (start, end) = source
     template_source = origin.reload()
@@ -55,9 +55,9 @@ def get_template_info(source, context_lines=3):
     for num, next in enumerate(linebreak_iter(template_source)):
         if start >= upto and end <= next:
             line = num
-            before = template_source[upto:start]
-            during = template_source[start:end]
-            after = template_source[end:next]
+            # before = template_source[upto:start]
+            # during = template_source[start:end]
+            # after = template_source[end:next]
         source_lines.append((num, template_source[upto:next]))
         upto = next
 
@@ -75,4 +75,18 @@ def get_template_info(source, context_lines=3):
     return {
         'name': origin.name,
         'context': context,
-    }
\ No newline at end of file
+    }
+
+def get_name_from_obj(obj):
+    if hasattr(obj, '__name__'):
+        name = obj.__name__
+    elif hasattr(obj, '__class__') and hasattr(obj.__class__, '__name__'):
+        name = obj.__class__.__name__
+    else:
+        name = '<unknown>'
+    
+    if hasattr(obj, '__module__'):
+        module = obj.__module__
+        name = '%s.%s' % (module, name)
+
+    return name
\ No newline at end of file