SequenceCommand doesn't rollback properly in case of subcommand error.
|Reported by:||Tyndare||Owned by:||team|
|Keywords:||SequenceCommand undo||Cc:||Don-vip, bastiK|
I think found an issue in the SequenceCommand class: if one of the sub-commands fails, the preceding sub-commands that were already executed before are not undone.
This seems to contradict the code intention.
I also have problems with the continueOnError variable of this class. I tried to make a test (see the attached file) but I was not able to make it work.
Actually I'm not sure I fully understand what is the contract between executeCommand() and undo().
Is it ok/necessary/forbidden to call undo() if execucedCommand() returned false ?