Ticket #11100: r10753_get_comment_permalink.diff

django/contrib/comments/templatetags/comments.py

@@ -252 +252 @@
     """
     return comments.get_form_target()
 
+#@register.simple_tag
+def get_comment_permalink(comment, anchor_pattern=None):
+    """
+    Get the permalink for a comment, optionally specifying the format of the
+    named anchor to be appended to the end of the URL.
+    
+    Example::
+        {{ get_comment_permalink comment "#c%(id)s-by-%(user_name)s" }}
+    """
+    
+    if anchor_pattern:
+        return comment.get_absolute_url(anchor_pattern)
+    return comment.get_absolute_url()
+
 register.tag(get_comment_count)
 register.tag(get_comment_list)
 register.tag(get_comment_form)
 register.tag(render_comment_form)
 register.simple_tag(comment_form_target)
+register.simple_tag(get_comment_permalink)
+ No newline at end of file

docs/ref/contrib/comments/index.txt

@@ -104 +104 @@
 see :ref:`the comment model documentation <ref-contrib-comments-models>` for
 details.
 
+.. templatetag:: get_comment_permalink
+
+Linking to comments
+-------------------
+
+To provide a permalink to a specific comment, use :ttag:`get_comment_permalink`::
+    
+    {% get_comment_permalink comment_obj [format_string] %}
+
+By default, the named anchor that will be appended to the URL will be the letter
+'c' followed by the comment id, for example 'c82'. You may specify a custom
+format string if you wish to override this behavior::
+
+    {% get_comment_permalink comment "#c%(id)s-by-%(user_name)s"%}
+
+The format string is a standard python format string. Valid mapping keys
+include any attributes of the comment object.
+
+Regardless of whether you specify a custom anchor pattern, you must supply a
+matching named anchor at a suitable place in your template.
+
+For example::
+
+    {% for comment in comment_list %}
+        <a name="c{{ comment.id }}"></a>
+        <a href="{% get_comment_permalink comment %}">
+            permalink for comment #{{ forloop.counter }}
+        </a>
+        ...
+    {% endfor %}
+    
+.. warning::
+
+    There's a known bug in Safari / webkit which causes the named anchor to be
+    forgotten following a redirect. The practical impact for comments is that
+    the Safari/webkit browsers will arrive at the correct page but will not
+    scroll to the named anchor.
+
 .. templatetag:: get_comment_count
 
 Counting comments

tests/regressiontests/comment_tests/tests/templatetag_tests.py

@@ -63 +63 @@
 
     def testGetCommentListFromObject(self):
         self.testGetCommentList("{% get_comment_list for a as cl %}")
+        
+    def testGetCommentPermalink(self):
+        self.createSomeComments()
+        t = "{% load comments %}{% get_comment_list for comment_tests.author a.id as cl %}"
+        t += "{% get_comment_permalink cl.0 %}"
+        ctx, out = self.render(t, a=Author.objects.get(pk=1))
+        self.assertEqual(out, "/cr/13/1/#c2")
+
+    def testGetCommentPermalinkFormatted(self):
+        self.createSomeComments()
+        t = "{% load comments %}{% get_comment_list for comment_tests.author a.id as cl %}"
+        t += "{% get_comment_permalink cl.0 '#c%(id)s-by-%(user_name)s' %}"
+        ctx, out = self.render(t, a=Author.objects.get(pk=1))
+        self.assertEqual(out, "/cr/13/1/#c2-by-Joe Somebody")
+
+