Ticket #11549: comments_patch.diff

django/contrib/comments/views/comments.py

@@ -22 +22 @@
         if settings.DEBUG:
             self.content = render_to_string("comments/400-debug.html", {"why": why})
 
-def post_comment(request, next=None):
+def post_comment(request, next=None, error_page=None):
     """
     Post a comment.
 
     HTTP POST is required. If ``POST['submit'] == "preview"`` or if there are
     errors a preview template, ``comments/preview.html``, will be rendered.
+    
+    If error_page is given, error_page['template'] will
+    be used as the template instead of the preview page, and the key/value
+    pairs of error_page['data'] will be used as appended to the context.
     """
     # Fill out some initial data fields from an authenticated user, if present
     data = request.POST.copy()
@@ -79 +83 @@
             "comments/%s_preview.html" % model._meta.app_label,
             "comments/preview.html",
         ]
-        return render_to_response(
-            template_list, {
+        data_dict = {
                 "comment" : form.data.get("comment", ""),
                 "form" : form,
                 "next": next,
-            },
+            }
+        if form.errors and error_page:
+            # If the user has passed a custom error page, take its template
+            # and data dict and use those instead of the preview page
+            if error_page.has_key('template'):
+                template_list = [error_page['template']] + template_list
+            try:
+                for (key, value) in error_page['data'].items():
+                    data_dict[key] = value
+            except TypeError: pass
+        return render_to_response(
+            template_list,
+            data_dict,
             RequestContext(request, {})
         )
 

docs/ref/contrib/comments/custom.txt

@@ -118 +118 @@
 
 .. _custom-comment-app-api:
 
+Customizing which template renders the errors
+=============================================
+
+The ``post_comment`` view in ``contrib/comments/views/comments.py`` can take an
+extra "error_page" argument which conatins a dict contains another template
+location and extra context data::
+
+    error_page = {
+        'template': 'your_template_directory/template_name.html',
+        'data': {
+            'key1': value1,
+            'key2: vaue 2,
+            },
+    }
+
+
+This will make django load the specified template instead of the default
+preview one. This allows you to keep the user on the same page if they forget
+one of the comments fields by defining a wrapper around the ``post_comment``
+view which you point ``comments/post`` url to.
+
 Custom comment app API
 ======================