Use specific helper methods for alerts in OperationDetailsCtrl [#25963]

1 files changed, 12 insertions, 21 deletions

diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/OperationDetailsController.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/OperationDetailsController.java
index 3e37a04..7e66e22 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/OperationDetailsController.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/controller/OperationDetailsController.java
@@ -1,5 +1,10 @@
 package at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller;
 
+import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showAlertWithOkButtonAndWait;
+import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showServiceExceptionAlertAndWait;
+import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showSuccessAlertAndWait;
+import static at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.controller.Helper.showValidationErrorAlertAndWait;
+
 import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Operation;
 import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Operation.Status;
 import at.ac.tuwien.sepm.assignment.groupphase.einsatzverwaltung.dto.Vehicle;
@@ -13,7 +18,6 @@ import java.util.EnumSet;
 import java.util.stream.Collectors;
 import javafx.collections.FXCollections;
 import javafx.fxml.FXML;
-import javafx.scene.control.Alert;
 import javafx.scene.control.Alert.AlertType;
 import javafx.scene.control.Button;
 import javafx.scene.control.Label;
@@ -105,11 +109,7 @@ public class OperationDetailsController {
                             operationService.list(EnumSet.of(Status.ACTIVE))));
         } catch (ServiceException e) {
             LOG.error("ServiceException in fillActiveList(). ", e);
-            Alert alert = new Alert(AlertType.ERROR);
-            alert.setTitle("Fehler");
-            alert.setHeaderText("Fehler!");
-            alert.setContentText(e.getMessage());
-            alert.showAndWait();
+            showServiceExceptionAlertAndWait(e.getMessage());
         }
     }
 
@@ -120,18 +120,14 @@ public class OperationDetailsController {
             operationService.complete(operation.id(), Status.COMPLETED);
         } catch (InvalidOperationException e) {
             LOG.debug("Validation error in closeOperationClicked(). ", e);
-            Helper.showAlertWithOkButtonAndWait(
-                    AlertType.ERROR, "Validierungsfehler", e.getMessage());
+            showAlertWithOkButtonAndWait(AlertType.ERROR, "Validierungsfehler", e.getMessage());
             return;
         } catch (ServiceException e) {
             LOG.error("Exception in closeOperationClicked(). ", e);
-            Helper.showAlertWithOkButtonAndWait(AlertType.ERROR, "Fehler", e.getMessage());
+            showServiceExceptionAlertAndWait(e.getMessage());
             return;
         }
-        Helper.showAlertWithOkButtonAndWait(
-                AlertType.CONFIRMATION,
-                "Erfolgreich aktualisiert",
-                "Der Einsatz wurde erfolgreich aktualisiert.");
+        showSuccessAlertAndWait("Der Einsatz wurde erfolgreich aktualisiert");
         createOperationController.updateList();
     }
 
@@ -141,19 +137,14 @@ public class OperationDetailsController {
             operationService.complete(operation.id(), Status.CANCELLED);
         } catch (InvalidOperationException e) {
             LOG.debug("Validation error in cancelOperationClicked(). ", e);
-            Helper.showAlertWithOkButtonAndWait(
-                    AlertType.ERROR, "Validierungsfehler", e.getMessage());
+            showValidationErrorAlertAndWait(e.getMessage());
             return;
         } catch (ServiceException e) {
             LOG.error("Exception in cancelOperationClicked(). ", e);
-            Helper.showAlertWithOkButtonAndWait(AlertType.ERROR, "Fehler", e.getMessage());
+            showServiceExceptionAlertAndWait(e.getMessage());
             return;
         }
-        Helper.showAlertWithOkButtonAndWait(
-                AlertType.CONFIRMATION,
-                "Erfolgreich aktualisiert",
-                "Der Einsatz wurde erfolgreich aktualisiert.");
-        closeWindow();
+        showSuccessAlertAndWait("Der Einsatz wurde erfolgreich aktualisiert");
         createOperationController.updateList();
     }