[CMF-checkins] CVS: Products/CMFCore/interfaces - portal_discussion.py:1.9

Yvo Schubbe y.2004_ at wcm-solutions.de
Tue Jul 13 14:21:54 EDT 2004


Update of /cvs-repository/Products/CMFCore/interfaces
In directory cvs.zope.org:/tmp/cvs-serv16803/CMFCore/interfaces

Modified Files:
	portal_discussion.py 
Log Message:
Merged yuppie-collector119and201-branch:
- Fixed acquisition bug in getDiscussionFor. (Collector #119)
- Fixed acquisition bug in overrideDiscussionFor. (Collector #201)
- Some related cleanup.


=== Products/CMFCore/interfaces/portal_discussion.py 1.8 => 1.9 ===
--- Products/CMFCore/interfaces/portal_discussion.py:1.8	Mon Apr 26 08:34:10 2004
+++ Products/CMFCore/interfaces/portal_discussion.py	Tue Jul 13 14:21:24 2004
@@ -24,19 +24,23 @@
     """
     id = Attribute('id', 'Must be set to "portal_discussion"')
 
-    #getDiscussionFor__roles__ = None
     def getDiscussionFor(content):
-        """
-            Find / create the DiscussionItemContainer for 'content'.
+        """ Get DiscussionItemContainer for content, create it if necessary.
+
+        Permission -- Always available
+
+        Returns -- DiscussionItemContainer object
         """
 
-    #isDiscussionAllowedFor__roles__ = None
     def isDiscussionAllowedFor(content):
-        """
-            Return a boolean indicating whether discussion is
-            allowed for the specified content;  this may be looked
-            up via an object-specific value, or by place, or from
-            a site-wide policy.
+        """ Get boolean indicating whether discussion is allowed for content.
+
+        This may be looked up via an object-specific value, or by place, or
+        from a site-wide policy.
+
+        Permission -- Always available
+
+        Returns -- Boolean value
         """
 
 
@@ -44,11 +48,12 @@
     """ Links content to discussions.
     """
 
-    #getDiscussionFor__roles__ = None
     def overrideDiscussionFor(content, allowDiscussion):
-        """
-            if 'allowDiscussion' is None, then clear any overridden
-            setting for discussability, letting the site's default
-            policy apply.  Otherwise, set the override to match
-            the boolean equivalent of 'allowDiscussion'.
+        """ Override discussability for the given object or clear the setting.
+
+        If 'allowDiscussion' is None, then clear any overridden setting for
+        discussability, letting the site's default policy apply.  Otherwise,
+        set the override to match the boolean equivalent of 'allowDiscussion'.
+
+        Permission -- Always available
         """



More information about the CMF-checkins mailing list