Normalized, slugified, and made tags unique. Closes #1

generators.py

@@ -161,11 +161,11 @@ class TagsContentGenerator(ListingContentGenerator):
 
   @classmethod
   def get_resource_list(cls, post):
-    return post.tags
+    return post.normalized_tags
 
   @classmethod
   def _filter_query(cls, resource, q):
-    q.filter('tags =', resource)
+    q.filter('normalized_tags =', resource)
 generator_list.append(TagsContentGenerator)
 
 

handlers.py

@@ -12,7 +12,7 @@
 class PostForm(djangoforms.ModelForm):
   class Meta:
     model = models.BlogPost
-    exclude = [ 'path', 'published', 'updated', 'deps' ]
+    exclude = [ 'path', 'published', 'updated', 'deps', 'normalized_tags' ]
 
 
 def with_post(fun):

models.py

@@ -15,11 +15,15 @@ class BlogPost(db.Model):
   path = db.StringProperty()
   title = db.StringProperty(required=True, indexed=False)
   body = db.TextProperty(required=True)
-  tags = db.StringListProperty()
+  tags = aetycoon.SetProperty(basestring, indexed=False)
   published = db.DateTimeProperty(auto_now_add=True)
   updated = db.DateTimeProperty(auto_now=True)
   deps = aetycoon.PickleProperty()
 
+  @aetycoon.TransformProperty(tags)
+  def normalized_tags(tags):
+    return list(set(utils.slugify(x.lower()) for x in tags))
+
   @property
   def summary(self):
     """Returns a summary of the blog post."""

themes/default/listing.html

@@ -7,7 +7,7 @@ <h2><a href="{{post.path}}">{{post.title|escape}}</a></h2>
       Posted by {{config.author_name}}
       {% if post.tags %}
         | Filed under
-        {% for tag in post.tags %}
+        {% for tag in post.normalized_tags %}
           <a href="/tag/{{tag|escape}}">{{tag|escape}}</a>{% if not forloop.last %},{% endif %}
         {% endfor %}
       {% endif %}

themes/default/post.html

@@ -6,7 +6,7 @@ <h2>{{post.title|escape}}</h2>
       Posted by {{config.author_name}}
       {% if post.tags %}
         | Filed under
-        {% for tag in post.tags %}
+        {% for tag in post.normalized_tags %}
           <a href="/tag/{{tag|escape}}">{{tag|escape}}</a>{% if not forloop.last %},{% endif %}
         {% endfor %}
       {% endif %}