SVN: Zope/trunk/src/App/Undo.py `undoMultiple` was still broken as transactions were not undone in the proper
Godefroid Chapelle <[email protected]> Mon, 19 Sep 2011 06:03:54 -0400 (EDT)
| Newsgroups | gmane.comp.web.zope.cvs |
|---|---|
| Message-ID | <20110919100354.C4991941FB__21113.1732132801$1316426658$gmane$org@cvs.zope.org> |
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
_______________________________________________
Zope-Checkins maillist - [email protected]
https://mail.zope.org/mailman/listinfo/zope-checkins