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

nobody at svn.erp5.org nobody at svn.erp5.org
Fri Jul 9 12:07:32 CEST 2010


Author: luke
Date: Fri Jul  9 12:07:30 2010
New Revision: 37005

URL: http://svn.erp5.org?rev=37005&view=rev
Log:
 - revert 36980, as it gives too much performance penalty

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=37005&r1=37004&r2=37005&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Catalog/CatalogTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Catalog/CatalogTool.py [utf8] Fri Jul  9 12:07:30 2010
@@ -125,12 +125,7 @@ class IndexableObjectWrapper(object):
         for user, roles in localroles.iteritems():
           prefix = 'user:' + user
           for role in roles:
-            if (role in role_dict) and (getUserById(user) is not None
-                # support special case - reindex person for first time, when
-                # it is not returned yet by getUserById, as it is not available
-                # yet in catalog
-                or (self.getPortalType() == 'Person' and
-                  self.getReference() == user)):
+            if (role in role_dict) and (getUserById(user) is not None):
               # If role is monovalued, check if key is a user.
               # If not, continue to index it in roles_and_users table.
               user_role_dict[role] = user




More information about the Erp5-report mailing list