handle postgresql bug in SQL generation [SF#984591]

Author: Richard Jones

CHANGES.txt

@@ -20,6 +20,7 @@ Fixed:
 - rdbms backend full text search failure after import (sf bug 980314)
 - rdbms backends not filtering correctly on link=None
 - fix anydbm journal import (sf bug 983166)
+- handle postgresql bug in SQL generation (sf bug 984591)
 
 
 2004-06-24 0.7.5

doc/index.txt

@@ -130,6 +130,7 @@ Alexander Smishlajev,
 Nathaniel Smith,
 Mitchell Surface,
 Mike Thompson,
+Martin Uzak,
 Darryl VanDorp,
 J Vickroy.
 

roundup/backends/rdbms_common.py

@@ -1,4 +1,4 @@
-# $Id: rdbms_common.py,v 1.118 2004-07-03 16:51:52 a1s Exp $
+# $Id: rdbms_common.py,v 1.119 2004-07-03 23:05:15 richard Exp $
 ''' Relational database (SQL) backend common code.
 
 Basics:
@@ -1992,7 +1992,7 @@ def filter(self, search_matches, filterspec, sort=(None,None),
         timezone = self.db.getUserTimezone()
 
         # vars to hold the components of the SQL statement
-        frum = ['_'+cn] # FROM clauses
+        frum = [] # FROM clauses
         loj = []        # LEFT OUTER JOIN clauses
         where = []      # WHERE clauses
         args = []       # *any* positional arguments
@@ -2164,6 +2164,7 @@ def filter(self, search_matches, filterspec, sort=(None,None),
                 orderby.append(o)
 
         # construct the SQL
+        frum.append('_'+cn)
         frum = ','.join(frum)
         if where:
             where = ' where ' + (' and '.join(where))
@@ -2183,6 +2184,7 @@ def filter(self, search_matches, filterspec, sort=(None,None),
         cols = ','.join(cols)
         loj = ' '.join(loj)
         sql = 'select %s from %s %s %s%s'%(cols, frum, loj, where, order)
+        print sql
         args = tuple(args)
         __traceback_info__ = (sql, args)
         self.db.sql(sql, args)

test/db_test_base.py

@@ -15,7 +15,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-# $Id: db_test_base.py,v 1.38 2004-07-03 22:43:59 richard Exp $ 
+# $Id: db_test_base.py,v 1.39 2004-07-03 23:05:15 richard Exp $ 
 
 import unittest, os, shutil, errno, imp, sys, time, pprint
 
@@ -836,15 +836,15 @@ def filteringSetup(self):
         iss = self.db.issue
         for issue in (
                 {'title': 'issue one', 'status': '2', 'assignedto': '1',
-                    'foo': date.Interval('1:10'), 'priority': '1',
+                    'foo': date.Interval('1:10'), 'priority': '3',
                     'deadline': date.Date('2003-01-01.00:00')},
                 {'title': 'issue two', 'status': '1', 'assignedto': '2',
                     'foo': date.Interval('1d'), 'priority': '3',
                     'deadline': date.Date('2003-02-16.22:50')},
                     {'title': 'issue three', 'status': '1', 'priority': '2',
                     'nosy': ['1','2'], 'deadline': date.Date('2003-02-18')},
                 {'title': 'non four', 'status': '3',
-                    'foo': date.Interval('0:10'), 'priority': '1',
+                    'foo': date.Interval('0:10'), 'priority': '2',
                     'nosy': ['1'], 'deadline': date.Date('2004-03-08')}):
             self.db.issue.create(**issue)
         file_content = ''.join([chr(i) for i in range(255)])
@@ -893,6 +893,8 @@ def testFilteringMultilink(self):
         ae, filt = self.filteringSetup()
         ae(filt(None, {'nosy': '2'}, ('+','id'), (None,None)), ['3'])
         ae(filt(None, {'nosy': '-1'}, ('+','id'), (None,None)), ['1', '2'])
+        ae(filt(None, {'nosy': ['1','2']}, ('+', 'status'),
+            ('-', 'activity')), ['4', '3'])
 
     def testFilteringMany(self):
         ae, filt = self.filteringSetup()