From f0cbc787ab03bae4eb792db267a6898141dc7a01 Mon Sep 17 00:00:00 2001
From: Timo Stollenwerk <timo@zmag.de>
Date: Wed, 7 Oct 2009 20:07:55 +0000
Subject: [PATCH] has_replies function added to get only published comments for
 anon users. elro review fix. refs #9288

svn path=/plone.app.discussion/trunk/; revision=30252
---
 plone/app/discussion/browser/comments.pt |  3 +-
 plone/app/discussion/browser/comments.py | 40 +++++++++++++++++++++---
 2 files changed, 37 insertions(+), 6 deletions(-)

diff --git a/plone/app/discussion/browser/comments.pt b/plone/app/discussion/browser/comments.pt
index c50950c..7c11b90 100644
--- a/plone/app/discussion/browser/comments.pt
+++ b/plone/app/discussion/browser/comments.pt
@@ -4,6 +4,7 @@
                    isAnon view/is_anonymous;
                    canManage view/can_manage;
                    replies python:view.get_replies(canManage);
+                   has_replies python:view.has_replies(canManage);
                    showCommenterImage view/show_commenter_image;
                    errors options/state/getErrors|nothing;
                    wtool context/@@plone_tools/workflow"
@@ -132,7 +133,7 @@
     </div>
 
     <div class="reply"
-         tal:condition="python: isAnon and not isAnonymousDiscussionAllowed and replies">
+         tal:condition="python: isAnon and not isAnonymousDiscussionAllowed and has_replies">
         <form tal:attributes="action view/login_action">
             <input class="standalone"
                    style="margin-bottom: 1.25em;"
diff --git a/plone/app/discussion/browser/comments.py b/plone/app/discussion/browser/comments.py
index 2e95f56..1ee2ea0 100644
--- a/plone/app/discussion/browser/comments.py
+++ b/plone/app/discussion/browser/comments.py
@@ -239,15 +239,35 @@ class CommentsViewlet(ViewletBase, layout.FormWrapper):
         conversation = IConversation(context)
         return conversation.enabled()
 
+    def has_replies(self, workflow_actions=False):
+        """Returns true if there are replies.
+        """
+        try:
+            self.get_replies(workflow_actions).next()
+            return True
+        except StopIteration:
+            return None
+
+
     def get_replies(self, workflow_actions=False):
+        """Returns all replies to a content object.
+
+        If workflow_actions is false, only published
+        comments are returned.
+
+        If workflow actions is true, comments are
+        returned with workflow actions.
+        """
         context = aq_inner(self.context)
         conversation = IConversation(context)
 
-        def replies_with_workflow_actions():
-            # Return dict with workflow actions
-            #context = aq_inner(self.context)
-            wf = getToolByName(context, 'portal_workflow')
+        wf = getToolByName(context, 'portal_workflow')
 
+        # workflow_actions is only true when user
+        # has 'Manage portal' permission
+
+        def replies_with_workflow_actions():
+            # Generator that returns replies dict with workflow actions
             for r in conversation.getThreads():
                 comment_obj = r['comment']
                 # list all possible workflow actions
@@ -257,12 +277,22 @@ class CommentsViewlet(ViewletBase, layout.FormWrapper):
                 r['actions'] = actions
                 yield r
 
+        def published_replies():
+            # Generator that returns replies dict with workflow status.
+            for r in conversation.getThreads():
+                comment_obj = r['comment']
+                workflow_status = wf.getInfoFor(comment_obj, 'review_state')
+                if workflow_status == 'published':
+                    r = r.copy()
+                    r['workflow_status'] = workflow_status
+                    yield r
+
         # Return all direct replies
         if conversation.total_comments > 0:
             if workflow_actions:
                 return replies_with_workflow_actions()
             else:
-                return conversation.getThreads()
+                return published_replies()
         else:
             return None