From 45e0f62e6dd7ed8c92352f450d66bc6a0628bcd2 Mon Sep 17 00:00:00 2001
From: Robert Goldmann <deadlocker@gmx.de>
Date: Fri, 23 Apr 2021 22:09:52 +0200
Subject: [PATCH] fixed alerts

---
 .../smarttime/charts/ChartGUIController.java  |  2 +-
 .../smarttime/controller/Controller.java      | 26 +++++++++----------
 .../smarttime/controller/EditController.java  |  2 +-
 .../controller/InsertTimeController.java      | 12 ++++-----
 .../controller/ProjectController.java         |  2 +-
 .../controller/ReportController.java          |  4 +--
 .../deadlocker8/smarttime/core/Exporter.java  |  4 +--
 .../deadlocker8/smarttime/core/Importer.java  | 10 +++----
 .../de/deadlocker8/smarttime/main/Main.java   |  2 +-
 9 files changed, 31 insertions(+), 33 deletions(-)

diff --git a/src/main/java/de/deadlocker8/smarttime/charts/ChartGUIController.java b/src/main/java/de/deadlocker8/smarttime/charts/ChartGUIController.java
index 6361c11..f3facdc 100644
--- a/src/main/java/de/deadlocker8/smarttime/charts/ChartGUIController.java
+++ b/src/main/java/de/deadlocker8/smarttime/charts/ChartGUIController.java
@@ -412,7 +412,7 @@ public class ChartGUIController
 
 	private void showErrorMessage()
 	{
-		Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Erstellen des Diagramms.", stage);
+		Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Erstellen des Diagramms.", stage).show();
 	}
 
 	private void showPieChart(PieChart chart)
diff --git a/src/main/java/de/deadlocker8/smarttime/controller/Controller.java b/src/main/java/de/deadlocker8/smarttime/controller/Controller.java
index 75503f0..df134b4 100644
--- a/src/main/java/de/deadlocker8/smarttime/controller/Controller.java
+++ b/src/main/java/de/deadlocker8/smarttime/controller/Controller.java
@@ -124,7 +124,7 @@ public class Controller
 			}
 			else
 			{
-				Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Kein Projekt ausgewählt.", stage);
+				Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Kein Projekt ausgewählt.", stage).show();
 				startButton.setSelected(false);
 			}
 		});
@@ -139,7 +139,7 @@ public class Controller
 		// Prüft, ob die Stoppuhr noch läuft
 		if(isTimerRunning())
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Stoppuhr läuft noch!", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Stoppuhr läuft noch!", stage).show();
 		}
 		else
 		{
@@ -195,7 +195,7 @@ public class Controller
 		}
 		else
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Stoppuhr läuft noch!", "Projekt und Task können nur geändert werden,\nwenn die Stoppuhr nicht läuft.", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Stoppuhr läuft noch!", "Projekt und Task können nur geändert werden,\nwenn die Stoppuhr nicht läuft.", stage).show();
 		}
 	}
 
@@ -266,7 +266,7 @@ public class Controller
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Laden der Daten ist ein Fehler aufgetreten.", stage);
+			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Laden der Daten ist ein Fehler aufgetreten.", stage).show();
 		}
 	}
 
@@ -298,7 +298,7 @@ public class Controller
 			catch(Exception ex)
 			{
 				Logger.error(ex);
-				Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Erstellen der Datenbank.", stage);
+				Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Erstellen der Datenbank.", stage).show();
 			}
 		}
 	}
@@ -472,7 +472,7 @@ public class Controller
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Speichern des Eintrags.", stage);
+			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Speichern des Eintrags.", stage).show();
 		}
 
 		loadAll();
@@ -501,7 +501,7 @@ public class Controller
 			newStage.setTitle("Zeit nachträglich einfügen");
 
 			InsertTimeController controller = fxmlLoader.getController();
-			controller.init(newStage, this, settings, icon);
+			controller.init(newStage, this, settings);
 			newStage.getIcons().add(icon);
 			newStage.initOwner(stage);
 
@@ -599,10 +599,10 @@ public class Controller
 			catch(IOException e)
 			{
 				Logger.error(e);
-				Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Exportieren der Daten ist ein Fehler aufgetreten.", stage);
+				Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Exportieren der Daten ist ein Fehler aufgetreten.", stage).show();
 			}
 
-			Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich exportiert", "Export erfolgreich abgeschlossen.", stage);
+			Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich exportiert", "Export erfolgreich abgeschlossen.", stage).show();
 		}
 	}
 
