[Erp5-report] r25971 - /erp5/trunk/products/ZSQLCatalog/SQLExpression.py

[nobody at svn.erp5.org]

Author: vincent
Date: Wed Mar 11 14:10:29 2009
New Revision: 25971

URL: http://svn.erp5.org?rev=25971&view=rev
Log:
Add a special case for SearchableText: when there are mustiple MATCH subqueries, sum up all scores. This used to fail and should be the most sensible default behaviour.

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

Modified: erp5/trunk/products/ZSQLCatalog/SQLExpression.py
URL: http://svn.erp5.org/erp5/trunk/products/ZSQLCatalog/SQLExpression.py?rev=25971&r1=25970&r2=25971&view=diff
==============================================================================
--- erp5/trunk/products/ZSQLCatalog/SQLExpression.py [utf8] (original)
+++ erp5/trunk/products/ZSQLCatalog/SQLExpression.py [utf8] Wed Mar 11 14:10:29 2009
@@ -299,14 +299,18 @@
       for alias, column in sql_expression.getSelectDict().iteritems():
         existing_value = result.get(alias)
         if existing_value not in (None, column):
-          message = '%r is a known alias for column %r, can\'t alias it now to column %r' % (alias, existing_value, column)
-          if DEBUG:
-            message = message + '. I was created by %r, and I am working on %r (%r) out of [%s]' % (
-              self.query,
-              sql_expression,
-              sql_expression.query,
-              ', '.join('%r (%r)' % (x, x.query) for x in self.sql_expression_list))
-          raise ValueError, message
+          if alias == 'SearchableText':
+            # Custom conflict resolution
+            column = '%s + %s' % (existing_value, column)
+          else:
+            message = '%r is a known alias for column %r, can\'t alias it now to column %r' % (alias, existing_value, column)
+            if DEBUG:
+              message = message + '. I was created by %r, and I am working on %r (%r) out of [%s]' % (
+                self.query,
+                sql_expression,
+                sql_expression.query,
+                ', '.join('%r (%r)' % (x, x.query) for x in self.sql_expression_list))
+            raise ValueError, message
         result[alias] = column
     return result