From fed2f58dda88649d78a63e2aa3e63705677cbed9 Mon Sep 17 00:00:00 2001 From: Tobias Kellner Date: Thu, 30 Jun 2016 15:37:01 +0200 Subject: Replace signature note if it's the previous default --- .../workflow/states/PrepareConfigurationState.java | 41 ++++++++++++++++------ 1 file changed, 30 insertions(+), 11 deletions(-) (limited to 'pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow') 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 327bb0c3..b1af315e 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 @@ -29,6 +29,7 @@ import java.io.IOException; import java.io.InputStream; import java.io.OutputStream; import java.net.URI; +import java.util.Locale; import org.apache.commons.httpclient.HttpClient; import org.apache.commons.httpclient.methods.GetMethod; @@ -118,7 +119,7 @@ public class PrepareConfigurationState extends State { // we only check for resource config file if it is the // default value! try { - InputStream is = this.getClass().getResourceAsStream( + InputStream is = getClass().getResourceAsStream( Constants.RES_PKG_PATH + filename); getStateMachine().getConfigProvider() .loadConfiguration(is); @@ -138,6 +139,23 @@ public class PrepareConfigurationState extends State { } } + /** + * Update configuration values as necessary + */ + private void updateConfiguration() { + ConfigProvider config = getStateMachine().getConfigProvider(); + + //Update signature note if old default is used + String note = config.getSignatureNote(); + Locale loc = config.getSignatureLocale(); + + String note_old = Messages.getString("simple_config.Note_Default_Old", loc); //$NON-NLS-1$ + if (note.equals(note_old)) { + note = Messages.getString("simple_config.Note_Default", loc); //$NON-NLS-1$ + getStateMachine().getConfigManipulator().setSignatureNote(note); + } + } + private void initializeFromArguments(String[] args, ArgumentHandler handler) throws InitializationException { handler.handleArguments(args); @@ -155,7 +173,7 @@ public class PrepareConfigurationState extends State { InputStream inputStream = null; FileOutputStream pdfOverConfig = null; try { - inputStream = this.getClass().getResourceAsStream( + inputStream = getClass().getResourceAsStream( Constants.RES_PKG_PATH + Constants.DEFAULT_CONFIG_FILENAME); pdfOverConfig = new FileOutputStream( getStateMachine().getConfigProvider().getConfigurationDirectory() + @@ -199,7 +217,7 @@ public class PrepareConfigurationState extends State { InputStream inputStream = null; FileOutputStream pdfOverConfig = null; try { - inputStream = this.getClass().getResourceAsStream( + inputStream = getClass().getResourceAsStream( Constants.RES_PKG_PATH + Constants.DEFAULT_LOG4J_FILENAME); String filename = getStateMachine().getConfigProvider().getConfigurationDirectory() + FILE_SEPARATOR + Constants.DEFAULT_LOG4J_FILENAME; @@ -238,7 +256,7 @@ public class PrepareConfigurationState extends State { } private void unzipPdfAsConfig(File configDir) throws InitializationException { - InputStream is = this.getClass().getResourceAsStream(Constants.RES_CFG_ZIP); + InputStream is = getClass().getResourceAsStream(Constants.RES_CFG_ZIP); try { Zipper.unzip(is, configDir.getAbsolutePath()); @@ -268,7 +286,7 @@ public class PrepareConfigurationState extends State { } private void initializeConfig() throws InitializationException { - this.initializeFromConfigurationFile(getStateMachine() + initializeFromConfigurationFile(getStateMachine() .getConfigProvider().getConfigurationFile()); getStateMachine().getConfigManipulator().setSignatureNote(Messages.getString("simple_config.Note_Default")); //$NON-NLS-1$ @@ -394,7 +412,7 @@ public class PrepareConfigurationState extends State { // Read cli arguments for config file first try { - this.initializeFromArguments(stateMachine.getCmdArgs(), + initializeFromArguments(stateMachine.getCmdArgs(), this.configFileHandler); } catch (InitializationException e) { log.error("Error in cmd line arguments: ", e); //$NON-NLS-1$ @@ -407,11 +425,12 @@ public class PrepareConfigurationState extends State { } // initialize from config file - this.initializeFromConfigurationFile(config.getConfigurationFile()); + initializeFromConfigurationFile(config.getConfigurationFile()); + updateConfiguration(); // Read cli arguments try { - this.initializeFromArguments(stateMachine.getCmdArgs(), + initializeFromArguments(stateMachine.getCmdArgs(), this.handler); } catch (InitializationException e) { log.error("Error in cmd line arguments: ", e); //$NON-NLS-1$ @@ -487,7 +506,7 @@ public class PrepareConfigurationState extends State { status.setSignaturePosition(getStateMachine().getConfigProvider() .getDefaultSignaturePosition()); - this.setNextState(new OpenState(stateMachine)); + setNextState(new OpenState(stateMachine)); } catch (InitializationException e) { log.error("Failed to initialize: ", e); //$NON-NLS-1$ @@ -496,7 +515,7 @@ public class PrepareConfigurationState extends State { Messages.getString("error.Initialization"), //$NON-NLS-1$ BUTTONS.OK); // error.setException(e); - // this.setNextState(error); + // setNextState(error); error.open(); getStateMachine().exit(); } @@ -524,6 +543,6 @@ public class PrepareConfigurationState extends State { @Override public String toString() { - return this.getClass().getName(); + return getClass().getName(); } } -- cgit v1.2.3