@@ -1,6 +1,6 @@ import datetime from django.conf import settings -from django.core.urlresolvers import resolve +from django.core.urlresolvers import resolve, Resolver404 from googleanalytics import Connection from googleanalytics.exception import GoogleAnalyticsClientError @@ -71,9 +71,10 @@ def get_popular_items(model, num=5, days_ago=7, objects = [] for item in data: try: - obj = lookup_func(item.dimension) + url = item.dimension.split('?')[0] + obj = lookup_func(url) objects.append((obj, item.metric)) - except model.DoesNotExist: + except (model.DoesNotExist, Resolver404): pass return objects except GoogleAnalyticsClientError: popular/__init__.py d37dbaea7dd2b97ee349e13d1ebea7e4ae7765b8 James Turk james.p.turk@gmail.com http://github.com/sunlightlabs/django-popular/commit/334749ecbd4fc77c90e90e8cf8973ffd9b1cdb18 334749ecbd4fc77c90e90e8cf8973ffd9b1cdb18 2009-10-29T07:29:49-07:00 2009-10-29T07:29:49-07:00 fix querystring bug and catch Resolver errors a6c9de33bbf00ee5c4c8ae82de04adc91fefcd9c James Turk james.p.turk@gmail.com