[ https://issues.jboss.org/browse/DROOLS-3309?page=com.atlassian.jira.plugi... ] Gabriele Cardosi updated DROOLS-3309: ------------------------------------- Description: Clean up code # replace GIVEN/EXPECT Strings with Enum # move classes to proper packages # rename classes with "old" meanings (for ex, BaseExpressionGridTheme is the name of the class originally copied from DMN) # BaseExpressionGridTheme.getHeaderBackground remove get(0) used to set background color # remove injection of ScenarioSimulationDocksHandler from ScenarioSimulationEditorPresenter was: Clean up code # replace GIVEN/EXPECT Strings with Enum # move classes to proper packages # rename classes with "old" meanings (for ex, BaseExpressionGridTheme is the name of the class originally copied from DMN) # BaseExpressionGridTheme.getHeaderBackground remove get(0) used to set background color # remove injection of ScenarioSimulationDocksHandler from ScenarioSimulationEditorPresenter ## inject ScenarioSimulationDocksHandler inside CommandExecutor ## create one specific GWT event with four Enum to choose which action to execute ## register CommandExecutor for that event ## inside ScenarioSimulationEditorPresenter fire the required event when necessary instead of direct calls top DocksHandler ## let CommandExecutor manage the event -- This message was sent by Atlassian Jira (v7.12.1#712002)