Fix comment author link when author is a local user.

4 files changed, 14 insertions, 15 deletions

diff --git a/quoins/controllers.py b/quoins/controllers.py
index bf27ddd..f627c61 100644
--- a/quoins/controllers.py
+++ b/quoins/controllers.py
@@ -335,6 +335,8 @@ class BlogController(TGController):
             u = tg.url(os.path.join(self.path, 'post', str(obj.id)))
         elif isinstance(obj, Media):
             u = tg.url(os.path.join(self.path, 'media', str(obj.post.id), str(obj.name)))
+        elif isinstance(obj, TGUser):
+            u = tg.url(os.path.join(self.path, 'author', str(obj.user_name)))
         return u
 
     def absolute_url(self, obj=None):
@@ -346,6 +348,11 @@ class BlogController(TGController):
             port = ':'+port
         return 'http://%s%s%s'%(tg.config.get('server.webhost'), port, u)
 
+    def comment_author_url(self, comment):
+        if comment.author and (not comment.url):
+            return self.url(comment.author)
+        return comment.url
+
     def get_html(self, data):
         return HTML(data)
 
diff --git a/quoins/model/blog.py b/quoins/model/blog.py
index c1689ee..ee99d83 100644
--- a/quoins/model/blog.py
+++ b/quoins/model/blog.py
@@ -201,14 +201,6 @@ class BaseComment(object):
         return 'Anonymous'
     author_name = property(get_author_name)
         
-    def get_author_url(self):
-        if hasattr(self, 'author') and self.author:
-            return self.author.url
-        if self.url:
-            return self.url
-        return None
-    author_url = property(get_author_url)
-
 class Comment(BaseComment):
     pass
 
diff --git a/quoins/templates/delete_post.html b/quoins/templates/delete_post.html
index e237dad..94df1d3 100644
--- a/quoins/templates/delete_post.html
+++ b/quoins/templates/delete_post.html
@@ -57,10 +57,10 @@
                     </a>
                   </span>
                   &nbsp;
-                  <span class="blog-comment-user" py:if="comment.author_url">
-                    <a href="${comment.author_url}" rel="nofollow">${comment.author_name}</a>
+                  <span class="blog-comment-user" py:if="quoins.comment_author_url(comment)">
+                    <a href="${quoins.comment_author_url(comment)}" rel="nofollow">${comment.author_name}</a>
                   </span>
-                  <span class="blog-comment-user" py:if="not comment.author_url">
+                  <span class="blog-comment-user" py:if="not quoins.comment_author_url(comment)">
                     ${comment.author_name}
                   </span>
                   <br />
diff --git a/quoins/templates/post.html b/quoins/templates/post.html
index 72ee748..d290f3b 100644
--- a/quoins/templates/post.html
+++ b/quoins/templates/post.html
@@ -51,10 +51,10 @@
                     </a>
                   </span>
                   &nbsp;
-                  <span class="blog-comment-user" py:if="comment.author_url">
-                    <a href="${comment.author_url}" rel="nofollow">${comment.author_name}</a>
+                  <span class="blog-comment-user" py:if="quoins.comment_author_url(comment)">
+                    <a href="${quoins.comment_author_url(comment)}" rel="nofollow">${comment.author_name}</a>
                   </span>
-                  <span class="blog-comment-user" py:if="not comment.author_url">
+                  <span class="blog-comment-user" py:if="not quoins.comment_author_url(comment)">
                     ${comment.author_name}
                   </span>
                   <br />
@@ -62,7 +62,7 @@
                      ${comment.created.strftime('%B %d, %Y at %I:%M %p')}
                   </span>
 
-                  <span py:if="tg.identity and 'blog-post' in [x.permission_name for x in tg.identity.user.permissions]">
+                  <span py:if="'blog-post' in [x.permission_name for x in tg.identity.user.permissions]">
                     &nbsp; &nbsp; 
                     <a href="${quoins.url('/delete_comment/%s'%comment.id)}">
                       Delete comment