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

nobody at svn.erp5.org nobody at svn.erp5.org
Mon Jan 19 17:24:53 CET 2009


Author: romain
Date: Mon Jan 19 17:24:47 2009
New Revision: 25177

URL: http://svn.erp5.org?rev=25177&view=rev
Log:
Do not add security_uid_list if empty.

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=25177&r1=25176&r2=25177&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Catalog/CatalogTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Catalog/CatalogTool.py [utf8] Mon Jan 19 17:24:47 2009
@@ -596,7 +596,10 @@
         if security_uid_list:
           query = ComplexQuery(Query(security_uid=security_uid_list, operator='IN'),
                                query, operator='OR')
-      else:
+      elif security_uid_list:
+        # If security_uid_list is empty, adding it to criterions will only
+        # result in "false or [...]", so avoid useless overhead by not
+        # adding it at all.
         query = Query(security_uid=security_uid_list, operator='IN')
 
       if local_role_column_dict:




More information about the Erp5-report mailing list