@@ -692,7 +692,7 @@ public class Controller
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Aktualisieren des Eintrags ist ein Fehler aufgetreten.", stage);
+			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Aktualisieren des Eintrags ist ein Fehler aufgetreten.", stage).show();
 		}
 	}
 
@@ -706,7 +706,7 @@ public class Controller
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Löschen des Eintrags ist ein Fehler aufgetreten.", stage);
+			Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Löschen des Eintrags ist ein Fehler aufgetreten.", stage).show();
 		}
 	}
 
@@ -732,7 +732,7 @@ public class Controller
 					catch(IOException e)
 					{
 						Logger.error(e);
-						Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Verschieben der Datenbank ist ein Fehler aufgetreten.", stage);
+						Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Verschieben der Datenbank ist ein Fehler aufgetreten.", stage).show();
 					}
 				}
 				else
@@ -767,7 +767,7 @@ public class Controller
 			catch(Exception e)
 			{
 				Logger.error(e);
-				Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Löschen der Datenbank ist ein Fehler aufgetreten.", stage);
+				Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Löschen der Datenbank ist ein Fehler aufgetreten.", stage).show();
 			}
 		}
 	}
diff --git a/src/main/java/de/deadlocker8/smarttime/controller/EditController.java b/src/main/java/de/deadlocker8/smarttime/controller/EditController.java
index ab850c5..27c6d6f 100644
--- a/src/main/java/de/deadlocker8/smarttime/controller/EditController.java
+++ b/src/main/java/de/deadlocker8/smarttime/controller/EditController.java
@@ -84,7 +84,7 @@ public class EditController
 
 		if(project == null || project.equals("") || task == null || task.equals(""))
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Felder dürfen nicht leer sein.", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Felder dürfen nicht leer sein.", stage).show();
 		}
 		else
 		{
diff --git a/src/main/java/de/deadlocker8/smarttime/controller/InsertTimeController.java b/src/main/java/de/deadlocker8/smarttime/controller/InsertTimeController.java
index 9e4bab2..c305746 100644
--- a/src/main/java/de/deadlocker8/smarttime/controller/InsertTimeController.java
+++ b/src/main/java/de/deadlocker8/smarttime/controller/InsertTimeController.java
@@ -56,14 +56,12 @@ public class InsertTimeController
 	private Stage stage;
 	private Controller controller;
 	private String savePath;
-	private Image icon;
 
-	public void init(Stage stage, Controller controller, Settings settings, Image icon)
+	public void init(Stage stage, Controller controller, Settings settings)
 	{
 		this.savePath = settings.getSavePath() + "/" + Utils.DATABASE_NAME;
 		this.stage = stage;
 		this.controller = controller;
-		this.icon = icon;
 
 		final FontIcon fontIconClock = new FontIcon(FontAwesomeType.CLOCK_ALT);
 		fontIconClock.setSize(14);
@@ -203,21 +201,21 @@ public class InsertTimeController
 				catch(Exception e)
 				{
 					Logger.error(e);
-					Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Speichern des Eintrags.", stage);
+					Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Fehler beim Speichern des Eintrags.", stage).show();
 				}
 
-				Alerts.getInstance().createAlert(AlertType.INFORMATION, "Gespeichert", "Der Eintrag wurde erfolgreich gespeichert.", stage);
+				Alerts.getInstance().createAlert(AlertType.INFORMATION, "Gespeichert", "Der Eintrag wurde erfolgreich gespeichert.", stage).show();
 				stage.close();
 				controller.loadAll();
 			}
 			else
 			{
-				Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Endzeit muss vor Startzeit liegen.", stage);
+				Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Startzeit muss vor Endzeit liegen.", stage).show();
 			}
 		}
 		else
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Felder für Projekt und Task dürfen nicht leer sein.", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Felder für Projekt und Task dürfen nicht leer sein.", stage).show();
 		}
 	}
 
