From 913f3a8c1165c6ea3f7438a8162b74f261c9663a Mon Sep 17 00:00:00 2001
From: Jean-Paul Smets <jp@nexedi.com>
Date: Sat, 21 Jan 2006 01:55:21 +0000
Subject: [PATCH] Added some comments to define placeholders for group support

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@5216 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5Catalog/CatalogTool.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/product/ERP5Catalog/CatalogTool.py b/product/ERP5Catalog/CatalogTool.py
index 2b900d4ae0..30e537f716 100755
--- a/product/ERP5Catalog/CatalogTool.py
+++ b/product/ERP5Catalog/CatalogTool.py
@@ -364,7 +364,10 @@ class CatalogTool (UniqueObject, ZCatalog, CMFCoreCatalogTool, ActiveObject):
 
         # Patch for ERP5 by JP Smets in order
         # to implement worklists and search of local roles
+        # This allows for displaying results based on local roles
+        # rather than based on view permissions
         if kw.has_key('local_roles'):
+          # XXX user is not enough - we should also include groups of the user
           user = _getAuthenticatedUser(self)
           # Only consider local_roles if it is not empty
           if kw['local_roles'] != '' and  kw['local_roles'] != [] and  kw['local_roles'] is not None:
@@ -402,7 +405,10 @@ class CatalogTool (UniqueObject, ZCatalog, CMFCoreCatalogTool, ActiveObject):
 
         # Patch for ERP5 by JP Smets in order
         # to implement worklists and search of local roles
+        # This allows for counting results based on local roles
+        # rather than based on view permissions
         if kw.has_key('local_roles'):
+          # XXX user is not enough - we should also include groups of the user
           user = _getAuthenticatedUser(self)
           # Only consider local_roles if it is not empty
           if kw['local_roles'] != '' and  kw['local_roles'] != [] and  kw['local_roles'] is not None:
-- 
2.30.9