don't have RDBMS backends list retired nodes [SF#767319]

Author: Richard Jones

CHANGES.txt

@@ -9,6 +9,7 @@ are given with the most recent entry first.
 - timelog editing via csv interface crashes (sf bug 699837)
 - sort multilinks a little better for grouping (sf bug 772935)
 - batch the (list) listings at 500 entries per page (sf bug 759906)
+- don't have RDBMS backends list retired nodes (sf bug 767319)
 
 
 2003-07-29 0.6.0b4

roundup/backends/rdbms_common.py

@@ -1,4 +1,4 @@
-# $Id: rdbms_common.py,v 1.57 2003-07-03 23:43:46 richard Exp $
+# $Id: rdbms_common.py,v 1.58 2003-08-12 02:18:46 richard Exp $
 ''' Relational database (SQL) backend common code.
 
 Basics:
@@ -1479,7 +1479,7 @@ def retire(self, nodeid):
         self.fireReactors('retire', nodeid, None)
 
     def restore(self, nodeid):
-        '''Restpre a retired node.
+        '''Restore a retired node.
 
         Make node available for all operations like it was before retirement.
         '''
@@ -1891,6 +1891,9 @@ def filter(self, search_matches, filterspec, sort=(None,None),
                     where.append('_%s=%s'%(k, a))
                     args.append(v)
 
+        # don't match retired nodes
+        where.append('__retired__ <> 1')
+
         # add results of full text search
         if search_matches is not None:
             v = search_matches.keys()

test/test_db.py

@@ -15,7 +15,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-# $Id: test_db.py,v 1.89 2003-05-09 01:47:51 richard Exp $ 
+# $Id: test_db.py,v 1.90 2003-08-12 02:22:22 richard Exp $ 
 
 import unittest, os, shutil, time
 
@@ -684,6 +684,11 @@ def testFilteringLink(self):
         ae, filt = self.filteringSetup()
         ae(filt(None, {'status': '1'}, ('+','id'), (None,None)), ['2','3'])
 
+    def testFilteringRetired(self):
+        ae, filt = self.filteringSetup()
+        self.db.issue.retire('2')
+        ae(filt(None, {'status': '1'}, ('+','id'), (None,None)), ['3'])
+
     def testFilteringMultilink(self):
         ae, filt = self.filteringSetup()
         ae(filt(None, {'nosy': '2'}, ('+','id'), (None,None)), ['3'])