[Zope-CMF] Unable to 'Undo'

[Florent Guillaume]

Florent Guillaume  <fg@nuxeo.com> wrote:
> Carl Rendell  <cer@sol43.com> wrote:
> > More research and testing reveals that there are three lines in the 
> > CMFCore.UndoTool that are giving me [ is it just me? ] the 
> > behavior - lines 114 - 116
> > 
> >          for tinfo in transaction_info:
> >              if not xids.get( tinfo, None ):
> >                  raise Unauthorized
> 
> Ok, the problem is that:
>  - transaction ids contain "\n" (due to encode64)
>  - they get passed to the user browser as checkboxes value=
>  - when they return thay contain "\r\n" instead of "\n"

This is now fixed.

The UndoTool still has a few wrinkles though:

 - The skin is doing a redirect to cmfroot/folder_contents which
   doesn't work.

 - If I change listUndoableTransactionsFor to use the object as a
   context for undoable_transactions instead of the portal, I still
   get transactions like /cmfroot/newsitem_edit where I'd expect
   /cmfroot/Members/bob/foo/newsitem_edit. So doing an
   object-dependant undo doesn't work.

Florent

-- 
Florent Guillaume, Nuxeo (Paris, France)
+33 1 40 33 79 87  http://nuxeo.com  mailto:fg@nuxeo.com