Added the generic foreign key query trick to the tag_detail page, try…

…ing to reduce queries there as well. Fixes #20.

coltrane/views.py

@@ -144,8 +144,8 @@ def tag_detail(request, tag):
             raise Http404
 
     # Pull all the items with that tag.
-    taggeditem_list = tag.items.all()
-    
+    taggeditem_list = TaggedItem.objects.filter(tag=tag).fetch_generic_relations()
+
     # Loop through the tagged items and return just the items
     object_list = [i.object for i in taggeditem_list if getattr(i.object, 'pub_date', False)]
 

tagging/managers.py

@@ -4,7 +4,6 @@
 """
 from django.contrib.contenttypes.models import ContentType
 from django.db import models
-
 from tagging.models import Tag, TaggedItem
 
 class ModelTagManager(models.Manager):

tagging/models.py

@@ -12,7 +12,7 @@
 from django.db import connection, models
 from django.db.models.query import QuerySet
 from django.utils.translation import ugettext_lazy as _
-
+from coltrane.managers import GFKQuerySet
 from tagging import settings
 from tagging.utils import calculate_cloud, get_tag_list, get_queryset_and_model, parse_tag_input
 from tagging.utils import LOGARITHMIC
@@ -277,6 +277,9 @@ class TaggedItemManager(models.Manager):
           Now that the queryset-refactor branch is in the trunk, this can be
           tidied up significantly.
     """
+    def get_query_set(self):
+        return GFKQuerySet(self.model)
+
     def get_by_model(self, queryset_or_model, tags):
         """
         Create a ``QuerySet`` containing instances of the specified
@@ -474,7 +477,6 @@ class TaggedItem(models.Model):
     content_type = models.ForeignKey(ContentType, verbose_name=_('content type'))
     object_id    = models.PositiveIntegerField(_('object id'), db_index=True)
     object       = generic.GenericForeignKey('content_type', 'object_id')
-
     objects = TaggedItemManager()
 
     class Meta: