Changeset 9121 for django/trunk/django/contrib/comments/views

Timestamp:

10/05/08 00:10:35 (3 months ago)

Author:

mtredinnick

Message:

Fixed #8963 -- Use the require_POST decorator instead of doing things manually in one of the comment views. Thanks, zgoda.

Files:

• django/trunk/django/contrib/comments/views/comments.py (modified) (3 diffs)

django/trunk/django/contrib/comments/views/comments.py

@@ -8 +8 @@
-from django.template.loader import render_to_string
-from django.utils.html import escape
+from django.views.decorators.http import require_POST
-from django.contrib import comments
-from django.contrib.comments import signals
@@ -29 +30 @@
-    errors a preview template, ``comments/preview.html``, will be rendered.
-    """
-
-    # Require POST
-    if request.method != 'POST':
-        return http.HttpResponseNotAllowed(["POST"])
-
-    # Fill out some initial data fields from an authenticated user, if present
-    data = request.POST.copy()
@@ -117 +113 @@
-    return next_redirect(data, next, comment_done, c=comment._get_pk_val())
-
+post_comment = require_POST(post_comment)
+
-comment_done = confirmation_view(
-    template = "comments/posted.html",