[Erp5-report] r19108 - /erp5/trunk/products/ERP5Catalog/CatalogTool.py

[nobody at svn.erp5.org]

Author: nicolas
Date: Wed Feb  6 16:51:33 2008
New Revision: 19108

URL: http://svn.erp5.org?rev=19108&view=rev
Log:
Generate allowRoleAndUsers with Role only if View permission. This patch should reduce entries in roles_and_users table.

Modified:
    erp5/trunk/products/ERP5Catalog/CatalogTool.py

Modified: erp5/trunk/products/ERP5Catalog/CatalogTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Catalog/CatalogTool.py?rev=19108&r1=19107&r2=19108&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Catalog/CatalogTool.py (original)
+++ erp5/trunk/products/ERP5Catalog/CatalogTool.py Wed Feb  6 16:51:33 2008
@@ -150,19 +150,19 @@
                 allowed[user] = 1
               else:
                 allowed['user:' + user] = 1
-            # Added for ERP5 project by JP Smets
-            # The reason why we do not want to keep Owner is because we are
-            # trying to reduce the number of security definitions
-            # However, this is a bad idea if we start to use Owner role
-            # as a kind of bamed Assignee and if we need it for worklists. Therefore
-            # we may sometimes catalog the owner user ID whenever the Owner
-            # has view permission (see getAllowedRolesAndUsers bellow
-            # as well as getViewPermissionOwner method in Base)
-            if role != 'Owner':
-              if withnuxgroups:
-                allowed[user + ':' + role] = 1
-              else:
-                allowed['user:' + user + ':' + role] = 1
+              # Added for ERP5 project by JP Smets
+              # The reason why we do not want to keep Owner is because we are
+              # trying to reduce the number of security definitions
+              # However, this is a bad idea if we start to use Owner role
+              # as a kind of bamed Assignee and if we need it for worklists. Therefore
+              # we may sometimes catalog the owner user ID whenever the Owner
+              # has view permission (see getAllowedRolesAndUsers bellow
+              # as well as getViewPermissionOwner method in Base)
+              if role != 'Owner':
+                if withnuxgroups:
+                  allowed[user + ':' + role] = 1
+                else:
+                  allowed['user:' + user + ':' + role] = 1
         if allowed.has_key('Owner'):
           del allowed['Owner']
         return list(allowed.keys())