From 8e9c75a6fc6270ef9e2c8463a0c247ced088bd0a Mon Sep 17 00:00:00 2001
From: Timo Stollenwerk <timo@zmag.de>
Date: Wed, 5 Aug 2009 09:01:14 +0000
Subject: [PATCH] Set fields as class attribute to make captcha formextender
 work again. This is kind of ugly, the fields should be enirely omitted and
 not only hidden.

svn path=/plone.app.discussion/trunk/; revision=28370
---
 plone/app/discussion/browser/comments.py | 40 +++++++++---------------
 1 file changed, 14 insertions(+), 26 deletions(-)

diff --git a/plone/app/discussion/browser/comments.py b/plone/app/discussion/browser/comments.py
index ea2a6cc..d4e0d47 100644
--- a/plone/app/discussion/browser/comments.py
+++ b/plone/app/discussion/browser/comments.py
@@ -54,36 +54,24 @@ class CommentForm(extensible.ExtensibleForm, form.Form):
 
     ignoreContext = True # don't use context to get widget data
     label = _(u"Add a comment")
-
-    @property
-    def fields(self):
-        portal_membership = getToolByName(self.context, 'portal_membership')
-        if portal_membership.isAnonymousUser():
-            return field.Fields(IComment).omit('portal_type',
-                                               '__parent__',
-                                               '__name__',
-                                               'comment_id',
-                                               'mime_type',
-                                               'creator',
-                                               'creation_date',
-                                               'modification_date',
-                                               'author_username',)
-        else:
-            return field.Fields(IComment).omit('portal_type',
-                                               '__parent__',
-                                               '__name__',
-                                               'comment_id',
-                                               'mime_type',
-                                               'creator',
-                                               'creation_date',
-                                               'modification_date',
-                                               'author_username',
-                                               'author_name',
-                                               'author_email',)
+    fields = field.Fields(IComment).omit('portal_type',
+                                         '__parent__',
+                                         '__name__',
+                                         'comment_id',
+                                         'mime_type',
+                                         'creator',
+                                         'creation_date',
+                                         'modification_date',
+                                         'author_username')
 
     def updateWidgets(self):
         super(CommentForm, self).updateWidgets()
         self.widgets['in_reply_to'].mode = interfaces.HIDDEN_MODE
+        portal_membership = getToolByName(self.context, 'portal_membership')
+        if not portal_membership.isAnonymousUser():
+            # XXX: This is ugly. The fields should be omitted, not hidden.
+            self.widgets['author_name'].mode = interfaces.HIDDEN_MODE
+            self.widgets['author_email'].mode = interfaces.HIDDEN_MODE
 
     @button.buttonAndHandler(_(u"Comment"))
     def handleComment(self, action):