diff --git a/src/main/java/de/deadlocker8/smarttime/controller/ProjectController.java b/src/main/java/de/deadlocker8/smarttime/controller/ProjectController.java
index 857bb96..026b176 100644
--- a/src/main/java/de/deadlocker8/smarttime/controller/ProjectController.java
+++ b/src/main/java/de/deadlocker8/smarttime/controller/ProjectController.java
@@ -91,7 +91,7 @@ public class ProjectController
 
 		if(project == null || project.equals("") || task == null || task.equals(""))
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Felder dürfen nicht leer sein.", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Felder dürfen nicht leer sein.", stage).show();
 		}
 		else
 		{
diff --git a/src/main/java/de/deadlocker8/smarttime/controller/ReportController.java b/src/main/java/de/deadlocker8/smarttime/controller/ReportController.java
index 16cb654..79f0a2b 100644
--- a/src/main/java/de/deadlocker8/smarttime/controller/ReportController.java
+++ b/src/main/java/de/deadlocker8/smarttime/controller/ReportController.java
@@ -127,13 +127,13 @@ public class ReportController
 		String project = comboBoxProject.getValue();
 		if(project == null || project.equals(""))
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Bitte wähle ein Projekt aus", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Bitte wähle ein Projekt aus", stage).show();
 			return;
 		}
 
 		if(!project.equals(ALL_PROJECTS) && getNumberOfActivatedTasks() == 0)
 		{
-			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Bitte wähle mindestens einen Task aus", stage);
+			Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Bitte wähle mindestens einen Task aus", stage).show();
 			return;
 		}
 
diff --git a/src/main/java/de/deadlocker8/smarttime/core/Exporter.java b/src/main/java/de/deadlocker8/smarttime/core/Exporter.java
index 678a914..860e5d6 100644
--- a/src/main/java/de/deadlocker8/smarttime/core/Exporter.java
+++ b/src/main/java/de/deadlocker8/smarttime/core/Exporter.java
@@ -55,12 +55,12 @@ public class Exporter
 			out.write(allItems.toString());
 			out.close();
 
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich exportiert", "Der Exportvorgang wurde erfolgreich abgeschlossen.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich exportiert", "Der Exportvorgang wurde erfolgreich abgeschlossen.", stage).show());
 		}
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Exportieren der Daten ist ein Fehler aufgetreten.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Exportieren der Daten ist ein Fehler aufgetreten.", stage).show());
 		}
 	}
 }
\ No newline at end of file
diff --git a/src/main/java/de/deadlocker8/smarttime/core/Importer.java b/src/main/java/de/deadlocker8/smarttime/core/Importer.java
index a6a1c38..3ea2a55 100644
--- a/src/main/java/de/deadlocker8/smarttime/core/Importer.java
+++ b/src/main/java/de/deadlocker8/smarttime/core/Importer.java
@@ -50,12 +50,12 @@ public class Importer
 				sql.insert(o);
 			}
 
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich importiert", "Der Importvorgang wurde erfolgreich abgeschlossen.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich importiert", "Der Importvorgang wurde erfolgreich abgeschlossen.", stage).show());
 		}
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Importieren der Daten ist ein Fehler aufgetreten.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Importieren der Daten ist ein Fehler aufgetreten.", stage).show());
 		}
 	}
 
@@ -71,12 +71,12 @@ public class Importer
 				currentDB.insert(item);
 			}
 
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich importiert", "Der Importvorgang wurde erfolgreich abgeschlossen.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.INFORMATION, "Erfolgreich importiert", "Der Importvorgang wurde erfolgreich abgeschlossen.", stage).show());
 		}
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Importieren der Daten ist ein Fehler aufgetreten.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Importieren der Daten ist ein Fehler aufgetreten.", stage).show());
 		}
 	}
 
@@ -117,7 +117,7 @@ public class Importer
 		catch(Exception e)
 		{
 			Logger.error(e);
-			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Importieren der Daten ist ein Fehler aufgetreten.", stage));
+			Platform.runLater(() -> Alerts.getInstance().createAlert(AlertType.ERROR, "Fehler", "Beim Importieren der Daten ist ein Fehler aufgetreten.", stage).show());
 		}
 	}
 
diff --git a/src/main/java/de/deadlocker8/smarttime/main/Main.java b/src/main/java/de/deadlocker8/smarttime/main/Main.java
index b76ea1c..ca20664 100644
--- a/src/main/java/de/deadlocker8/smarttime/main/Main.java
+++ b/src/main/java/de/deadlocker8/smarttime/main/Main.java
@@ -49,7 +49,7 @@ public class Main extends Application
 			stage.setOnCloseRequest(we -> {
 				if(controller.isTimerRunning())
 				{
-					Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Stoppuhr läuft noch!", stage);
+					Alerts.getInstance().createAlert(AlertType.WARNING, "Warnung", "Die Stoppuhr läuft noch!", stage).show();
 
 					// "schluckt" die Aufforderung das Fenster zu schließen
 					// (Fenster wird dadurch nicht geschlossen)
-- 
GitLab