Re: [Planner Dev] Task/Resource and Project property redo/undo patch.

["lincoln phipps openmutual net" <lincoln phipps openmutual net>]

Richard Hult wrote:

> On mån, 2004-05-24 at 05:27 +0100, lincoln phipps openmutual net wrote:
>
> > Attached is version 0.9 of this patch.
>
>
> Thanks a lot! Committed with a few style changes.

Have found a little bug in retesting CVS after commit. The
patch works but the store (model) for the view remembers
the old pointer to property whereas REDO uses a new one.

Most changes all work fine. Its only when you UNDO to nothing
and then REDO that the model gets out of step. I can see why
it wasn't picked up in initial tests - individual tasks
like re-label + undo + redo or  add + undo +redo, or remove
+undo + redo work OK but the redo for an add actually makes
the model reuse an old property pointer so it looks OK but
isn't really.

Keep the code as is now in CVS and I'll diff to that and fix
this up ASAP.

/Lincoln.

> /Richard