[Zope3-checkins] CVS: Zope3/src/zope/app/undo - undo.py:1.3

[Philipp von Weitershausen]

Update of /cvs-repository/Zope3/src/zope/app/undo
In directory cvs.zope.org:/tmp/cvs-serv25980

Modified Files:
	undo.py 
Log Message:
Understood where and how and why we and up with a '/' for a user's path in
a transactional note. Commented that, fixed ZODBUndoManager and adjusted
the tests. This is just temporary until we have the new simplified undo
model.


=== Zope3/src/zope/app/undo/undo.py 1.2 => 1.3 ===
--- Zope3/src/zope/app/undo/undo.py:1.2	Sat Mar 13 18:55:28 2004
+++ Zope3/src/zope/app/undo/undo.py	Thu Mar 18 09:33:21 2004
@@ -46,12 +46,15 @@
         # description -> transaction description
 
         if user_name is not None:
-            # !?&%!! The 'user' in the transactions is some combination
-            # of 'path' and 'user'. 'path' seems to only ever be '/' at
-            # the moment - I can't find anything that calls it :-(
-            # At the moment the path is hacked onto the user_name in the 
-            # PageTemplate 'undo_log.pt' (to minimise the nastiness).
-            specification = {'user_name':user_name}
+
+            # XXX The 'user' in the transactions is a concatenation of
+            # 'path' and 'user' (id). 'path' used to be the path of
+            # the user folder in Zope 2. ZopePublication currently
+            # does not set a path, so it defaults to '/'. Maybe we can
+            # find a new meaning for 'path' in Zope 3 (the principal
+            # source?)
+            path = '/' # default for now
+            specification = {'user_name': path + ' ' + user_name}
         else:
             specification = None
         entries = self.__db.undoInfo(first, last, specification)