[Erp5-report] r22234 - /erp5/trunk/products/ZSQLCatalog/Query/SimpleQuery.py

[nobody at svn.erp5.org]

Author: jerome
Date: Thu Jul  3 11:12:37 2008
New Revision: 22234

URL: http://svn.erp5.org?rev=22234&view=rev
Log:
don't use [] as default values because it's generally wrong and because sql
worklist generation sometimes passes None

Modified:
    erp5/trunk/products/ZSQLCatalog/Query/SimpleQuery.py

Modified: erp5/trunk/products/ZSQLCatalog/Query/SimpleQuery.py
URL: http://svn.erp5.org/erp5/trunk/products/ZSQLCatalog/Query/SimpleQuery.py?rev=22234&r1=22233&r2=22234&view=diff
==============================================================================
--- erp5/trunk/products/ZSQLCatalog/Query/SimpleQuery.py (original)
+++ erp5/trunk/products/ZSQLCatalog/Query/SimpleQuery.py Thu Jul  3 11:12:37 2008
@@ -153,9 +153,9 @@
                        'select_expression_list': select_expression_list,}
     return sql_expressions
     
-  def asSQLExpression(self, key_alias_dict = None, keyword_search_keys = [],
-                      datetime_search_keys = [], full_text_search_keys = [],
-                      ignore_empty_string = 1, stat__ = 0):
+  def asSQLExpression(self, key_alias_dict=None, keyword_search_keys=None,
+                      datetime_search_keys=None, full_text_search_keys=None,
+                      ignore_empty_string=1, stat__=0):
     """
     Build the sql expressions string
     """
@@ -168,6 +168,13 @@
     search_mode = self.getSearchMode()
     range_value = self.getRange()
     search_key = self.getSearchKey()
+
+    if keyword_search_keys is None:
+      keyword_search_keys = []
+    if datetime_search_keys is None:
+      datetime_search_keys = []
+    if full_text_search_keys is None:
+      full_text_search_keys = []
 
     # key can have an alias definition which we should acquire
     if key_alias_dict is not None: