Summary of bug description

• Consistency checking of statemachine patterns didn't work in mitkStateMachineFactory. Lines with warnings were commented. So it worked, but no warnings of inconsistent statemachine patterns were given (i.e. patterns with three states, but the statemachine can only stay in one state. The two other states are not reachable.). Therefore a bool value was passed recusively during statemachine pattern buildup. An warning should have been thrown if bool = false but the warning was commented. Furthermore, the bool value is not set to true.

• StateMachineFactory shall warn the user if a statemachine pattern is loaded, that will not work during runtime. For this, the bool variable should be initialized with true and changed to false, it an error was found.

Cause of the bug

A mixture of commenting the warnings and continuing the build up process if an error was found.

Proposed solution

Default of the bool variable is true and will be changed if an error was found. Furthermore, the warnings shall be changed to LogMessages. They should state the name of the statemachine pattern.

Affected classes

mitkStateMachineFactory.cpp and statemachine.xml

How will the bugfix get tested?

manual test