Added a line break for the generated files and the explorer opens at …

…the current output path

Implemented issue #10  and #5

src/main/java/de/skymatic/appstore_invoices/gui/OutputController.java

@@ -31,6 +31,7 @@
 import java.nio.file.Path;
 import java.nio.file.StandardCopyOption;
 import java.util.Map;
+import java.util.concurrent.TimeUnit;
 
 public class OutputController {
 
@@ -54,11 +55,11 @@ public class OutputController {
 	private final Stage owner;
 	private final SettingsProvider settingsProvider;
 	private final BooleanProperty isReadyToGenerate;
+	private static final int REVEAL_TIMEOUT_MS = 5000;
 
 	private Settings settings;
 	private MonthlyInvoices monthlyInvoices;
 
-
 	public OutputController(Stage owner, SettingsProvider settingsProvider, MonthlyInvoices monthlyInvoices) {
 		this.owner = owner;
 		this.settingsProvider = settingsProvider;
@@ -188,6 +189,7 @@ public void generateInvoices() {
 		try {
 			Map<String, StringBuilder> htmlInvoices = htmlGenerator.createHTMLInvoices(templatePath.get(), invoices);
 			new HTMLWriter().write(outputPath.get(), htmlInvoices);
+			reveal(outputPath.get());
 		} catch (IOException e) {
 			//TODO: better error handling
 			Alerts.genericError(e, "Generating the invoices from template and save them to hard disk.").showAndWait();
@@ -200,6 +202,27 @@ public void back() {
 		owner.setScene(parseSF.createScene());
 	}
 
+	private boolean reveal(Path pathToReveal) {
+		try {
+			ProcessBuilder revealCommand = new ProcessBuilder("explorer", pathToReveal.toString());
+			Process proc = revealCommand.start();
+			boolean finishedInTime = proc.waitFor(REVEAL_TIMEOUT_MS, TimeUnit.MILLISECONDS);
+			if (finishedInTime) {
+				// The check proc.exitValue() == 0 is always false since Windows explorer return every time an exit value of 1
+				return true;
+			} else {
+				proc.destroyForcibly();
+				return false;
+			}
+		} catch (InterruptedException e) {
+			Thread.currentThread().interrupt();
+			return false;
+		} catch (IOException e) {
+			Alerts.genericError(e, "Failed to open output path").showAndWait();
+			return false;
+		}
+	}
+
 	// Getter & Setter
 
 	public BooleanProperty isReadyToGenerateProperty() {

src/main/java/de/skymatic/appstore_invoices/output/HTMLGenerator.java

@@ -45,6 +45,7 @@ public Map<String, StringBuilder> createHTMLInvoices(Path templatePath, Collecti
 				}
 				for (Invoice i : invoices) {
 					sbs.get(i.getNumberString()).append(line, lastEndPosPlus2, line.length());
+					sbs.get(i.getNumberString()).append("\n");
 				}
 
 			});