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

[nobody at svn.erp5.org]

Author: vincent
Date: Mon Aug  6 09:28:35 2007
New Revision: 15480

URL: http://svn.erp5.org?rev=15480&view=rev
Log:
Add a warning about suspicious code.

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=15480&r1=15479&r2=15480&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Catalog/CatalogTool.py (original)
+++ erp5/trunk/products/ERP5Catalog/CatalogTool.py Mon Aug  6 09:28:35 2007
@@ -475,6 +475,8 @@
                                   "deprecated. Please update your catalog "\
                                   "business template."
       if allowedRolesAndUsers:
+        # XXX: What with this string transformation ?! Souldn't it be done in
+        # dtml instead ?
         allowedRolesAndUsers = ["'%s'" % (role, ) for role in allowedRolesAndUsers]
         security_uid_list = [x.uid for x in method(security_roles_list = allowedRolesAndUsers)]
       else: