Skip to main content
Aaronius9er9er
Inspiring
May 28, 2010 3:15 PM UTC
Answered

EditManager.performRedo() should include redoPsuedoOp in complete event

I think this is a bug but I just want to verify.  In EditManager.performRedo(), it dispatches a FlowOperationEvent.FLOW_OPERATION_COMPLETE event but passes in "op" as the operation parameter when I think it should be passing in "redoPsuedoOp".  This would make it consistent with performUndo().  Without passing redoPsuedoOp, it's difficult to tell from an event handler whether the operation was a redo or a brand new operation.

This topic is closed to new replies. Start a new post to keep the conversation going.
rdermerCorrect answer
Inspiring
May 28, 2010 10:55 PM UTC

Looks like a bug to me.  Probably should have an _imeSession check as well.  Hmmm so should performUndo.

Thanks for the report.


Richard

1 reply

rdermerCorrect answer
Inspiring
May 28, 2010 10:55 PM UTC

Looks like a bug to me.  Probably should have an _imeSession check as well.  Hmmm so should performUndo.

Thanks for the report.


Richard

Translate