[Erp5-report] r27768 - /erp5/trunk/products/ZSQLCatalog/interfaces/query_catalog.py

[nobody at svn.erp5.org]

Author: vincent
Date: Tue Jun 23 10:30:35 2009
New Revision: 27768

URL: http://svn.erp5.org?rev=27768&view=rev
Log:
Document ignore_empty_string as being a bad practice.

Modified:
    erp5/trunk/products/ZSQLCatalog/interfaces/query_catalog.py

Modified: erp5/trunk/products/ZSQLCatalog/interfaces/query_catalog.py
URL: http://svn.erp5.org/erp5/trunk/products/ZSQLCatalog/interfaces/query_catalog.py?rev=27768&r1=27767&r2=27768&view=diff
==============================================================================
--- erp5/trunk/products/ZSQLCatalog/interfaces/query_catalog.py [utf8] (original)
+++ erp5/trunk/products/ZSQLCatalog/interfaces/query_catalog.py [utf8] Tue Jun 23 10:30:35 2009
@@ -55,6 +55,9 @@
           performed on them.
       ignore_empty_string (boolean)
         If True, values from kw which are empty will be skipped.
+        This parameter should ultimately disapear *and* be disabled by
+        default, as it is bad to ignore parameters based on their value if
+        that value has a meaning in SQL.
       operator (string)
         Used to explicit the logical relation between kw entries.
         It must be a valid ComplexQuery logical operator ('and', 'or').