State Machine refactoring

git-svn-id: https://joinup.ec.europa.eu/svn/pdf-over/trunk@23 174cde9d-5d70-4d2a-aa98-46368bc2aaf7

1 files changed, 33 insertions, 14 deletions

diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java
index 5b764084..c7c92a89 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java
@@ -20,10 +20,10 @@ import org.eclipse.swt.SWT;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import at.asit.pdfover.gui.components.PositioningComposite;
+import at.asit.pdfover.gui.MainWindowBehavior;
+import at.asit.pdfover.gui.MainWindow.Buttons;
+import at.asit.pdfover.gui.composites.PositioningComposite;
 import at.asit.pdfover.gui.workflow.StateMachine;
-import at.asit.pdfover.gui.workflow.StateMachineImpl;
-import at.asit.pdfover.gui.workflow.State;
 
 /**
  * Decides where to position the signature block
@@ -31,6 +31,13 @@ import at.asit.pdfover.gui.workflow.State;
 public class PositioningState extends State {
 
 	/**
+	 * @param stateMachine
+	 */
+	public PositioningState(StateMachine stateMachine) {
+		super(stateMachine);
+	}
+
+	/**
 	 * SFL4J Logger instance
 	 **/
 	@SuppressWarnings("unused")
@@ -39,35 +46,47 @@ public class PositioningState extends State {
 
 	private PositioningComposite positionComposite = null;
 
-	private PositioningComposite getPositioningComosite(StateMachineImpl workflow) {
+	private PositioningComposite getPositioningComosite() {
 		if (this.positionComposite == null) {
 			this.positionComposite = new PositioningComposite(
-					workflow.getComposite(), SWT.NONE, workflow);
+					this.stateMachine.getGUIProvider().getComposite(), SWT.NONE, this);
 		}
 
 		return this.positionComposite;
 	}
 
 	@Override
-	public void run(StateMachine stateMachine) {
+	public void run() {
 		
-		if(workflow.getParameter().getSignaturePosition() == null) {
-			PositioningComposite position = this.getPositioningComosite(workflow);
+		if(this.stateMachine.getStatus().getSignaturePosition() == null) {
+			PositioningComposite position = this.getPositioningComosite();
 			
-			workflow.setTopControl(position);
+			this.stateMachine.getGUIProvider().display(position);
 			
-			workflow.getParameter().setSignaturePosition(position.getPosition());
+			this.stateMachine.getStatus().setSignaturePosition(position.getPosition());
 			
-			if(workflow.getParameter().getSignaturePosition() == null) {
-				this.setNextState(this);
+			if(this.stateMachine.getStatus().getSignaturePosition() == null) {
 				return;
 			}
 		}
-		this.setNextState(new BKUSelectionState());
+		this.setNextState(new BKUSelectionState(this.stateMachine));
+	}
+
+	/* (non-Javadoc)
+	 * @see at.asit.pdfover.gui.workflow.states.State#setMainWindowBehavior()
+	 */
+	@Override
+	public void updateMainWindowBehavior() {
+		MainWindowBehavior behavior = this.stateMachine.getStatus().getBehavior();
+		behavior.reset();
+		behavior.setEnabled(Buttons.CONFIG, true);
+		behavior.setEnabled(Buttons.OPEN, true);
+		behavior.setActive(Buttons.OPEN, true);
+		behavior.setActive(Buttons.POSITION, true);
 	}
 
 	@Override
 	public String toString()  {
-		return "PositioningState";
+		return this.getClass().getName();
 	}
 }