diff options
author | tkellner <tkellner@3a0b52a2-8410-0410-bc02-ff6273a87459> | 2012-10-01 08:30:43 +0000 |
---|---|---|
committer | tkellner <tkellner@3a0b52a2-8410-0410-bc02-ff6273a87459> | 2012-10-01 08:30:43 +0000 |
commit | daf7891de6360b416deece201b342a3de9f52a07 (patch) | |
tree | d1f1abf0d4f3c2991c5cc090dc227c112fcf87ef /trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java | |
parent | e48b7b84c057762ce8e8fa43d26814711752d399 (diff) | |
download | pdf-over-daf7891de6360b416deece201b342a3de9f52a07.tar.gz pdf-over-daf7891de6360b416deece201b342a3de9f52a07.tar.bz2 pdf-over-daf7891de6360b416deece201b342a3de9f52a07.zip |
Removed Error State and replaced it with ErrorDialog
To get a stable user experience in case of an error.
ErrorDialog was improved to let the user decide if he wants to retry an
action.
git-svn-id: https://svn.iaik.tugraz.at/svn/egiz/prj/current/12PDF-OVER-4.0@12539 3a0b52a2-8410-0410-bc02-ff6273a87459
Diffstat (limited to 'trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java')
-rw-r--r-- | trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java | 44 |
1 files changed, 25 insertions, 19 deletions
diff --git a/trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java b/trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java index d95bb466..bb6d47a5 100644 --- a/trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java +++ b/trunk/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ConfigurationComposite.java @@ -54,9 +54,9 @@ public class ConfigurationComposite extends StateComposite { */ private final class ConfigurationModeSelectionListener implements SelectionListener { - + /** - * Constructor + * Constructor */ public ConfigurationModeSelectionListener() { // Nothing to do @@ -117,13 +117,12 @@ public class ConfigurationComposite extends StateComposite { BaseConfigurationComposite configComposite; /** - * configuration container - * Keeps state for current configuration changes + * configuration container Keeps state for current configuration changes */ ConfigurationContainer configurationContainer = new ConfigurationContainerImpl(); /** - * The stack layout + * The stack layout */ StackLayout compositeStack = new StackLayout(); @@ -290,8 +289,7 @@ public class ConfigurationComposite extends StateComposite { this.configManipulator .setDefaultSignaturePosition(new SignaturePosition()); } else { - this.configManipulator - .setDefaultSignaturePosition(null); + this.configManipulator.setDefaultSignaturePosition(null); } this.configManipulator @@ -311,21 +309,29 @@ public class ConfigurationComposite extends StateComposite { getShell(), SWT.NONE, "Invalid settings are still present. Please check your input.", - e); + e, false); dialog.open(); return false; } - // Save current config to file - try { - this.configManipulator.saveCurrentConfiguration(); - } catch (IOException e) { - log.error("Failed to save configuration to file!", e); //$NON-NLS-1$ - ErrorDialog dialog = new ErrorDialog(getShell(), SWT.NONE, - "Failed to save configuration file!", e); - dialog.open(); - return false; - } - return true; + + boolean status = false; + boolean redo = false; + do { + // Save current config to file + try { + this.configManipulator.saveCurrentConfiguration(); + redo = false; + status = true; + } catch (IOException e) { + log.error("Failed to save configuration to file!", e); //$NON-NLS-1$ + ErrorDialog dialog = new ErrorDialog(getShell(), SWT.NONE, + "Failed to save configuration file!", e, true); + redo = dialog.open(); + + //return false; + } + } while (redo); + return status; } /** |