[Erp5-report] r12861 - /erp5/trunk/products/ERP5Catalog/CatalogTool.py
nobody at svn.erp5.org
nobody at svn.erp5.org
Mon Feb 19 09:58:51 CET 2007
Author: romain
Date: Mon Feb 19 09:58:49 2007
New Revision: 12861
URL: http://svn.erp5.org?rev=12861&view=rev
Log:
Revert previous modification.
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=12861&r1=12860&r2=12861&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Catalog/CatalogTool.py (original)
+++ erp5/trunk/products/ERP5Catalog/CatalogTool.py Mon Feb 19 09:58:49 2007
@@ -134,10 +134,18 @@
allowed[user] = 1
else:
allowed['user:' + user] = 1
- 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 could be a bad idea if we start to use Owner role
+ # as a kind of Assignee and if we need it for worklists.
+ 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())
class RelatedBaseCategory(Method):
@@ -407,7 +415,12 @@
# Local roles now has precedence (since it comes from a WorkList)
for user_or_group in allowedRolesAndUsers:
for role in local_roles:
- new_allowedRolesAndUsers.append('%s:%s' % (user_or_group, role))
+ if role == "Owner":
+ # This is for now only a placeholder to handle the case of Owner
+ # which may not be supported (see above comment arround line 135
+ new_allowedRolesAndUsers.append('%s:%s' % (user_or_group, role))
+ else:
+ new_allowedRolesAndUsers.append('%s:%s' % (user_or_group, role))
allowedRolesAndUsers = new_allowedRolesAndUsers
return allowedRolesAndUsers
More information about the Erp5-report
mailing list