From fdc0f9a9dfc7dabdc96bb830a079a353f160395c Mon Sep 17 00:00:00 2001 From: tkellner Date: Wed, 10 Apr 2013 18:57:41 +0000 Subject: Fixed some TODOs... git-svn-id: https://joinup.ec.europa.eu/svn/pdf-over/trunk@57 174cde9d-5d70-4d2a-aa98-46368bc2aaf7 --- .../gui/composites/ConfigurationComposite.java | 33 ++++++++++++++++- .../gui/composites/DataSourceSelectComposite.java | 2 +- .../pdfover/gui/composites/ErrorComposite.java | 2 +- .../composites/MobileBKUEnterNumberComposite.java | 3 +- .../gui/composites/MobileBKUEnterTANComposite.java | 5 +-- .../pdfover/gui/composites/OutputComposite.java | 8 ++-- .../composites/SimpleConfigurationComposite.java | 15 ++++---- .../asit/pdfover/gui/controls/MainBarButton.java | 43 ++++++++++++++-------- .../pdfover/gui/controls/MainBarEndButton.java | 1 + .../pdfover/gui/controls/MainBarMiddleButton.java | 1 + .../gui/controls/MainBarRectangleButton.java | 1 + .../pdfover/gui/controls/MainBarStartButton.java | 1 + .../asit/pdfover/gui/workflow/ConfigProvider.java | 2 - .../pdfover/gui/workflow/StateMachineImpl.java | 10 ++--- .../pdfover/gui/workflow/states/ErrorState.java | 3 +- .../pdfover/gui/workflow/states/LocalBKUState.java | 20 +++++----- .../pdfover/gui/workflow/states/OutputState.java | 15 -------- .../gui/workflow/states/PrepareSigningState.java | 7 ---- .../pdfover/gui/workflow/states/SigningState.java | 7 +--- .../workflow/states/mobilebku/ATrustHelper.java | 2 - .../states/mobilebku/PostSLRequestThread.java | 2 +- 21 files changed, 96 insertions(+), 87 deletions(-) (limited to 'pdf-over-gui') diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java index 0e78e507..d95bb466 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java @@ -50,12 +50,13 @@ import org.eclipse.swt.layout.FormAttachment; public class ConfigurationComposite extends StateComposite { /** - * + * Configuration Mode selection listener */ private final class ConfigurationModeSelectionListener implements SelectionListener { + /** - * + * Constructor */ public ConfigurationModeSelectionListener() { // Nothing to do @@ -100,22 +101,50 @@ public class ConfigurationComposite extends StateComposite { private static final Logger log = LoggerFactory .getLogger(ConfigurationComposite.class); + /** + * configuration manipulator + */ ConfigManipulator configManipulator = null; + /** + * configuration provider + */ ConfigProvider configProvider = null; + /** + * current base configuration composite + */ BaseConfigurationComposite configComposite; + /** + * configuration container + * Keeps state for current configuration changes + */ ConfigurationContainer configurationContainer = new ConfigurationContainerImpl(); + /** + * The stack layout + */ StackLayout compositeStack = new StackLayout(); + /** + * SWT style + */ int style; + /** + * base configuration container + */ Composite containerComposite; + /** + * checks wether the user is done + */ boolean userDone = false; + /** + * Button advanced + */ Button btnAdvanced; /** diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java index afa7be3e..8146a02d 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java @@ -195,7 +195,7 @@ public class DataSourceSelectComposite extends StateComposite { } }); - final Label lbl_drag = new Label(this.drop_area, SWT.NONE | SWT.RESIZE); + final Label lbl_drag = new Label(this.drop_area, SWT.NONE | SWT.RESIZE ); this.fd_lbl_drag = new FormData(); this.fd_lbl_drag.left = new FormAttachment(0, 10); this.fd_lbl_drag.right = new FormAttachment(100, -10); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java index f8a6fbcb..c85d046e 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java @@ -114,7 +114,7 @@ public class ErrorComposite extends StateComposite { */ @Override public void doLayout() { - // TODO Auto-generated method stub + // Nothing to do } } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterNumberComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterNumberComposite.java index 595ca1bd..6552c3d5 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterNumberComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterNumberComposite.java @@ -99,8 +99,7 @@ public class MobileBKUEnterNumberComposite extends StateComposite { */ @Override public void widgetDefaultSelected(SelectionEvent e) { - // TODO Auto-generated method stub - + // Nothing to do here } } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterTANComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterTANComposite.java index 95184775..ff8d5c63 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterTANComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/MobileBKUEnterTANComposite.java @@ -61,8 +61,6 @@ public class MobileBKUEnterTANComposite extends StateComposite { return; } - // TODO: Check TAN! - MobileBKUEnterTANComposite.this.tan = tan; MobileBKUEnterTANComposite.this.setUserAck(true); MobileBKUEnterTANComposite.this.state.updateStateMachine(); @@ -227,8 +225,7 @@ public class MobileBKUEnterTANComposite extends StateComposite { */ @Override public void doLayout() { - // TODO Auto-generated method stub - + // Nothing to do } } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java index 26e5480c..afa2a2e7 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java @@ -32,6 +32,7 @@ import org.eclipse.swt.widgets.FileDialog; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import at.asit.pdfover.gui.controls.ErrorDialog; import at.asit.pdfover.gui.workflow.states.State; import at.asit.pdfover.signator.DocumentSource; @@ -116,7 +117,9 @@ public class OutputComposite extends StateComposite { return; } } else { - // TODO: Handle exception ... + log.error("OutputComposite:OpenSelectionListener:widgetSelected -> source is null!!"); //$NON-NLS-1$ + ErrorDialog dialog = new ErrorDialog(getShell(), SWT.NONE, "Failed to get signed document.", ""); + dialog.open(); } } catch (Exception ex) { log.error("OpenSelectionListener: ", ex); //$NON-NLS-1$ @@ -200,8 +203,7 @@ public class OutputComposite extends StateComposite { */ @Override public void doLayout() { - // TODO Auto-generated method stub - + // Nothing to do } } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/SimpleConfigurationComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/SimpleConfigurationComposite.java index 3830d6cf..a0977b9e 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/SimpleConfigurationComposite.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/SimpleConfigurationComposite.java @@ -54,6 +54,7 @@ import org.eclipse.swt.widgets.Text; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import at.asit.pdfover.gui.controls.ErrorDialog; import at.asit.pdfover.gui.controls.ErrorMarker; import at.asit.pdfover.gui.exceptions.InvalidEmblemFile; import at.asit.pdfover.gui.exceptions.InvalidNumberException; @@ -75,7 +76,7 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { * */ public ImageFileBrowser() { - // TODO Auto-generated constructor stub + // Nothing to do } @Override @@ -224,8 +225,9 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { // String filename = this.txtEmblemFile.getText(); plainEmblemSetter(filename); } catch (Exception ex) { - // TODO: Show error message! log.error("processEmblemChanged: ", ex); //$NON-NLS-1$ + ErrorDialog dialog = new ErrorDialog(getShell(), SWT.NONE, "Failed to load the emblem", ex); + dialog.open(); } } @@ -244,7 +246,6 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { this.txtMobileNumberErrorMarker.setVisible(false); plainMobileNumberSetter(); } catch (Exception ex) { - // TODO: Show error message! this.txtMobileNumberErrorMarker.setVisible(true); this.txtMobileNumberErrorMarker .setToolTipText("Phone number is invalid! Please provide in the form: +43676123456789"); @@ -271,7 +272,6 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { this.proxyHostErrorMarker.setVisible(false); plainProxyHostSetter(); } catch (Exception ex) { - // TODO: Show error message! this.proxyHostErrorMarker.setVisible(true); this.proxyHostErrorMarker.setToolTipText(ex.getMessage()); log.error("processProxyHost: ", ex); //$NON-NLS-1$ @@ -291,7 +291,6 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { this.txtProxyPortErrorMarker.setVisible(false); plainProxyPortSetter(); } catch (Exception ex) { - // TODO: Show error message! this.txtProxyPortErrorMarker.setVisible(true); this.txtProxyPortErrorMarker.setToolTipText(ex.getMessage()); log.error("processProxyPort: ", ex); //$NON-NLS-1$ @@ -513,8 +512,7 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { SimpleConfigurationComposite.this.configurationContainer .setEmblem(null); } catch (InvalidEmblemFile e1) { - // TODO Auto-generated catch block - e1.printStackTrace(); + log.error("THIS EXCEPTION IS IMPOSSIBLE! ", e1); //$NON-NLS-1$ } } } @@ -729,8 +727,9 @@ public class SimpleConfigurationComposite extends BaseConfigurationComposite { this.setEmblemFileInternal(emblemFile, true); this.btnUseImage.setSelection(true); } catch (Exception e1) { - // TODO Auto-generated catch block log.error("Failed to load emblem: ", e1); //$NON-NLS-1$ + ErrorDialog dialog = new ErrorDialog(getShell(), SWT.NONE, "Failed to load emblem.", e1); + dialog.open(); } } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarButton.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarButton.java index 37e4f85b..45a8d230 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarButton.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarButton.java @@ -17,17 +17,12 @@ package at.asit.pdfover.gui.controls; // Imports import org.eclipse.swt.SWT; -import org.eclipse.swt.events.MouseAdapter; -import org.eclipse.swt.events.MouseEvent; -import org.eclipse.swt.events.MouseListener; -import org.eclipse.swt.events.MouseMoveListener; import org.eclipse.swt.events.PaintEvent; import org.eclipse.swt.events.PaintListener; import org.eclipse.swt.graphics.Color; import org.eclipse.swt.graphics.Cursor; import org.eclipse.swt.graphics.Image; import org.eclipse.swt.graphics.Point; -import org.eclipse.swt.graphics.Rectangle; import org.eclipse.swt.graphics.Region; import org.eclipse.swt.widgets.Canvas; import org.eclipse.swt.widgets.Composite; @@ -37,7 +32,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; /** - * + * Main Bar Button implementation */ public abstract class MainBarButton extends Canvas { /** @@ -100,15 +95,16 @@ public abstract class MainBarButton extends Canvas { private Image image = null; /** - * @return the imgage + * Gets the image + * @return the image */ public Image getImage() { return this.image; } /** - * @param imgage - * the imgage to set + * Sets the Image + * @param image the imgage to set */ public void setImage(Image image) { this.image = image; @@ -140,6 +136,7 @@ public abstract class MainBarButton extends Canvas { } /** + * Gets the button text * @return the text */ public String getText() { @@ -147,8 +144,8 @@ public abstract class MainBarButton extends Canvas { } /** - * @param text - * the text to set + * Sets the text for the button + * @param text the text to set */ public void setText(String text) { this.text = text; @@ -157,9 +154,15 @@ public abstract class MainBarButton extends Canvas { /** * SLF4J Logger instance **/ + @SuppressWarnings("unused") private static final Logger log = LoggerFactory .getLogger(MainBarButton.class); + /** + * Paint 3D style borders + * + * @param e + */ protected void paintBackground(PaintEvent e) { Point size = this.getSize(); @@ -185,20 +188,29 @@ public abstract class MainBarButton extends Canvas { } - private void paintControl(PaintEvent e) { + /** + * Main painting method + * @param e + */ + void paintControl(PaintEvent e) { this.paintBackground(e); this.paintButton(e); - this.paintBorderAndText(e); + this.paintText(e); } + /** + * paint the inner button + * @param e + */ protected void paintButton(PaintEvent e) { // could be overwritten by subclasses } /** + * Paint the text or image on the button * @param e */ - protected void paintBorderAndText(PaintEvent e) { + protected void paintText(PaintEvent e) { Point size = this.getSize(); int height = size.y - 2; @@ -227,7 +239,8 @@ public abstract class MainBarButton extends Canvas { } /** - * @return + * Gets the region of the button + * @return the button region */ protected abstract Region getCustomRegion(); } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarEndButton.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarEndButton.java index acbff332..923df73e 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarEndButton.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarEndButton.java @@ -38,6 +38,7 @@ public class MainBarEndButton extends MainBarButton { /** * SLF4J Logger instance **/ + @SuppressWarnings("unused") private static final Logger log = LoggerFactory .getLogger(MainBarEndButton.class); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarMiddleButton.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarMiddleButton.java index f46e7e42..5e64826f 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarMiddleButton.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarMiddleButton.java @@ -38,6 +38,7 @@ public class MainBarMiddleButton extends MainBarButton { /** * SLF4J Logger instance **/ + @SuppressWarnings("unused") private static final Logger log = LoggerFactory .getLogger(MainBarMiddleButton.class); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarRectangleButton.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarRectangleButton.java index 39392307..aff1fa7e 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarRectangleButton.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarRectangleButton.java @@ -40,6 +40,7 @@ public class MainBarRectangleButton extends MainBarButton { /** * SLF4J Logger instance **/ + @SuppressWarnings("unused") private static final Logger log = LoggerFactory .getLogger(MainBarRectangleButton.class); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarStartButton.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarStartButton.java index 9101e274..8cdafefa 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarStartButton.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/controls/MainBarStartButton.java @@ -38,6 +38,7 @@ public class MainBarStartButton extends MainBarButton { /** * SLF4J Logger instance **/ + @SuppressWarnings("unused") private static final Logger log = LoggerFactory .getLogger(MainBarStartButton.class); diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/ConfigProvider.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/ConfigProvider.java index 1adc45a4..4850a313 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/ConfigProvider.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/ConfigProvider.java @@ -25,8 +25,6 @@ import at.asit.pdfover.signator.SignaturePosition; * */ public interface ConfigProvider { - //TODO: define interface for config provider .... - /** * Regex for parsing signature position */ diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachineImpl.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachineImpl.java index 90b2fba9..ece1cf7f 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachineImpl.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachineImpl.java @@ -18,6 +18,7 @@ package at.asit.pdfover.gui.workflow; //Imports import java.lang.reflect.Constructor; +import org.eclipse.swt.SWTException; import org.eclipse.swt.widgets.Composite; import org.eclipse.swt.widgets.Display; import org.eclipse.swt.widgets.Shell; @@ -25,6 +26,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import at.asit.pdfover.gui.MainWindow; +import at.asit.pdfover.gui.workflow.states.ErrorState; import at.asit.pdfover.gui.workflow.states.PrepareConfigurationState; import at.asit.pdfover.gui.workflow.states.State; @@ -84,7 +86,9 @@ public class StateMachineImpl implements StateMachine, GUIProvider { current.run(); } catch (Exception e) { log.error("StateMachine update: ", e); //$NON-NLS-1$ - // TODO: GOTO generic error state! + ErrorState errorState = new ErrorState(this); + errorState.setException(e); + jumpToState(errorState); } if (this.exit) { @@ -203,10 +207,6 @@ public class StateMachineImpl implements StateMachine, GUIProvider { this.createMainWindow(); } - if (this.container == null) { - // TODO throw Exception... - } - return this.container; } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ErrorState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ErrorState.java index af648dfd..dd240616 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ErrorState.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/ErrorState.java @@ -100,8 +100,7 @@ public class ErrorState extends State { */ @Override public void updateMainWindowBehavior() { - // TODO Auto-generated method stub - + // Stay in previous state behavior } /** 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 ca0e164c..9962d968 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 @@ -20,11 +20,14 @@ import org.apache.commons.httpclient.HttpClient; import org.apache.commons.httpclient.HttpException; import org.apache.commons.httpclient.HttpStatus; import org.apache.commons.httpclient.methods.PostMethod; +import org.eclipse.swt.SWT; +import org.eclipse.swt.widgets.Display; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import at.asit.pdfover.gui.MainWindow.Buttons; import at.asit.pdfover.gui.MainWindowBehavior; +import at.asit.pdfover.gui.controls.ErrorDialog; import at.asit.pdfover.gui.workflow.StateMachine; import at.asit.pdfover.gui.workflow.Status; import at.asit.pdfover.signator.SLRequest; @@ -86,7 +89,7 @@ public class LocalBKUState extends State { log.debug("SL REQUEST: " + sl_request); //$NON-NLS-1$ method.addParameter("XMLRequest", sl_request); //$NON-NLS-1$ - + int returnCode = client.executeMethod(method); if(returnCode == HttpStatus.SC_OK) @@ -115,13 +118,12 @@ public class LocalBKUState extends State { SLResponse slResponse = new SLResponse(response, server, userAgent, signatureLayout); this.state.signingState.setSignatureResponse(slResponse); } else { - // TODO: Create HTTP exception this.state.threadException = new HttpException(method.getResponseBodyAsString()); } } catch (Exception e) { log.error("SignLocalBKUThread: ", e); //$NON-NLS-1$ - // TODO: Is local BKU running? + // this.state.threadException = e; } finally { this.state.stateMachine.invokeUpdate(); @@ -168,17 +170,13 @@ public class LocalBKUState extends State { } if(this.threadException != null) { - ErrorState error = new ErrorState(this.stateMachine); - error.setException(this.threadException); - this.setNextState(error); + ErrorDialog dialog = new ErrorDialog(Display.getCurrent().getActiveShell(), SWT.NONE, "Please check if a local BKU is running", this.threadException); + dialog.open(); + this.threadException = null; + this.run(); return; } - if(!this.signingState.hasSignatureResponse()) { - // The thread should set the response or the thread exception!!! - // TODO: Jump to error state! - } - // OK this.setNextState(new SigningState(this.stateMachine)); } 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 2ebc3f39..a1748287 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 @@ -66,8 +66,6 @@ public class OutputState extends State { @Override public void run() { - // TODO Preform output operations ... end workflow - Status status = this.stateMachine.getStatus(); if (status.getSignResult() != null) { @@ -97,20 +95,7 @@ public class OutputState extends State { } } - /* - * DocumentSource signedDocument = - * status.getSignResult().getSignedDocument(); - * - * FileOutputStream output; try { output = new FileOutputStream(new - * File("/tmp/test.pdf")); - * output.write(signedDocument.getByteArray(), 0, - * signedDocument.getByteArray().length); output.close(); } catch - * (IOException e) { // TODO Auto-generated catch block - * e.printStackTrace(); } - */ } - - // this.stateMachine.exit(); } /* 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 88acc84b..af6cc3bd 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 @@ -76,8 +76,6 @@ public class PrepareSigningState extends State { this.state.signatureParameter.setSignatureDevice(status.getBKU()); this.state.signatureParameter.setSignaturePosition(status.getSignaturePosition()); - // TODO: Fill library specific signature Parameters ... - if(configuration.getDefaultEmblem() != null && !configuration.getDefaultEmblem().equals("")) { //$NON-NLS-1$ this.state.signatureParameter.setEmblem(new FileNameEmlbem(configuration.getDefaultEmblem())); } @@ -145,11 +143,6 @@ public class PrepareSigningState extends State { return; } - if(this.signingState == null || this.signingState.getSignatureRequest() == null) { - // This shouldnot happen!! PrepareDocumentThread allready performed, either we have a valid signingState or an exception!! - // TODO: Jump to error state! - } - // We got the Request set it into status and move on to next state ... status.setSigningState(this.signingState); 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 20813f00..413f125c 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 @@ -89,12 +89,7 @@ public class SigningState extends State { this.setNextState(error); return; } - - if(status.getSignResult() == null) { - // The thread should set the threadException or create a valid signResult - // TODO: Jump to error state - } - + this.setNextState(new OutputState(this.stateMachine)); } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/ATrustHelper.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/ATrustHelper.java index a1868462..ee984e96 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/ATrustHelper.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/ATrustHelper.java @@ -67,11 +67,9 @@ public class ATrustHelper { if (endidx > startidx) { return data.substring(startidx, endidx); } - // TODO: throw proper exception log.error("extracting Tag: end tag not valid!: " + start + " ... " + end); //$NON-NLS-1$//$NON-NLS-2$ throw new Exception("end tag not available!"); //$NON-NLS-1$ } - // TODO: throw proper exception log.error("extracting Tag: start tag not valid!: " + start + " ... " + end); //$NON-NLS-1$//$NON-NLS-2$ throw new Exception("start tag not available!"); //$NON-NLS-1$ } diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/PostSLRequestThread.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/PostSLRequestThread.java index bd2f5b53..fb52fce8 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/PostSLRequestThread.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/mobilebku/PostSLRequestThread.java @@ -77,7 +77,7 @@ public class PostSLRequestThread implements Runnable { PostMethod method = new PostMethod(url); method.addParameter("XMLRequest", sl_request); //$NON-NLS-1$ - + int returnCode = client.executeMethod(method); String redirectLocation = null; -- cgit v1.2.3