From 2ae4f2443bdf48a26be09b126b7720771bd6d477 Mon Sep 17 00:00:00 2001 From: Matthijs Kooijman Date: Fri, 31 Oct 2008 20:51:54 +0100 Subject: [PATCH] Disallow staff to set the markup of new comments. --- influences/forms.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/influences/forms.py b/influences/forms.py index 7a222e2..b9416f9 100644 --- a/influences/forms.py +++ b/influences/forms.py @@ -22,17 +22,22 @@ class AdminInfluenceCommentForm(ThreadedCommentForm): class Meta(ThreadedCommentForm.Meta): fields = ThreadedCommentForm.Meta.fields + ('is_public',) + exclude = ('markup', ) class AdminPrivateInfluenceCommentForm(ThreadedCommentForm): def __init__(self, *args, **kwargs): super(AdminPrivateInfluenceCommentForm, self).__init__(*args, **kwargs) self.instance.is_public = False + class Meta(ThreadedCommentForm.Meta): + exclude = ('markup', ) + def get_influence_comment_form(is_staff, reply_to): """ Gets the form class that a user can use to reply to the given comment. reply_to can be None, in which case the form class for a new comment is returned. """ - allow_markup = allow_private = is_staff + allow_markup = False + allow_private = is_staff if reply_to: allow_public = reply_to.is_public else: @@ -43,9 +48,9 @@ def _get_influence_comment_form(allow_markup, allow_public, allow_private): """ Internal wrapper that selects the right form class depending on the given options. Should not be called directly, call get_influence_comment_form instead. """ - if allow_markup and allow_public and allow_private: + if not allow_markup and allow_public and allow_private: return AdminInfluenceCommentForm - elif allow_markup and not allow_public and allow_private: + elif not allow_markup and not allow_public and allow_private: return AdminPrivateInfluenceCommentForm elif not allow_markup and allow_public and not allow_private: return InfluenceCommentForm -- 2.30.2