Add alias resign bug fixed

Author: gerardszczepanski

runit-view/src/main/java/szczepanski/gerard/runnit/view/controller/SettingsDirectoriesPaneTabController.java

@@ -51,14 +51,29 @@ private void updateDirAliases(Settings settings) {
 	@FXML
 	public void handleAddDirAlias() {
 		LOG.debug("handleAddDirAlias");
-		Alias newDirAlias = getNewAliasFromUserInput();
-		addDirAlias(newDirAlias);
+		Optional<Alias> optionalNewDirAlias = getNewAliasFromUserInput();
+		
+		if (optionalNewDirAlias.isPresent()) {
+			addDirAlias(optionalNewDirAlias.get());
+		}
 	}
 
-	private Alias getNewAliasFromUserInput() {
-		Optional<String> optAliasName = DialogDisplayer.showInputDialog(getStage(dirAliasTableView), "Web Alias Name");
-		Optional<String> optAliasValue = DialogDisplayer.showInputDialog(getStage(dirAliasTableView), "Directory path");
-		return new Alias(optAliasName.get(), optAliasValue.get());
+	/**
+	 *  TODO Gerard Szczepanski 22.01.2017 -> Ugly method, to future refactor
+	 */
+	private Optional<Alias> getNewAliasFromUserInput() {
+		Optional<String> optAliasName = DialogDisplayer.showInputDialog(getStage(dirAliasTableView), "Directory Alias Name");
+		Optional<String> optAliasValue = Optional.empty();
+		
+		if (optAliasName.isPresent()) {
+			optAliasValue = DialogDisplayer.showInputDialog(getStage(dirAliasTableView), "Directory path");
+		}
+		
+		if (optAliasName.isPresent() && optAliasValue.isPresent()) {
+			return Optional.of(new Alias(optAliasName.get(), optAliasValue.get()));
+		}
+		
+		return Optional.empty();
 	}
 
 	private void addDirAlias(Alias newDirAlias) {

runit-view/src/main/java/szczepanski/gerard/runnit/view/controller/SettingsWebPaneTabController.java

@@ -51,14 +51,28 @@ private void updateWebAliases(Settings settings) {
 	@FXML
 	public void handleAddWebAlias() {
 		LOG.debug("handleAddWebAlias");
-		Alias newWebAlias = getNewAliasFromUserInput();
-		addWebAlias(newWebAlias);
+		Optional<Alias> optionalNewWebAlias = getNewAliasFromUserInput();
+		if(optionalNewWebAlias.isPresent()) {
+			addWebAlias(optionalNewWebAlias.get());	
+		}
 	}
 
-	private Alias getNewAliasFromUserInput() {
+	/**
+	 *  TODO Gerard Szczepanski 22.01.2017 -> Ugly method, to future refactor
+	 */
+	private Optional<Alias> getNewAliasFromUserInput() {
 		Optional<String> optAliasName = DialogDisplayer.showInputDialog(getStage(webAliasTableView), "Web Alias Name");
-		Optional<String> optAliasValue = DialogDisplayer.showInputDialog(getStage(webAliasTableView), "Web Alias Address");
-		return new Alias(optAliasName.get(), optAliasValue.get());
+		Optional<String> optAliasValue = Optional.empty();
+		
+		if (optAliasName.isPresent()) {
+			optAliasValue = DialogDisplayer.showInputDialog(getStage(webAliasTableView), "Web Alias Address");
+		}
+		
+		if (optAliasName.isPresent() && optAliasValue.isPresent()) {
+			return Optional.of(new Alias(optAliasName.get(), optAliasValue.get()));
+		}
+		
+		return Optional.empty();
 	}
 
 	private void addWebAlias(Alias newWebAlias) {

runit.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<launch4jConfig>
+  <dontWrapJar>false</dontWrapJar>
+  <headerType>gui</headerType>
+  <jar>C:\Users\Ziemniak\Documents\runit\runit-program\target\runit-program-0.9.jar</jar>
+  <outfile>C:\Users\Ziemniak\Desktop\runit-exe\runit-0.9.exe</outfile>
+  <errTitle></errTitle>
+  <cmdLine></cmdLine>
+  <chdir>.</chdir>
+  <priority>normal</priority>
+  <downloadUrl>http://java.com/download</downloadUrl>
+  <supportUrl></supportUrl>
+  <stayAlive>false</stayAlive>
+  <restartOnCrash>false</restartOnCrash>
+  <manifest></manifest>
+  <icon>C:\Users\Ziemniak\Documents\runit\runit-view\src\main\resources\images\icon\icon.ico</icon>
+  <jre>
+    <path></path>
+    <bundledJre64Bit>false</bundledJre64Bit>
+    <bundledJreAsFallback>false</bundledJreAsFallback>
+    <minVersion>1.8.0_92</minVersion>
+    <maxVersion></maxVersion>
+    <jdkPreference>preferJre</jdkPreference>
+    <runtimeBits>64/32</runtimeBits>
+    <maxHeapSize>64</maxHeapSize>
+  </jre>
+</launch4jConfig>