diff options
Diffstat (limited to 'pdf-over-gui/src/main')
15 files changed, 141 insertions, 182 deletions
| diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java index 105b769f..85bccd8e 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java @@ -18,8 +18,6 @@ package at.asit.pdfover.gui;  // Imports  import java.io.IOException;  import java.io.InputStream; -import java.lang.reflect.Field; -import java.lang.reflect.Method;  import java.util.EnumMap;  import java.util.Map; @@ -41,7 +39,6 @@ import org.eclipse.swt.widgets.Composite;  import org.eclipse.swt.widgets.Control;  import org.eclipse.swt.widgets.Display;  import org.eclipse.swt.widgets.Event; -import org.eclipse.swt.widgets.Listener;  import org.eclipse.swt.widgets.Monitor;  import org.eclipse.swt.widgets.Shell;  import org.slf4j.Logger; @@ -206,7 +203,7 @@ public class MainWindow {  	 */  	protected void createContents() {  		this.shell = new Shell(); -		getShell().setSize(this.stateMachine.getConfigProvider().getMainWindowSize()); +		getShell().setSize(this.stateMachine.configProvider.getMainWindowSize());  		if (System.getProperty("os.name").toLowerCase().contains("mac")) {  			if (System.getProperty("os.name").contains("OS X")) {  				hookupOSXMenu(); @@ -231,9 +228,9 @@ public class MainWindow {  			@Override  			public void shellClosed(ShellEvent e) {  				log.debug("Closing main window"); -				MainWindow.this.stateMachine.getConfigManipulator().setMainWindowSize(getShell().getSize()); +				MainWindow.this.stateMachine.configProvider.setMainWindowSize(getShell().getSize());  				try { -					MainWindow.this.stateMachine.getConfigManipulator().saveCurrentConfiguration(); +					MainWindow.this.stateMachine.configProvider.saveCurrentConfiguration();  				} catch (IOException e1) {  					log.error("Error saving configuration", e);  				} @@ -305,9 +302,8 @@ public class MainWindow {  			@Override  			public void mouseUp(MouseEvent e) { -				if (MainWindow.this.stateMachine.getStatus().getCurrentState() instanceof OpenState) { -					((OpenState) MainWindow.this.stateMachine.getStatus() -							.getCurrentState()).openFileDialog(); +				if (MainWindow.this.stateMachine.status.getCurrentState() instanceof OpenState) { +					((OpenState) MainWindow.this.stateMachine.status.getCurrentState()).openFileDialog();  				} else {  					MainWindow.this.stateMachine.jumpToState(new OpenState(  							MainWindow.this.stateMachine)); @@ -396,7 +392,7 @@ public class MainWindow {  			},  			/* preferencesListener */  			(Event arg0) -> { -				if (MainWindow.this.stateMachine.getStatus().behavior.getEnabled(Buttons.CONFIG)) +				if (MainWindow.this.stateMachine.status.behavior.getEnabled(Buttons.CONFIG))  					MainWindow.this.stateMachine.jumpToState(new ConfigurationUIState(MainWindow.this.stateMachine));  			}  		); @@ -406,10 +402,10 @@ public class MainWindow {  	 * Update MainWindow to fit new status  	 */  	public void applyBehavior() { -		MainWindowBehavior behavior = this.stateMachine.getStatus().behavior; +		MainWindowBehavior behavior = this.stateMachine.status.behavior;  		log.debug("Updating MainWindow state for : " -				+ this.stateMachine.getStatus().getCurrentState().toString()); +				+ this.stateMachine.status.getCurrentState().toString());  		for (Buttons button : Buttons.values()) {  			boolean active = behavior.getActive(button); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/cliarguments/Argument.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/cliarguments/Argument.java index b2ecb3aa..9fbe6eba 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/cliarguments/Argument.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/cliarguments/Argument.java @@ -57,7 +57,7 @@ public abstract class Argument {  	 * @return the configuration overlay manipulator  	 */  	protected ConfigOverlayManipulator getConfiguration() { -		return this.stateMachine.getConfigOverlayManipulator(); +		return this.stateMachine.configProvider;  	}  	/** @@ -65,7 +65,7 @@ public abstract class Argument {  	 * @return the status  	 */  	protected Status getStatus() { -		return this.stateMachine.getStatus(); +		return this.stateMachine.status;  	}  	/** diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java index 5a1248d7..68eeb78b 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java @@ -44,11 +44,10 @@ public class StateMachine implements GUIProvider {  	private static final Logger log = LoggerFactory.getLogger(StateMachine.class); -	private Status status; - -	private PDFSigner pdfSigner; - -	private ConfigProviderImpl configProvider; +	public final Status status; +	public final PDFSigner pdfSigner; +	public final ConfigProviderImpl configProvider; +	public final String[] cmdLineArgs;  	/**  	 * Default constructor @@ -60,7 +59,7 @@ public class StateMachine implements GUIProvider {  		this.status.setCurrentState(new PrepareConfigurationState(this));  		this.pdfSigner = new PDFSigner();  		this.configProvider = new ConfigProviderImpl(); -		setCmdLineArgs(cmdLineArgs); +		this.cmdLineArgs = cmdLineArgs;  	}  	/** @@ -268,37 +267,6 @@ public class StateMachine implements GUIProvider {  			display.dispose();  		}  	} -	public ConfigProvider getConfigProvider() { -		return this.configProvider; -	} -	public PersistentConfigProvider getPersistentConfigProvider() { -		return this.configProvider; -	} -	public ConfigManipulator getConfigManipulator() { -		return this.configProvider; -	} -	public ConfigOverlayManipulator getConfigOverlayManipulator() { -		return this.configProvider; -	} -	public Status getStatus() { -		return this.status; -	} - -	public PDFSigner getPDFSigner() { -		return this.pdfSigner; -	} - -	public GUIProvider getGUIProvider() { -		return this; -	} - -	private String[] cmdLineArgs = new String[] {}; -	private void setCmdLineArgs(String[] cmdLineArgs) { -		this.cmdLineArgs = cmdLineArgs; -	} -	public String[] getCmdArgs() { -		return this.cmdLineArgs; -	}  	public synchronized Shell getMainShell() { diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/BKUSelectionState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/BKUSelectionState.java index dcffe2c0..01df1863 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/BKUSelectionState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/BKUSelectionState.java @@ -53,11 +53,11 @@ public class BKUSelectionState extends State {  	private BKUSelectionComposite getSelectionComposite() {  		if (this.selectionComposite == null) {  			this.selectionComposite = -					getStateMachine().getGUIProvider().createComposite(BKUSelectionComposite.class, SWT.RESIZE, this); +					getStateMachine().createComposite(BKUSelectionComposite.class, SWT.RESIZE, this);  		} -		if (getStateMachine().getConfigProvider().getKeyStoreEnabled()) { -			File ks = new File(getStateMachine().getConfigProvider().getKeyStoreFile()); +		if (getStateMachine().configProvider.getKeyStoreEnabled()) { +			File ks = new File(getStateMachine().configProvider.getKeyStoreFile());  			this.selectionComposite.setKeystoreEnabled(ks.exists());  		} else  			this.selectionComposite.setKeystoreEnabled(false); @@ -67,7 +67,7 @@ public class BKUSelectionState extends State {  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		State previousState = status.getPreviousState();  		if (!(  		  (previousState instanceof OpenState) || @@ -76,14 +76,14 @@ public class BKUSelectionState extends State {  		)) {  			status.bku = BKUs.NONE;  		} else if(!(status.getPreviousState() instanceof BKUSelectionState)) { -			ConfigProvider config = getStateMachine().getConfigProvider(); +			ConfigProvider config = getStateMachine().configProvider;  			status.bku = config.getDefaultBKU();  		}  		if(status.bku == BKUs.NONE) {  			BKUSelectionComposite selection = this.getSelectionComposite(); -			getStateMachine().getGUIProvider().display(selection); +			getStateMachine().display(selection);  			selection.layout();  			status.bku = selection.getSelected(); @@ -109,7 +109,7 @@ public class BKUSelectionState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setEnabled(Buttons.CONFIG, true);  		behavior.setEnabled(Buttons.OPEN, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ConfigurationUIState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ConfigurationUIState.java index 8450629f..1e02893a 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ConfigurationUIState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ConfigurationUIState.java @@ -44,10 +44,10 @@ public class ConfigurationUIState extends State {  	private ConfigurationComposite getConfigurationComposite() {  		if (this.configurationComposite == null) {  			this.configurationComposite = -					getStateMachine().getGUIProvider().createComposite(ConfigurationComposite.class, SWT.RESIZE, this); -			this.configurationComposite.setConfigManipulator(getStateMachine().getConfigManipulator()); -			this.configurationComposite.setConfigProvider(getStateMachine().getPersistentConfigProvider()); -			this.configurationComposite.setSigner(getStateMachine().getPDFSigner()); +					getStateMachine().createComposite(ConfigurationComposite.class, SWT.RESIZE, this); +			this.configurationComposite.setConfigManipulator(getStateMachine().configProvider); +			this.configurationComposite.setConfigProvider(getStateMachine().configProvider); +			this.configurationComposite.setSigner(getStateMachine().pdfSigner);  		}  		return this.configurationComposite; @@ -66,11 +66,11 @@ public class ConfigurationUIState extends State {  	 */  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		ConfigurationComposite config = this.getConfigurationComposite(); -		getStateMachine().getGUIProvider().display(config); +		getStateMachine().display(config);  		if(config.isUserDone())  		{ @@ -96,7 +96,7 @@ public class ConfigurationUIState extends State {  	@Override  	public void updateMainWindowBehavior() {  		// Leave the state as it is -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.setEnabled(Buttons.CONFIG, false);  		behavior.setMainBarVisible(false);  	} @@ -105,6 +105,6 @@ public class ConfigurationUIState extends State {  	 * Triggers to reload the resources  	 */  	public void reloadResources() { -		getStateMachine().getGUIProvider().reloadResources(); +		getStateMachine().reloadResources();  	}  } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/KSState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/KSState.java index 1ecf880c..94171262 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/KSState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/KSState.java @@ -61,10 +61,10 @@ public class KSState extends State {  	 */  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		SigningState signingState = status.signingState; -		ConfigProvider config = getStateMachine().getConfigProvider(); +		ConfigProvider config = getStateMachine().configProvider;  		try {  			String file = config.getKeyStoreFile(); @@ -72,7 +72,7 @@ public class KSState extends State {  			if (!f.isFile()) {  				log.error("Keystore not found");  				ErrorDialog dialog = new ErrorDialog( -						getStateMachine().getGUIProvider().getMainShell(), +						getStateMachine().getMainShell(),  						String.format(Messages.getString("error.KeyStoreFileNotExist"), f.getName()),  						BUTTONS.RETRY_CANCEL);  				if (dialog.open() != SWT.RETRY) { @@ -87,7 +87,7 @@ public class KSState extends State {  			String storePass = config.getKeyStoreStorePass();  			if (storePass.isEmpty()) {  				PasswordInputDialog pwd = new PasswordInputDialog( -						getStateMachine().getGUIProvider().getMainShell(), +						getStateMachine().getMainShell(),  						Messages.getString("keystore_config.KeystoreStorePass"),  						Messages.getString("keystore.KeystoreStorePassEntry"));  				storePass = pwd.open(); @@ -95,7 +95,7 @@ public class KSState extends State {  			String keyPass = config.getKeyStoreKeyPass();  			if (keyPass.isEmpty()) {  				PasswordInputDialog pwd = new PasswordInputDialog( -						getStateMachine().getGUIProvider().getMainShell(), +						getStateMachine().getMainShell(),  						Messages.getString("keystore_config.KeystoreKeyPass"),  						Messages.getString("keystore.KeystoreKeyPassEntry"));  				keyPass = pwd.open(); @@ -105,7 +105,7 @@ public class KSState extends State {  		} catch (SignatureException e) {  			log.error("Error loading keystore", e);  			ErrorDialog dialog = new ErrorDialog( -					getStateMachine().getGUIProvider().getMainShell(), +					getStateMachine().getMainShell(),  					Messages.getString("error.KeyStore"),  					BUTTONS.RETRY_CANCEL);  			if (dialog.open() != SWT.RETRY) { @@ -138,7 +138,7 @@ public class KSState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setActive(Buttons.OPEN, true);  		behavior.setActive(Buttons.POSITION, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java index 3ec134d4..51151338 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java @@ -171,7 +171,7 @@ public class LocalBKUState extends State {  	 */  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		SigningState signingState = status.signingState; @@ -185,7 +185,7 @@ public class LocalBKUState extends State {  		if (this.threadException != null) {  			ErrorDialog dialog = new ErrorDialog( -					getStateMachine().getGUIProvider().getMainShell(), +					getStateMachine().getMainShell(),  					Messages.getString("error.LocalBKU"),  					BUTTONS.RETRY_CANCEL);  			if (dialog.open() != SWT.RETRY) { @@ -219,7 +219,7 @@ public class LocalBKUState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setActive(Buttons.OPEN, true);  		behavior.setActive(Buttons.POSITION, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java index a2c65938..6d59dfd5 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java @@ -64,16 +64,16 @@ public class MobileBKUState extends State {  	public MobileBKUState(StateMachine stateMachine) {
  		super(stateMachine);
 -		ConfigProvider provider = stateMachine.getConfigProvider();
 +		ConfigProvider provider = stateMachine.configProvider;
  		switch(provider.getMobileBKUType()) {
  			case A_TRUST:
  				this.status = new ATrustStatus(provider);
 -				this.handler = new ATrustHandler(this, stateMachine.getGUIProvider().getMainShell(), provider.getMobileBKUBase64());
 +				this.handler = new ATrustHandler(this, stateMachine.getMainShell(), provider.getMobileBKUBase64());
  				break;
  			case IAIK:
  				this.status = new IAIKStatus(provider);
 -				this.handler = new IAIKHandler(this, stateMachine.getGUIProvider().getMainShell());
 +				this.handler = new IAIKHandler(this, stateMachine.getMainShell());
  				break;
  			default:
 @@ -86,7 +86,7 @@ public class MobileBKUState extends State {  	WaitingForAppComposite waitingForAppComposite = null;
  	WaitingForAppComposite getWaitingForAppComposite() {
  		if (this.waitingForAppComposite == null) {
 -			this.waitingForAppComposite = getStateMachine().getGUIProvider()
 +			this.waitingForAppComposite = getStateMachine()
  					.createComposite(WaitingForAppComposite.class, SWT.RESIZE, this);
  		}
 @@ -96,7 +96,7 @@ public class MobileBKUState extends State {  	WaitingComposite waitingComposite = null;
  	WaitingComposite getWaitingComposite() {
  		if (this.waitingComposite == null) {
 -			this.waitingComposite = getStateMachine().getGUIProvider()
 +			this.waitingComposite = getStateMachine()
  					.createComposite(WaitingComposite.class, SWT.RESIZE, this);
  		}
 @@ -105,7 +105,7 @@ public class MobileBKUState extends State {  	MobileBKUEnterTANComposite getMobileBKUEnterTANComposite() {
  		if (this.mobileBKUEnterTANComposite == null) {
 -			this.mobileBKUEnterTANComposite = getStateMachine().getGUIProvider()
 +			this.mobileBKUEnterTANComposite = getStateMachine()
  					.createComposite(MobileBKUEnterTANComposite.class, SWT.RESIZE, this);
  		}
 @@ -115,7 +115,7 @@ public class MobileBKUState extends State {  	MobileBKUQRComposite mobileBKUQRComposite = null;
  	MobileBKUQRComposite getMobileBKUQRComposite() {
  		if (this.mobileBKUQRComposite == null) {
 -			this.mobileBKUQRComposite = getStateMachine().getGUIProvider()
 +			this.mobileBKUQRComposite = getStateMachine()
  					.createComposite(MobileBKUQRComposite.class, SWT.RESIZE, this);
  		}
 @@ -125,7 +125,7 @@ public class MobileBKUState extends State {  	MobileBKUEnterNumberComposite mobileBKUEnterNumberComposite = null;
  	MobileBKUEnterNumberComposite getMobileBKUEnterNumberComposite() {
  		if (this.mobileBKUEnterNumberComposite == null) {
 -			this.mobileBKUEnterNumberComposite = getStateMachine().getGUIProvider()
 +			this.mobileBKUEnterNumberComposite = getStateMachine()
  					.createComposite(MobileBKUEnterNumberComposite.class, SWT.RESIZE, this);
  		}
 @@ -135,7 +135,7 @@ public class MobileBKUState extends State {  	MobileBKUFingerprintComposite mobileBKUFingerprintComposite = null;
  	MobileBKUFingerprintComposite getMobileBKUFingerprintComposite() {
  		if (this.mobileBKUFingerprintComposite == null) {
 -			this.mobileBKUFingerprintComposite = getStateMachine().getGUIProvider()
 +			this.mobileBKUFingerprintComposite = getStateMachine()
  					.createComposite(MobileBKUFingerprintComposite.class, SWT.RESIZE, this);
  		}
 @@ -147,7 +147,7 @@ public class MobileBKUState extends State {  	 * @return the mobile BKU URL
  	 */
  	public String getURL() {
 -		return getStateMachine().getConfigProvider().getMobileBKUURL();
 +		return getStateMachine().configProvider.getMobileBKUURL();
  	}
  	/**
 @@ -181,7 +181,7 @@ public class MobileBKUState extends State {  	public void displayError(final String message) {
  		log.error(message);
  		Display.getDefault().syncExec(() -> {
 -			ErrorDialog error = new ErrorDialog(getStateMachine().getGUIProvider().getMainShell(), message, BUTTONS.OK);
 +			ErrorDialog error = new ErrorDialog(getStateMachine().getMainShell(), message, BUTTONS.OK);
  			error.open();
  		});
  	}
 @@ -222,9 +222,9 @@ public class MobileBKUState extends State {  					ui.setMobilePassword(mobileStatus.mobilePassword);
  				}
  				ui.enableButton();
 -				getStateMachine().getGUIProvider().display(ui);
 +				getStateMachine().display(ui);
 -				Display display = getStateMachine().getGUIProvider().getMainShell().getDisplay();
 +				Display display = getStateMachine().getMainShell().getDisplay();
  				while (!ui.userAck && !ui.userCancel) {
  					if (!display.readAndDispatch()) {
  						display.sleep();
 @@ -246,7 +246,7 @@ public class MobileBKUState extends State {  			mobileStatus.mobilePassword = ui.getMobilePassword();
  			// show waiting composite
 -			getStateMachine().getGUIProvider().display(this.getWaitingComposite());
 +			getStateMachine().display(this.getWaitingComposite());
  		});
  	}
 @@ -272,9 +272,9 @@ public class MobileBKUState extends State {  					tan.setTries(mobileStatus.tanTries);
  				}
  				tan.enableButton();
 -				getStateMachine().getGUIProvider().display(tan);
 +				getStateMachine().display(tan);
 -				Display display = getStateMachine().getGUIProvider().getMainShell().getDisplay();
 +				Display display = getStateMachine().getMainShell().getDisplay();
  				while (!tan.isUserAck() && !tan.isUserCancel()) {
  					if (!display.readAndDispatch()) {
  						display.sleep();
 @@ -294,7 +294,7 @@ public class MobileBKUState extends State {  			mobileStatus.tan = tan.getTan();
  			// show waiting composite
 -			getStateMachine().getGUIProvider().display(getWaitingComposite());
 +			getStateMachine().display(getWaitingComposite());
  		});
  	}
 @@ -315,7 +315,7 @@ public class MobileBKUState extends State {  					if (handler.handleQRResponse(resp)) {
  						log.debug("Signature page response: " + resp);
  						getMobileBKUQRComposite().setDone(true);
 -						Display display = getStateMachine().getGUIProvider().
 +						Display display = getStateMachine().
  								getMainShell().getDisplay();
  						display.wake();
  						checkDone.cancel();
 @@ -338,9 +338,9 @@ public class MobileBKUState extends State {  				this.threadException = new Exception(Messages.getString("error.FailedToLoadQRCode"));
  			}
  			qr.setQR(qrcode);
 -			getStateMachine().getGUIProvider().display(qr);
 +			getStateMachine().display(qr);
 -			Display display = getStateMachine().getGUIProvider().getMainShell().getDisplay();
 +			Display display = getStateMachine().getMainShell().getDisplay();
  			while (!qr.isUserCancel() && !qr.isUserSMS() && !qr.isDone()) {
  				if (!display.readAndDispatch()) {
  					display.sleep();
 @@ -364,7 +364,7 @@ public class MobileBKUState extends State {  				qr.setDone(false);
  			// show waiting composite
 -			getStateMachine().getGUIProvider().display(this.getWaitingComposite());
 +			getStateMachine().display(this.getWaitingComposite());
  		});
  	}
 @@ -377,9 +377,9 @@ public class MobileBKUState extends State {  		Display.getDefault().syncExec(() -> {
  			WaitingForAppComposite waitingForAppcomposite = this.getWaitingForAppComposite();
 -			getStateMachine().getGUIProvider().display(waitingForAppcomposite);
 +			getStateMachine().display(waitingForAppcomposite);
 -			Display display = getStateMachine().getGUIProvider().getMainShell().getDisplay();
 +			Display display = getStateMachine().getMainShell().getDisplay();
  			undecidedPolling();
  			long timeoutTime = System.nanoTime() + (300 * ((long)1e9));
 @@ -402,7 +402,7 @@ public class MobileBKUState extends State {  				status.errorMessage = "sms";
  				status.isSMSTan = true;
  				// show waiting composite
 -				getStateMachine().getGUIProvider().display(this.getWaitingComposite());
 +				getStateMachine().display(this.getWaitingComposite());
  				return;
  			}
 @@ -459,7 +459,7 @@ public class MobileBKUState extends State {  					if (handler.handleQRResponse(resp)) {
  						log.debug("Signature page response: " + resp);
  						getMobileBKUFingerprintComposite().setDone(true);
 -						Display display = getStateMachine().getGUIProvider().getMainShell().getDisplay();
 +						Display display = getStateMachine().getMainShell().getDisplay();
  						display.wake();
  						checkDone.cancel();
  					}
 @@ -475,9 +475,9 @@ public class MobileBKUState extends State {  			fingerprintComposite.setRefVal(status.refVal);
  			fingerprintComposite.setSignatureData(status.signatureDataURL);
  			fingerprintComposite.setErrorMessage(status.errorMessage);
 -			getStateMachine().getGUIProvider().display(fingerprintComposite);
 +			getStateMachine().display(fingerprintComposite);
 -			Display display = getStateMachine().getGUIProvider().getMainShell().getDisplay();
 +			Display display = getStateMachine().getMainShell().getDisplay();
  			while (!fingerprintComposite.isUserCancel() && !fingerprintComposite.isUserSMS() && !fingerprintComposite.isDone()) {
  				if (!display.readAndDispatch()) {
  					display.sleep();
 @@ -500,7 +500,7 @@ public class MobileBKUState extends State {  				fingerprintComposite.setDone(false);
  			// show waiting composite
 -			getStateMachine().getGUIProvider().display(this.getWaitingComposite());
 +			getStateMachine().display(this.getWaitingComposite());
  		});
  	}
 @@ -521,7 +521,7 @@ public class MobileBKUState extends State {  	 */
  	@Override
  	public void run() {
 -		this.signingState = getStateMachine().getStatus().signingState;
 +		this.signingState = getStateMachine().status.signingState;
  		this.signingState.setBKUConnector(new MobileBKUConnector(this));
  		log.debug("Setting base64 request to " + this.handler.useBase64Request());
 @@ -534,7 +534,7 @@ public class MobileBKUState extends State {  			if (errormsg != null && !errormsg.isEmpty())
  				message += ": " + errormsg;
  			ErrorDialog error = new ErrorDialog(
 -					getStateMachine().getGUIProvider().getMainShell(),
 +					getStateMachine().getMainShell(),
  					message, BUTTONS.OK);
  			// error.setException(this.threadException);
  			// this.setNextState(error);
 @@ -543,7 +543,7 @@ public class MobileBKUState extends State {  			return;
  		}
 -		getStateMachine().getGUIProvider().display(
 +		getStateMachine().display(
  				this.getWaitingComposite());
  		this.setNextState(new at.asit.pdfover.gui.workflow.states.SigningState(getStateMachine()));
 @@ -573,7 +573,7 @@ public class MobileBKUState extends State {  	 */
  	@Override
  	public void updateMainWindowBehavior() {
 -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior;
 +		MainWindowBehavior behavior = getStateMachine().status.behavior;
  		behavior.reset();
  		behavior.setActive(Buttons.OPEN, true);
  		behavior.setActive(Buttons.POSITION, true);
 diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OpenState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OpenState.java index 64de1640..4457f902 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OpenState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OpenState.java @@ -74,15 +74,15 @@ public class OpenState extends State {  	private DataSourceSelectComposite getSelectionComposite() {  		if (this.selectionComposite == null) {  			this.selectionComposite = -					getStateMachine().getGUIProvider().createComposite(DataSourceSelectComposite.class, SWT.RESIZE, this); +					getStateMachine().createComposite(DataSourceSelectComposite.class, SWT.RESIZE, this);  		}  		return this.selectionComposite;  	}  	@Override  	public void run() { -		ConfigProvider config = getStateMachine().getConfigProvider(); -		Status status = getStateMachine().getStatus(); +		ConfigProvider config = getStateMachine().configProvider; +		Status status = getStateMachine().status;  		if (!(status.getPreviousState() instanceof PrepareConfigurationState)  				&& !(status.getPreviousState() instanceof OpenState)) {  			status.bku = config.getDefaultBKU(); @@ -96,7 +96,7 @@ public class OpenState extends State {  		if (status.document == null) {  			DataSourceSelectComposite selection = this.getSelectionComposite(); -			getStateMachine().getGUIProvider().display(selection); +			getStateMachine().display(selection);  			selection.layout();  			status.document = selection.getSelected(); @@ -106,12 +106,12 @@ public class OpenState extends State {  				return;  			}  		} -		log.debug("Got Datasource: " + getStateMachine().getStatus().document.getAbsolutePath()); +		log.debug("Got Datasource: " + getStateMachine().status.document.getAbsolutePath());  		// scan for signature placeholders  		// - see if we want to scan for placeholders in the settings  		if (config.getEnablePlaceholderUsage()) { -			try (PDDocument pddocument = PDDocument.load(getStateMachine().getStatus().document)) { +			try (PDDocument pddocument = PDDocument.load(getStateMachine().status.document)) {  				// - scan for placeholders  				boolean useSignatureFields = config.getUseSignatureFields();  				boolean useMarker = config.getUseMarker(); @@ -126,7 +126,7 @@ public class OpenState extends State {  						{  							// create a dialog with ok and cancel buttons and a question  							// icon -							MessageBox dialog = new MessageBox(getStateMachine().getGUIProvider().getMainShell(), +							MessageBox dialog = new MessageBox(getStateMachine().getMainShell(),  									SWT.ICON_QUESTION | SWT.YES | SWT.NO | SWT.CANCEL);  							dialog.setText(Messages.getString("dataSourceSelection.usePlaceholderTitle"));  							dialog.setMessage(Messages.getString("dataSourceSelection.usePlaceholderText")); @@ -143,11 +143,11 @@ public class OpenState extends State {  								} else if (fields.size() > 1) {  									PlaceholderSelectionGui gui = new PlaceholderSelectionGui( -											getStateMachine().getGUIProvider().getMainShell(), 65570, "text", +											getStateMachine().getMainShell(), 65570, "text",  											"select the fields", fields);  									int res = gui.open();  									if (res != -1) { -										getStateMachine().getStatus().searchForPlacehoderSignature = true; +										getStateMachine().status.searchForPlacehoderSignature = true;  										addPlaceholderSelectionToConfig(fields.get(res));  										this.setNextState(new BKUSelectionState(getStateMachine()));  									} @@ -156,7 +156,7 @@ public class OpenState extends State {  								}  							} else if (result == SWT.NO) { -								getStateMachine().getStatus().searchForPlacehoderSignature = false; +								getStateMachine().status.searchForPlacehoderSignature = false;  							} else {  								status.document = null;  								return; @@ -172,7 +172,7 @@ public class OpenState extends State {  					if (null != signaturePlaceholderData) {  						// create a dialog with ok and cancel buttons and a question icon -						MessageBox dialog = new MessageBox(getStateMachine().getGUIProvider().getMainShell(), +						MessageBox dialog = new MessageBox(getStateMachine().getMainShell(),  								SWT.ICON_QUESTION | SWT.YES | SWT.NO | SWT.CANCEL);  						dialog.setText(Messages.getString("dataSourceSelection.usePlaceholderTitle"));  						dialog.setMessage(Messages.getString("dataSourceSelection.usePlaceholderText")); @@ -190,10 +190,10 @@ public class OpenState extends State {  									signaturePlaceholderData.getTablePos().getPosY(),  									signaturePlaceholderData.getTablePos().getPage()); -							getStateMachine().getStatus().searchForPlacehoderSignature = true; +							getStateMachine().status.searchForPlacehoderSignature = true;  						} else if (result == SWT.NO) { -							getStateMachine().getStatus().searchForPlacehoderSignature = false; +							getStateMachine().status.searchForPlacehoderSignature = false;  						} else {  							status.document = null;  							return; @@ -256,7 +256,7 @@ public class OpenState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setEnabled(Buttons.CONFIG, true);  		behavior.setEnabled(Buttons.OPEN, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OutputState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OutputState.java index bae7cb13..e8b9402d 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OutputState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/OutputState.java @@ -46,11 +46,11 @@ public class OutputState extends State {  	private OutputComposite getOutputComposite() {  		if (this.outputComposite == null) { -			this.outputComposite = getStateMachine().getGUIProvider() +			this.outputComposite = getStateMachine()  					.createComposite(OutputComposite.class, SWT.RESIZE, this); -			ConfigProvider config = getStateMachine().getConfigProvider(); -			Status status = getStateMachine().getStatus(); +			ConfigProvider config = getStateMachine().configProvider; +			Status status = getStateMachine().status;  			File tmpDir = new File(config.getConfigurationDirectory() + File.separator + "tmp"); @@ -78,10 +78,10 @@ public class OutputState extends State {  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		if (status.signResult == null) { -			ErrorDialog error = new ErrorDialog(getStateMachine().getGUIProvider().getMainShell(), +			ErrorDialog error = new ErrorDialog(getStateMachine().getMainShell(),  					Messages.getString("error.Signatur"), BUTTONS.RETRY_CANCEL);  			if(error.open() == SWT.RETRY) {  				this.setNextState(new PrepareSigningState(getStateMachine())); @@ -94,7 +94,7 @@ public class OutputState extends State {  		OutputComposite outputComposite = this.getOutputComposite();  		// Display dialog -		getStateMachine().getGUIProvider().display(outputComposite); +		getStateMachine().display(outputComposite);  	}  	/* @@ -105,7 +105,7 @@ public class OutputState extends State {  	@Override  	public void cleanUp() { -		getStateMachine().getStatus().signResult = null; +		getStateMachine().status.signResult = null;  		if (this.outputComposite != null)  			this.outputComposite.dispose(); @@ -118,7 +118,7 @@ public class OutputState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setEnabled(Buttons.CONFIG, true);  		behavior.setEnabled(Buttons.OPEN, true); 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 427edb85..6d00cce0 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 @@ -82,7 +82,7 @@ public class PositioningState extends State {  	private void openPDFDocument() throws IOException {  		closePDFDocument(); -		File documentPath = getStateMachine().getStatus().document; +		File documentPath = getStateMachine().status.document;  		PDFFile pdf = null;  		RandomAccessFile rafile = new RandomAccessFile(documentPath, "r");  		FileChannel chan = rafile.getChannel(); @@ -114,13 +114,13 @@ public class PositioningState extends State {  		StateMachine stateMachine = getStateMachine();  		if (this.positionComposite == null) {  			this.positionComposite = -					stateMachine.getGUIProvider().createComposite(PositioningComposite.class, SWT.RESIZE, this); -			log.debug("Displaying " +  stateMachine.getStatus().document); +					stateMachine.createComposite(PositioningComposite.class, SWT.RESIZE, this); +			log.debug("Displaying " +  stateMachine.status.document);  			this.positionComposite.displayDocument(document);  		}  		// Update possibly changed values -		ConfigProvider config = stateMachine.getConfigProvider(); -		SignatureParameter param = stateMachine.getPDFSigner().getPDFSigner().newParameter(); +		ConfigProvider config = stateMachine.configProvider; +		SignatureParameter param = stateMachine.pdfSigner.getPDFSigner().newParameter();  		Emblem emblem = new CachedFileNameEmblem(config.getDefaultEmblem());  		param.setEmblem(emblem);  		if(config.getSignatureNote() != null && !config.getSignatureNote().isEmpty()) { @@ -146,7 +146,7 @@ public class PositioningState extends State {  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		if (!(status.getPreviousState() instanceof PositioningState) &&  			!(status.getPreviousState() instanceof OpenState))  		{ @@ -155,7 +155,7 @@ public class PositioningState extends State {  		}  		if ((this.document == null) || -				(this.loadedDocumentPath != getStateMachine().getStatus().document)) { +				(this.loadedDocumentPath != getStateMachine().status.document)) {  			log.debug("Checking PDF document for encryption");  			try {  				openPDFDocument(); @@ -166,7 +166,7 @@ public class PositioningState extends State {  				if (message == null)  					message = Messages.getString("error.IOError");  				ErrorDialog dialog = new ErrorDialog( -						getStateMachine().getGUIProvider().getMainShell(), +						getStateMachine().getMainShell(),  						message, BUTTONS.RETRY_CANCEL);  				if(dialog.open() == SWT.RETRY) {  					run(); @@ -184,7 +184,7 @@ public class PositioningState extends State {  			} catch(Exception ex) {  				log.error("Failed to create composite (probably a mac...)", ex);  				ErrorDialog dialog = new ErrorDialog( -						getStateMachine().getGUIProvider().getMainShell(), +						getStateMachine().getMainShell(),  						Messages.getString("error.PositioningNotPossible"), BUTTONS.OK);  				dialog.open();  				status.signaturePosition = new SignaturePosition(); @@ -192,7 +192,7 @@ public class PositioningState extends State {  				return;  			} -			getStateMachine().getGUIProvider().display(position); +			getStateMachine().display(position);  			status.signaturePosition = position.getPosition(); @@ -221,7 +221,7 @@ public class PositioningState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setEnabled(Buttons.CONFIG, true);  		behavior.setEnabled(Buttons.OPEN, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareConfigurationState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareConfigurationState.java index 22f66d32..451f2b55 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareConfigurationState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareConfigurationState.java @@ -110,9 +110,9 @@ public class PrepareConfigurationState extends State {  		try {  			try { -				getStateMachine().getConfigProvider().loadConfiguration( +				getStateMachine().configProvider.loadConfiguration(  						new FileInputStream( -								getStateMachine().getConfigProvider().getConfigurationDirectory() + FILE_SEPARATOR + filename)); +								getStateMachine().configProvider.getConfigurationDirectory() + FILE_SEPARATOR + filename));  				log.info("Loaded config from file : " + filename);  			} catch (FileNotFoundException ex) { @@ -122,7 +122,7 @@ public class PrepareConfigurationState extends State {  					try {  						InputStream is = getClass().getResourceAsStream(  								Constants.RES_PKG_PATH + filename); -						getStateMachine().getConfigProvider() +						getStateMachine().configProvider  								.loadConfiguration(is);  						log.info("Loaded config from resource : " + filename); @@ -144,7 +144,7 @@ public class PrepareConfigurationState extends State {  	 * Update configuration values as necessary  	 */  	private void updateConfiguration() { -		ConfigProvider config = getStateMachine().getConfigProvider(); +		ConfigProvider config = getStateMachine().configProvider;  		//Update signature note if old default is used  		String note = config.getSignatureNote(); @@ -156,7 +156,7 @@ public class PrepareConfigurationState extends State {  	}  	private void resetSignatureNoteField(ConfigProvider config){ -		getStateMachine().getConfigManipulator().setSignatureNote( +		getStateMachine().configProvider.setSignatureNote(  			Profile.getProfile(config.getSignatureProfile()).getDefaultSignatureBlockNote(config.getLocale())  		);  	} @@ -181,7 +181,7 @@ public class PrepareConfigurationState extends State {  			inputStream = getClass().getResourceAsStream(  					Constants.RES_PKG_PATH + Constants.DEFAULT_CONFIG_FILENAME);  			pdfOverConfig = new FileOutputStream( -					getStateMachine().getConfigProvider().getConfigurationDirectory() + +					getStateMachine().configProvider.getConfigurationDirectory() +  					FILE_SEPARATOR + Constants.DEFAULT_CONFIG_FILENAME);  			while ((byteCount = inputStream.read(buffer)) >= 0) { @@ -226,7 +226,7 @@ public class PrepareConfigurationState extends State {  		try {  			inputStream = getClass().getResourceAsStream(  					Constants.RES_PKG_PATH + Constants.DEFAULT_LOG4J_FILENAME); -			String filename = getStateMachine().getConfigProvider().getConfigurationDirectory() +			String filename = getStateMachine().configProvider.getConfigurationDirectory()  					+ FILE_SEPARATOR + Constants.DEFAULT_LOG4J_FILENAME;  			pdfOverConfig = new FileOutputStream(filename); @@ -294,12 +294,12 @@ public class PrepareConfigurationState extends State {  	private void initializeConfig() throws InitializationException {  		initializeFromConfigurationFile(getStateMachine() -				.getConfigProvider().getConfigurationFile()); +				.configProvider.getConfigurationFile()); -		resetSignatureNoteField(getStateMachine().getConfigProvider()); +		resetSignatureNoteField(getStateMachine().configProvider);  		try { -			getStateMachine().getConfigManipulator().saveCurrentConfiguration(); +			getStateMachine().configProvider.saveCurrentConfiguration();  		} catch (IOException e) {  			log.error(  					"Failed to set local configuration signature note!", e); @@ -399,8 +399,8 @@ public class PrepareConfigurationState extends State {  		// Read config file  		try {  			StateMachine stateMachine = getStateMachine(); -			ConfigProvider config = stateMachine.getConfigProvider(); -			final GUIProvider gui = stateMachine.getGUIProvider(); +			ConfigProvider config = stateMachine.configProvider; +			final GUIProvider gui = stateMachine;  			String cDir = config.getConfigurationDirectory();  			File configDir = new File(cDir);  			File configFile = new File(configDir, Constants.DEFAULT_CONFIG_FILENAME); @@ -419,8 +419,7 @@ public class PrepareConfigurationState extends State {  			// Read cli arguments for config file first  			try { -				initializeFromArguments(stateMachine.getCmdArgs(), -						this.configFileHandler); +				initializeFromArguments(stateMachine.cmdLineArgs, this.configFileHandler);  			} catch (InitializationException e) {  				log.error("Error in cmd line arguments: ", e);  				ErrorDialog error = new ErrorDialog(gui.getMainShell(), @@ -437,8 +436,7 @@ public class PrepareConfigurationState extends State {  			// Read cli arguments  			try { -				initializeFromArguments(stateMachine.getCmdArgs(), -						this.handler); +				initializeFromArguments(stateMachine.cmdLineArgs, this.handler);  			} catch (InitializationException e) {  				log.error("Error in cmd line arguments: ", e);  				ErrorDialog error; @@ -501,20 +499,20 @@ public class PrepareConfigurationState extends State {  			}  			// Set usedSignerLib ... -			stateMachine.getPDFSigner().setUsedPDFSignerLibrary( +			stateMachine.pdfSigner.setUsedPDFSignerLibrary(  					Signator.Signers.PDFAS4);  			// Create PDF Signer -			Status status = stateMachine.getStatus(); -			status.bku = getStateMachine().getConfigProvider().getDefaultBKU(); -			status.signaturePosition = getStateMachine().getConfigProvider().getDefaultSignaturePosition(); +			Status status = stateMachine.status; +			status.bku = getStateMachine().configProvider.getDefaultBKU(); +			status.signaturePosition = getStateMachine().configProvider.getDefaultSignaturePosition();  			setNextState(new OpenState(stateMachine));  		} catch (InitializationException e) {  			log.error("Failed to initialize: ", e);  			ErrorDialog error = new ErrorDialog(getStateMachine() -					.getGUIProvider().getMainShell(), +					.getMainShell(),  					Messages.getString("error.Initialization"),  					BUTTONS.OK);  			// error.setException(e); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java index e00ec6e7..5bc9ec54 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java @@ -67,10 +67,9 @@ public class PrepareSigningState extends State {  		public void run() {  			try { -				Status status = this.state.getStateMachine().getStatus(); +				Status status = this.state.getStateMachine().status; -				ConfigProvider configuration = this.state.getStateMachine() -						.getConfigProvider(); +				ConfigProvider configuration = this.state.getStateMachine().configProvider;  				// SET PROXY HOST and PORT settings  				final String proxyHost = configuration.getProxyHost(); @@ -124,13 +123,11 @@ public class PrepareSigningState extends State {  					});  				}  				if (this.state.signer == null) { -					this.state.signer = this.state.getStateMachine().getPDFSigner() -							.getPDFSigner(); +					this.state.signer = this.state.getStateMachine().pdfSigner.getPDFSigner();  				}  				if (this.state.signatureParameter == null) { -					this.state.signatureParameter = this.state.signer -							.newParameter(); +					this.state.signatureParameter = this.state.signer.newParameter();  				}  				this.state.signatureParameter.setInputDocument(new PDFFileDocumentSource(status.document)); @@ -149,7 +146,7 @@ public class PrepareSigningState extends State {  							"SIG_NOTE", configuration.getSignatureNote());  				} -				this.state.signatureParameter.setSearchForPlaceholderSignatures(getStateMachine().getStatus().searchForPlacehoderSignature); +				this.state.signatureParameter.setSearchForPlaceholderSignatures(getStateMachine().status.searchForPlacehoderSignature);  				this.state.signatureParameter.setSignatureLanguage(configuration.getSignatureLocale().getLanguage()); @@ -180,7 +177,7 @@ public class PrepareSigningState extends State {  	private WaitingComposite getSelectionComposite() {  		if (this.waitingComposite == null) { -			this.waitingComposite = getStateMachine().getGUIProvider() +			this.waitingComposite = getStateMachine()  					.createComposite(WaitingComposite.class, SWT.RESIZE, this);  		} @@ -197,11 +194,11 @@ public class PrepareSigningState extends State {  	public void run() {  		WaitingComposite waiting = this.getSelectionComposite(); -		getStateMachine().getGUIProvider().display(waiting); +		getStateMachine().display(waiting); -		this.signer = getStateMachine().getPDFSigner().getPDFSigner(); +		this.signer = getStateMachine().pdfSigner.getPDFSigner(); -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		if (this.signatureParameter == null) {  			this.signatureParameter = this.signer.newParameter(); @@ -215,7 +212,7 @@ public class PrepareSigningState extends State {  		if (this.threadException != null) {  			ErrorDialog error = new ErrorDialog(getStateMachine() -					.getGUIProvider().getMainShell(), +					.getMainShell(),  					Messages.getString("error.PrepareDocument"),  					BUTTONS.RETRY_CANCEL);  			this.threadException = null; @@ -263,7 +260,7 @@ public class PrepareSigningState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setActive(Buttons.OPEN, true);  		behavior.setActive(Buttons.POSITION, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/SigningState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/SigningState.java index 1bf901d3..82cdb31c 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/SigningState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/SigningState.java @@ -54,8 +54,8 @@ public class SigningState extends State {  		@Override  		public void run() {  			try { -				Signer signer = this.state.getStateMachine().getPDFSigner().getPDFSigner(); -				Status status = this.state.getStateMachine().getStatus(); +				Signer signer = this.state.getStateMachine().pdfSigner.getPDFSigner(); +				Status status = this.state.getStateMachine().status;  				status.signResult = signer.sign(status.signingState);  			} catch(Exception e) { @@ -82,7 +82,7 @@ public class SigningState extends State {  	@Override  	public void run() { -		Status status = getStateMachine().getStatus(); +		Status status = getStateMachine().status;  		if(status.signResult == null &&  			this.threadException == null) { @@ -120,7 +120,7 @@ public class SigningState extends State {  			// if we have gotten to this point, this is an actual exception  			log.error("FinishSignThread: ", this.threadException); -			ErrorDialog error = new ErrorDialog(getStateMachine().getGUIProvider().getMainShell(), +			ErrorDialog error = new ErrorDialog(getStateMachine().getMainShell(),  					message, BUTTONS.RETRY_CANCEL);  			this.threadException = null;  			if(error.open() == SWT.RETRY) { @@ -147,7 +147,7 @@ public class SigningState extends State {  	 */  	@Override  	public void updateMainWindowBehavior() { -		MainWindowBehavior behavior = getStateMachine().getStatus().behavior; +		MainWindowBehavior behavior = getStateMachine().status.behavior;  		behavior.reset();  		behavior.setActive(Buttons.OPEN, true);  		behavior.setActive(Buttons.POSITION, true); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/State.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/State.java index 55aee598..a6cca51f 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/State.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/State.java @@ -84,7 +84,7 @@ public abstract class State {  	/**  	 * Update the main window behavior of this state if necessary -	 * Should update this.stateMachine.getStatus().getBehavior() +	 * Should update this.stateMachine.status.getBehavior()  	 */  	public abstract void updateMainWindowBehavior();  } | 
