[Checkins] SVN: Zope/trunk/src/App/Undo.py `undoMultiple` was still broken as transactions were not undone in the proper

Godefroid Chapelle gotcha at bubblenet.be
Mon Sep 19 05:03:54 EST 2011


Log message for revision 122838:
  `undoMultiple` was still broken as transactions were not undone in the proper
    order : tids were stored and retrieved as dictionary keys. 
  

Changed:
  U   Zope/trunk/src/App/Undo.py

-=-
Modified: Zope/trunk/src/App/Undo.py
===================================================================
--- Zope/trunk/src/App/Undo.py	2011-09-19 09:57:57 UTC (rev 122837)
+++ Zope/trunk/src/App/Undo.py	2011-09-19 10:03:54 UTC (rev 122838)
@@ -132,15 +132,17 @@
     def manage_undo_transactions(self, transaction_info=(), REQUEST=None):
         """
         """
-        tids = {}
+        tids = []
+        descriptions = []
         for tid in transaction_info:
             tid = tid.split()
             if tid:
-                tids[decode64(tid[0])] = tid[-1]
+                tids.append(decode64(tid[0]))
+                descriptions.append(tid[-1])
 
         if tids:
-            transaction.get().note("Undo %s" % ' '.join(tids.values()))
-            self._p_jar.db().undoMultiple(tids.keys())
+            transaction.get().note("Undo %s" % ' '.join(descriptions))
+            self._p_jar.db().undoMultiple(tids)
 
         if REQUEST is None:
             return



More information about the checkins mailing list