diff roundup/backends/back_anydbm.py @ 1099:7362dc1f0226

Made "creator" property a String like it should be (we can't lookup() retired users reliably)
author Richard Jones <richard@users.sourceforge.net>
date Tue, 10 Sep 2002 08:04:57 +0000
parents 32e41ddf2edb
children db787cef1385
line wrap: on
line diff
--- a/roundup/backends/back_anydbm.py	Tue Sep 10 07:07:16 2002 +0000
+++ b/roundup/backends/back_anydbm.py	Tue Sep 10 08:04:57 2002 +0000
@@ -15,7 +15,7 @@
 # BASIS, AND THERE IS NO OBLIGATION WHATSOEVER TO PROVIDE MAINTENANCE,
 # SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
 # 
-#$Id: back_anydbm.py,v 1.73 2002-09-10 00:11:49 richard Exp $
+#$Id: back_anydbm.py,v 1.74 2002-09-10 08:04:56 richard Exp $
 '''
 This module defines a backend that saves the hyperdatabase in a database
 chosen by anydbm. It is guaranteed to always be available in python
@@ -976,14 +976,9 @@
                 raise ValueError, 'Journalling is disabled for this class'
             journal = self.db.getjournal(self.classname, nodeid)
             if journal:
-                name = self.db.getjournal(self.classname, nodeid)[0][2]
+                return self.db.getjournal(self.classname, nodeid)[0][2]
             else:
-                return None
-            try:
-                return self.db.user.lookup(name)
-            except KeyError:
-                # the journaltag user doesn't exist any more
-                return None
+                return self.db.journaltag
 
         # get the property (raises KeyErorr if invalid)
         prop = self.properties[propname]
@@ -1734,7 +1729,9 @@
             d['id'] = String()
             d['creation'] = hyperdb.Date()
             d['activity'] = hyperdb.Date()
-            d['creator'] = hyperdb.Link("user")
+            # can't be a link to user because the user might have been
+            # retired since the journal entry was created
+            d['creator'] = hyperdb.String()
         return d
 
     def addprop(self, **properties):

Roundup Issue Tracker: http://roundup-tracker.org/