From 7db103abe2fed5c9fb3f934495aac7ace20829ea Mon Sep 17 00:00:00 2001 From: Dominic Rogetzer Date: Sun, 17 Jun 2018 16:06:15 +0200 Subject: Separate validation and error messages in OperationDetailsCtrl [#25963] --- .../controller/OperationDetailsController.java | 45 +++++++++++----------- 1 file changed, 22 insertions(+), 23 deletions(-) (limited to 'src/main/java/at/ac/tuwien/sepm/assignment') 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 ab348e1..3e37a04 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 @@ -118,21 +118,20 @@ public class OperationDetailsController { LOG.debug("Button \"Abschließen\" clicked."); try { operationService.complete(operation.id(), Status.COMPLETED); - } catch (InvalidOperationException | ServiceException e) { + } catch (InvalidOperationException e) { + LOG.debug("Validation error in closeOperationClicked(). ", e); + Helper.showAlertWithOkButtonAndWait( + AlertType.ERROR, "Validierungsfehler", e.getMessage()); + return; + } catch (ServiceException e) { LOG.error("Exception in closeOperationClicked(). ", e); - Alert alert = new Alert(AlertType.ERROR); - alert.setTitle("Fehler"); - alert.setHeaderText("Fehler!"); - alert.setContentText(e.getMessage()); - alert.showAndWait(); + Helper.showAlertWithOkButtonAndWait(AlertType.ERROR, "Fehler", e.getMessage()); return; } - Alert alert = new Alert(AlertType.CONFIRMATION); - alert.setTitle("Erfolg"); - alert.setHeaderText("Erfolgreich aktualisiert"); - alert.setContentText("Der Einsatz wurde erfolgreich aktualisiert."); - alert.showAndWait(); - closeWindow(); + Helper.showAlertWithOkButtonAndWait( + AlertType.CONFIRMATION, + "Erfolgreich aktualisiert", + "Der Einsatz wurde erfolgreich aktualisiert."); createOperationController.updateList(); } @@ -140,20 +139,20 @@ public class OperationDetailsController { LOG.debug("Button \"Stornieren\" clicked."); try { operationService.complete(operation.id(), Status.CANCELLED); - } catch (InvalidOperationException | ServiceException e) { + } catch (InvalidOperationException e) { + LOG.debug("Validation error in cancelOperationClicked(). ", e); + Helper.showAlertWithOkButtonAndWait( + AlertType.ERROR, "Validierungsfehler", e.getMessage()); + return; + } catch (ServiceException e) { LOG.error("Exception in cancelOperationClicked(). ", e); - Alert alert = new Alert(AlertType.ERROR); - alert.setTitle("Fehler"); - alert.setHeaderText("Fehler!"); - alert.setContentText(e.getMessage()); - alert.showAndWait(); + Helper.showAlertWithOkButtonAndWait(AlertType.ERROR, "Fehler", e.getMessage()); return; } - Alert alert = new Alert(AlertType.CONFIRMATION); - alert.setTitle("Erfolg"); - alert.setHeaderText("Erfolgreich aktualisiert"); - alert.setContentText("Der Einsatz wurde erfolgreich aktualisiert."); - alert.showAndWait(); + Helper.showAlertWithOkButtonAndWait( + AlertType.CONFIRMATION, + "Erfolgreich aktualisiert", + "Der Einsatz wurde erfolgreich aktualisiert."); closeWindow(); createOperationController.updateList(); } -- cgit v1.2.3-70-g09d2