issue2550729: Fix password history display for anydbm backend...

...thanks to Ralf Hemmecke for reporting.

Author: Ralf Schlatterbeck

CHANGES.txt

@@ -24,6 +24,8 @@ Fixed:
   field of a user -- this would match substrings, e.g. if the user has
   [email protected] as an alternate email and an incoming mail
   is addressed to [email protected] this would (wrongly) match. (Ralf)
+- issue2550729: Fix password history display for anydbm backend, thanks
+  to Ralf Hemmecke for reporting.
 
 2011-07-15 1.4.19 (r4638)
 

roundup/backends/back_anydbm.py

@@ -575,6 +575,21 @@ def setjournal(self, classname, nodeid, journal):
         self.transactions.append((self.doSetJournal, (classname, nodeid,
             journal)))
 
+    def fix_journal(self, classname, journal):
+        """ fix password entries to correct type """
+        pwprops = {}
+        for pn, prop in self.getclass(classname).properties.iteritems():
+            if isinstance(prop, hyperdb.Password):
+                pwprops [pn] = 1
+        if not pwprops:
+            return journal
+        for j in journal:
+            if j[3] == 'set':
+                for k, v in j[4].items():
+                    if k in pwprops:
+                        j[4][k] = password.JournalPassword(j[4][k])
+        return journal
+
     def getjournal(self, classname, nodeid):
         """ get the journal for id
 
@@ -611,22 +626,22 @@ def getjournal(self, classname, nodeid):
                 raise
             if res:
                 # we have unsaved journal entries, return them
-                return res
+                return self.fix_journal (classname, res)
             raise IndexError('no such %s %s'%(classname, nodeid))
         try:
             journal = marshal.loads(db[nodeid])
         except KeyError:
             db.close()
             if res:
                 # we have some unsaved journal entries, be happy!
-                return res
+                return self.fix_journal (classname, res)
             raise IndexError('no such %s %s'%(classname, nodeid))
         db.close()
 
         # add all the saved journal entries for this node
         for nodeid, date_stamp, user, action, params in journal:
             res.append((nodeid, date.Date(date_stamp), user, action, params))
-        return res
+        return self.fix_journal (classname, res)
 
     def pack(self, pack_before):
         """ Delete all journal entries except "create" before 'pack_before'.