Merge pull request #191 from plone/180_view_permissions

#180 Added 'View comments' permission

news/180.feature

@@ -0,0 +1 @@
+- Added 'View comments' permission. [@razvanMiu]

plone/app/discussion/browser/configure.zcml

@@ -117,7 +117,7 @@
         manager="plone.app.layout.viewlets.interfaces.IBelowContent"
         view="plone.app.layout.globals.interfaces.IViewView"
         class=".comments.CommentsViewlet"
-        permission="zope2.View"
+        permission="plone.app.discussion.ViewComments"
         />
 
     <!-- Conversation view -->

plone/app/discussion/permissions.zcml

@@ -4,6 +4,11 @@
 
     <!-- custom permissions are defined here -->
 
+    <permission
+        id="plone.app.discussion.ViewComments"
+        title="View comments"
+        />
+
     <permission
         id="plone.app.discussion.ReviewComments"
         title="Review comments"

plone/app/discussion/profiles/default/metadata.xml

@@ -1,5 +1,5 @@
 <metadata>
- <version>1003</version>
+ <version>1004</version>
  <dependencies>
   <dependency>profile-plone.resource:default</dependency>
   <dependency>profile-plone.app.registry:default</dependency>

plone/app/discussion/profiles/default/rolemap.xml

@@ -1,6 +1,9 @@
 <?xml version="1.0"?>
 <rolemap>
     <permissions>
+        <permission name="View comments" acquire="True">
+            <role name="Anonymous"/>
+        </permission>
         <permission name="Review comments" acquire="True">
             <role name="Manager"/>
             <role name="Site Administrator"/>

plone/app/discussion/upgrades.zcml

@@ -80,6 +80,13 @@
       title="Update controlpanel icon"
       import_steps="controlpanel" />
 
+    <genericsetup:upgradeDepends
+      source="1003"
+      destination="1004"
+      profile="plone.app.discussion:default"
+      title="Add 'View comments' permission"
+      import_steps="rolemap" />
+
 
 
 </configure>