1 files changed, 63 insertions, 12 deletions

diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
index 71745e3..019fdc0 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
@@ -71,7 +71,18 @@ public class AnsprechpartnerListeController
       ansprechpartnerVornameTableColumn
             .setCellFactory(TextFieldTableCell.forTableColumn());
       ansprechpartnerVornameTableColumn.setOnEditCommit((data) -> {
-         data.getRowValue().setVorname(data.getNewValue().trim());
+         if (!data.getNewValue().isEmpty())
+         {
+            Ansprechpartner tmpAns = new Ansprechpartner(
+                  data.getRowValue().getOrganisation());
+            tmpAns.setNachname(data.getRowValue().getNachname());
+            tmpAns.setVorname(data.getNewValue().trim());
+
+            if (!main.getAnsprechpartner().contains(tmpAns))
+            {
+               data.getRowValue().setVorname(tmpAns.getVorname());
+            }
+         }
 
          // Hack, aktualisiert TableView, da Observable beim setzen nicht neu
          // abgefragt wird
@@ -84,7 +95,18 @@ public class AnsprechpartnerListeController
       ansprechpartnerNachnameTableColumn
             .setCellFactory(TextFieldTableCell.forTableColumn());
       ansprechpartnerNachnameTableColumn.setOnEditCommit((data) -> {
-         data.getRowValue().setNachname(data.getNewValue().trim());
+         if (!data.getNewValue().isEmpty())
+         {
+            Ansprechpartner tmpAns = new Ansprechpartner(
+                  data.getRowValue().getOrganisation());
+            tmpAns.setVorname(data.getRowValue().getVorname());
+            tmpAns.setNachname(data.getNewValue().trim());
+
+            if (!main.getAnsprechpartner().contains(tmpAns))
+            {
+               data.getRowValue().setNachname(tmpAns.getNachname());
+            }
+         }
 
          // Hack, aktualisiert TableView, da Observable beim setzen nicht neu
          // abgefragt wird
@@ -93,14 +115,26 @@ public class AnsprechpartnerListeController
       });
 
       organisationsNameTableColumn.setCellValueFactory((data) -> {
-         data.getValue().getOrganisation().nameProperty()
-               .addListener((listener) -> {
-            // Hack, aktualisiert TableView, da Organisation kein Observable
+         data.getValue().organisationProperty().addListener((listener) -> {
+            // Hack, aktualisiert TableView, da Organisation kein
+            // Observable
             // ist
             organisationsNameTableColumn.setVisible(false);
             organisationsNameTableColumn.setVisible(true);
          });
-         return new SimpleObjectProperty<>(data.getValue().getOrganisation());
+         if (data.getValue().getOrganisation() != null)
+         {
+            data.getValue().getOrganisation().nameProperty()
+                  .addListener((listener) -> {
+               // Hack, aktualisiert TableView, da Organisation kein
+               // Observable
+               // ist
+               organisationsNameTableColumn.setVisible(false);
+               organisationsNameTableColumn.setVisible(true);
+            });
+            return data.getValue().organisationProperty();
+         }
+         return new SimpleObjectProperty<Organisation>(new Organisation());
       });
       organisationsNameTableColumn.setOnEditCommit((data) -> {
          data.getRowValue().setOrganisation(data.getNewValue());
@@ -151,7 +185,10 @@ public class AnsprechpartnerListeController
       {
          Organisation organisation = new Organisation();
          organisation.setName(result.get().trim());
-         main.getOrganisation().add(organisation);
+         if (!main.getOrganisation().contains(organisation))
+         {
+            main.getOrganisation().add(organisation);
+         }
       }
    }
 
@@ -174,7 +211,12 @@ public class AnsprechpartnerListeController
       Optional<String> result = dialog.showAndWait();
       if (result.isPresent() && !result.get().trim().isEmpty())
       {
-         organisation.setName(result.get().trim());
+         Organisation tmpOrg = new Organisation();
+         tmpOrg.setName(result.get().trim());
+         if (!main.getOrganisation().contains(tmpOrg))
+         {
+            organisation.setName(tmpOrg.getName());
+         }
       }
    }
 
@@ -206,8 +248,14 @@ public class AnsprechpartnerListeController
 
          if (result == null || result.get() == ButtonType.OK)
          {
-            main.getAnsprechpartner()
-                  .removeAll(selectedItem.getAnsprechpartner());
+            Ansprechpartner[] ansArray = selectedItem.getAnsprechpartner()
+                  .toArray(new Ansprechpartner[selectedItem
+                        .getAnsprechpartner().size()]);
+            for (Ansprechpartner ans : ansArray)
+            {
+               ans.setOrganisation(null);
+            }
+
             main.getOrganisation().remove(organisationComboBox.getValue());
          }
       }
@@ -225,7 +273,7 @@ public class AnsprechpartnerListeController
 
       if (selectedItem != null)
       {
-         selectedItem.getOrganisation().removeAnsprechpartner(selectedItem);
+         selectedItem.setOrganisation(null);
          main.getAnsprechpartner().remove(selectedItem);
       }
    }
@@ -256,7 +304,10 @@ public class AnsprechpartnerListeController
             ansprechpartner.setVorname(result.get().getKey().trim());
             ansprechpartner.setNachname(result.get().getValue().trim());
 
-            main.getAnsprechpartner().add(ansprechpartner);
+            if (!main.getAnsprechpartner().contains(ansprechpartner))
+            {
+               main.getAnsprechpartner().add(ansprechpartner);
+            }
          }
       }
    }