Added 'Top referrers' in overview page

Derecho · Derecho · commit 97e2f4cdc68c · 2009-08-03T17:52:33.000+02:00
diff --git a/docs/index.txt b/docs/index.txt
@@ -32,7 +32,7 @@ Once you have installed ``django-request`` you can add it to a django project by
 
 .. note #2:
     Make sure that you got your domain name right in the Sites model.
-    If it's not, the unique visits will not be calculated right.
+    If it's not, the unique visits will not be calculated right, and top referers will not be shown correctly.
 
 django-admin.py
 ===============
diff --git a/request/managers.py b/request/managers.py
@@ -52,6 +52,34 @@ def paths(self, unique=True, count=False, qs=None, limit=None):
         
         return paths
     
+    def referrers(self, unique=True, count=False, qs=None, limit=None):
+        # There is a slight misspelling here, referer should be referrer,
+        # but even the http headers misspell it so I'll leave it here.
+        if not qs:
+            qs = self.all()
+        
+        if count: unique = False
+        
+        if unique:
+            referers = set([request.referer for request in qs.only('referer')])
+        else:
+            referers = [request.referer for request in qs.only('referer')]
+        
+        if count:
+            referer_count = {}
+            for referer in referers:
+                referer_count[referer] = len([None for x in referers if referer == x])
+            
+            referers = [(v, k) for k, v in referer_count.iteritems()]
+            referers.sort()
+            referers.reverse()
+            referers = [(k, v) for v, k in referers]
+        
+        if limit:
+            referers = referers[:limit]
+        
+        return referers
+    
     def year(self, year):
         return self.filter(time__year=year)
     
diff --git a/request/templates/admin/request/overview.html b/request/templates/admin/request/overview.html
@@ -149,5 +149,22 @@
           {% endfor %}
       </table>
     </div>
+
+    <div class="module" style="clear: both; float: left; width: 450px;">
+        <table summary="{% trans "Top referrers" %}" style="width: 100%;">
+            <caption>{% trans "Top referrers" %}</caption>
+            <tr>
+                <th>{% trans "Referrer" %}</th>
+                <th>{% trans "Hits" %}</th>
+            </tr>
+          {% for referrer,hits in top_referrers %}
+            <tr>
+                <td><a href="{{ referrer }}">{{ referrer }}</a></td>
+                <td>{{ hits }}</td>
+            </tr>
+          {% endfor %}
+      </table>
+    </div>
+
 </div>
 {% endblock %}
diff --git a/request/views.py b/request/views.py
@@ -41,6 +41,7 @@ def overview(request):
         'traffic_graph': simplejson.dumps([getattr(importlib.import_module(module_path[:module_path.rindex('.')]), module_path[module_path.rindex('.')+1:], None)(days) for module_path in settings.REQUEST_TRAFFIC_GRAPH_MODULES]),
         'top_paths': Request.objects.paths(count=True, limit=10, qs=Request.objects.filter(response__lt=400)),
         'top_error_paths': Request.objects.paths(count=True, limit=10, qs=(Request.objects.filter(response__gte=400))),
+        'top_referrers': Request.objects.referrers(count=True, limit=10, qs=Request.objects.exclude(referer__startswith=settings.REQUEST_BASE_URL).exclude(referer='')),
         
         'requests_url': '/admin/request/request/',
         'use_hosted_media': settings.REQUEST_USE_HOSTED_MEDIA
