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

@@ -8,17 +8,12 @@ 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;
@@ -66,7 +61,7 @@ public class AllMedicationController {
             cellData -> {
                 Medication medication = cellData.getValue();
                 List<Ingredient> ingredients = medication.getIngredients();
-                    if (ingredients.isEmpty()) {
+                if(ingredients.isEmpty()){
                     return new SimpleStringProperty("");
                 }
 
@@ -96,18 +91,6 @@ 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}.
      */
@@ -120,13 +103,8 @@ public class AllMedicationController {
         }
     }
 
-    /**
+    @FXML
-     * Internal method to create a {@link MedicationModalController MedicationModal}.
+    public void handleNewMedication() {
-     *
-     * @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")
@@ -136,10 +114,10 @@ public class AllMedicationController {
             Stage stage = new Stage();
 
             MedicationModalController controller = loader.getController();
-            controller.initialize(stage, this, medication);
+            controller.initialize(stage, this, null);
 
             stage.setScene(scene);
-            stage.setTitle(title);
+            stage.setTitle("NHPlus - Neues Medikament");
             stage.setResizable(true);
             stage.setAlwaysOnTop(true);
             stage.showAndWait();
@@ -147,21 +125,4 @@ 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,7 +43,6 @@ 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.
@@ -55,12 +54,11 @@ public class MedicationModalController {
             Medication medication
     ) {
         this.stage = stage;
-        this.controller = controller;
+        this.controller=controller;
 
-        if (medication != null) {
+        if( medication != null){
             this.medication =  medication;
-        } else {
+        }else {
-            isNewMedication = true;
             this.medication = new Medication(
                     "",
                     "",
@@ -94,7 +92,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());
@@ -119,12 +117,7 @@ public class MedicationModalController {
                 .toList()
         );
 
-        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" onMouseClicked="#handleMouseClick">
+        <TableView fx:id="tableView">
             <columns>
                 <TableColumn
                         fx:id="columnId"