Merge pull request '#26: Medikamente berabeiten' (#43) from story/medikamente-modul-medikamente-bearbeiten into main

Reviewed-on: #43
Reviewed-by: SZUT-Ole <ole.kueck@hmmh.de>

src/main/java/de/hitec/nhplus/medication/AllMedicationController.java

@@ -8,12 +8,17 @@ import java.util.stream.Collectors;
 import de.hitec.nhplus.Main;
 import de.hitec.nhplus.datastorage.DaoFactory;
 import de.hitec.nhplus.medication.database.MedicationDao;
+import de.hitec.nhplus.nurse.Nurse;
+import de.hitec.nhplus.patient.Patient;
+import de.hitec.nhplus.treatment.Treatment;
+import de.hitec.nhplus.treatment.TreatmentModalController;
 import javafx.beans.property.SimpleStringProperty;
 import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
 import javafx.fxml.FXML;
 import javafx.fxml.FXMLLoader;
 import javafx.scene.Scene;
+import javafx.scene.control.SelectionModel;
 import javafx.scene.control.TableColumn;
 import javafx.scene.control.TableView;
 import javafx.scene.control.cell.PropertyValueFactory;
@@ -58,20 +63,20 @@ public class AllMedicationController {
         this.columnName.setCellValueFactory(new PropertyValueFactory<>("name"));
         this.columnManufacturer.setCellValueFactory(new PropertyValueFactory<>("manufacturer"));
         this.columnIngredient.setCellValueFactory(
-            cellData -> {
+                cellData -> {
-                Medication medication = cellData.getValue();
+                    Medication medication = cellData.getValue();
-                List<Ingredient> ingredients = medication.getIngredients();
+                    List<Ingredient> ingredients = medication.getIngredients();
-                if(ingredients.isEmpty()){
+                    if (ingredients.isEmpty()) {
-                    return new SimpleStringProperty("");
+                        return new SimpleStringProperty("");
-                }
+                    }
 
-                return new SimpleStringProperty(
+                    return new SimpleStringProperty(
-                        ingredients
+                            ingredients
-                        .stream()
+                                    .stream()
-                        .map(ingredient -> ingredient.getName())
+                                    .map(ingredient -> ingredient.getName())
-                        .collect(Collectors.joining("\n"))
+                                    .collect(Collectors.joining("\n"))
-                );
+                    );
-            });
+                });
         this.columnPossibleSideEffects.setCellValueFactory(new PropertyValueFactory<>("possibleSideEffects"));
         this.columnAdministrationMethod.setCellValueFactory(new PropertyValueFactory<>("administrationMethod"));
         this.columnCurrentStock.setCellValueFactory(new PropertyValueFactory<>("currentStock"));
@@ -91,6 +96,18 @@ public class AllMedicationController {
         }
     }
 
+    /**
+     * Method to save the changes to a {@link Medication}.
+     */
+    public void updateMedication(Medication medication) {
+        dao = DaoFactory.getInstance().createMedicationDAO();
+        try {
+            dao.update(medication);
+        } catch (SQLException exception) {
+            exception.printStackTrace();
+        }
+    }
+
     /**
      * Method to create a new {@link Medication}.
      */
@@ -103,21 +120,26 @@ public class AllMedicationController {
         }
     }
 
-    @FXML
+    /**
-    public void handleNewMedication() {
+     * Internal method to create a {@link MedicationModalController MedicationModal}.
+     *
+     * @param medication The {@link Medication} which should be edited. Set null to create a new one.
+     * @param title      The Title of the created modal.
+     */
+    public void medicationWindow(Medication medication, String title) {
         try {
             FXMLLoader loader = new FXMLLoader(
-                Main.class.getResource("/de/hitec/nhplus/medication/MedicationModal.fxml")
+                    Main.class.getResource("/de/hitec/nhplus/medication/MedicationModal.fxml")
             );
             BorderPane pane = loader.load();
             Scene scene = new Scene(pane);
             Stage stage = new Stage();
 
             MedicationModalController controller = loader.getController();
-            controller.initialize(stage, this, null);
+            controller.initialize(stage, this, medication);
 
             stage.setScene(scene);
-            stage.setTitle("NHPlus - Neues Medikament");
+            stage.setTitle(title);
             stage.setResizable(true);
             stage.setAlwaysOnTop(true);
             stage.showAndWait();
@@ -125,4 +147,21 @@ public class AllMedicationController {
             exception.printStackTrace();
         }
     }
+
+    @FXML
+    public void handleNewMedication() {
+        medicationWindow(null, "NHPlus - Neues Medikament");
+    }
+
+    @FXML
+    public void handleMouseClick() {
+        tableView.setOnMouseClicked(event -> {
+            SelectionModel<Medication> selectionModel = tableView.getSelectionModel();
+            if (event.getClickCount() == 2 && (selectionModel.getSelectedItem() != null)) {
+                int index = selectionModel.getSelectedIndex();
+                Medication medication = this.medications.get(index);
+                medicationWindow(medication, "NHPlus - Medikament");
+            }
+        });
+    }
 }

src/main/java/de/hitec/nhplus/medication/MedicationModalController.java

@@ -43,6 +43,7 @@ public class MedicationModalController {
     private Medication medication;
     private final ObservableList<Ingredient> ingredients = FXCollections.observableArrayList();
     private AllMedicationController controller;
+    private boolean isNewMedication = false;
 
     /**
      * Initialization method that is called after the binding of all the fields.
@@ -54,11 +55,12 @@ public class MedicationModalController {
             Medication medication
     ) {
         this.stage = stage;
-        this.controller=controller;
+        this.controller = controller;
 
-        if( medication != null){
+        if (medication != null) {
-            this.medication =  medication;
+            this.medication = medication;
-        }else {
+        } else {
+            isNewMedication = true;
             this.medication = new Medication(
                     "",
                     "",
@@ -76,9 +78,9 @@ public class MedicationModalController {
 
         ChangeListener<String> inputMedicationValidationListener = (observableValue, oldText, newText) -> {
             boolean isValid = isValidMedicationName(textFieldName.getText())
-                    && isValidMedicationManufacturer(textFieldManufacturer.getText())
+                              && isValidMedicationManufacturer(textFieldManufacturer.getText())
-                    && isValidMedicationAdministrationMethod(textFieldAdministrationMethod.getText())
+                              && isValidMedicationAdministrationMethod(textFieldAdministrationMethod.getText())
-                    && isValidStock(textFieldCurrentStock.getText());
+                              && isValidStock(textFieldCurrentStock.getText());
 
             this.buttonSave.setDisable(!isValid);
         };
@@ -92,7 +94,7 @@ public class MedicationModalController {
     /**
      * Internal method to show the data in the view.
      */
-    private void showData(){
+    private void showData() {
         ingredients.setAll(medication.getIngredients());
         textFieldName.setText(medication.getName());
         textFieldManufacturer.setText(medication.getManufacturer());
@@ -117,7 +119,12 @@ public class MedicationModalController {
                 .toList()
         );
 
-        controller.createMedication(medication);
+        if (isNewMedication) {
+            controller.createMedication(medication);
+        } else {
+            controller.updateMedication(medication);
+        }
+
         controller.readAllAndShowInTableView();
         stage.close();
     }

src/main/resources/de/hitec/nhplus/medication/AllMedicationView.fxml

@@ -12,7 +12,7 @@
         <Insets top="8" left="8" right="8" bottom="8"/>
     </padding>
     <center>
-        <TableView fx:id="tableView">
+        <TableView fx:id="tableView" onMouseClicked="#handleMouseClick">
             <columns>
                 <TableColumn
                         fx:id="columnId"