[Zope-CMF] Nested Portals? (WAS: CMFBoringContent)

Tres Seaver tseaver@zope.com
Thu, 16 Aug 2001 10:59:28 -0400


marc lindahl wrote:

> 
>>From: "Joachim Werner" <joe@iuveno-net.de>
>>
>>I tried with "real" roles, not local roles. Using either Slave User Folder
>>or nested "normal" user folders. That's why you got the difference I guess
>>...
>>
> 
> Yes, my model was: everyone's a member, and local roles dictate what access
> you have -- organizing the site as folders in the top level of the portal
> for various 'projects' or areas or whatever.
> 
> If someone has a local role of Reviewer, and there's stuff in there to be
> reviewed, it should show up on their list, no??

That was the behavior I expected;  it turns out that the default
workflow was trying to optimize out the need for the catalog query,
and therefore that users who had the "Reviewer" role only locally
wouldn't see the action.

Here is a patch, which I am about to check in:

-------------------- Cut here ---------------------------------
diff -u -r1.5 DefaultWorkflow.py
--- CMFDefault/DefaultWorkflow.py       11 Jun 2001 19:07:27 -0000      1.5
+++ CMFDefault/DefaultWorkflow.py       16 Aug 2001 15:26:20 -0000
@@ -200,13 +200,12 @@
      security.declarePrivate('listGlobalActions')
      def listGlobalActions(self, info):
          '''
-        Allows this workflow to
-        include actions to be displayed in the actions box.
-        Called on every request.
+        Allows this workflow to include actions to be displayed
+        in the actions box.  Called on every request.
+
          Returns the actions to be displayed to the user.
          '''
-        if (info.isAnonymous or not _checkPermission(
-            'Review portal content', info.portal)):
+        if info.isAnonymous:
              return None



-------------------- Cut here ---------------------------------


Tres.
-- 
===============================================================
Tres Seaver                                tseaver@zope.com
Zope Corporation      "Zope Dealers"       http://www.zope.com