[Erp5-report] r25885 - /erp5/trunk/products/ZSQLCatalog/ColumnMap.py

[nobody at svn.erp5.org]

Author: vincent
Date: Thu Mar  5 16:01:06 2009
New Revision: 25885

URL: http://svn.erp5.org?rev=25885&view=rev
Log:
Remove this log, it is (currently) wrong.
 - providing pre-mapped column to catalog is the only way to make a mapping explicit when mapping fails because of ambiguous choice
   -> it must not be discouraged
 - having in catalog configuration pre-mapped columns (sql_search_result_keys) is currently unavoidable
   -> it must not trigger a warning

Modified:
    erp5/trunk/products/ZSQLCatalog/ColumnMap.py

Modified: erp5/trunk/products/ZSQLCatalog/ColumnMap.py
URL: http://svn.erp5.org/erp5/trunk/products/ZSQLCatalog/ColumnMap.py?rev=25885&r1=25884&r2=25885&view=diff
==============================================================================
--- erp5/trunk/products/ZSQLCatalog/ColumnMap.py [utf8] (original)
+++ erp5/trunk/products/ZSQLCatalog/ColumnMap.py [utf8] Thu Mar  5 16:01:06 2009
@@ -106,7 +106,6 @@
     if '.' in column:
       # Assuming the part before the dot is a real table name, not an alias.
       table, column = column.split('.')
-      LOG('ColumnMap', 100, 'Column %r is pre-mapped. This use is strongly discouraged.' % (raw_column, ))
     else:
       table = None