Revert vertical prototype changes

1 files changed, 12 insertions, 63 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 ee036e2..499c04e 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
@@ -71,18 +71,7 @@ public class AnsprechpartnerListeController
       ansprechpartnerVornameTableColumn
             .setCellFactory(TextFieldTableCell.forTableColumn());
       ansprechpartnerVornameTableColumn.setOnEditCommit((data) -> {
-         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());
-            }
-         }
+         data.getRowValue().setVorname(data.getNewValue().trim());
 
          // Hack, aktualisiert TableView, da Observable beim setzen nicht neu
          // abgefragt wird
@@ -95,18 +84,7 @@ public class AnsprechpartnerListeController
       ansprechpartnerNachnameTableColumn
             .setCellFactory(TextFieldTableCell.forTableColumn());
       ansprechpartnerNachnameTableColumn.setOnEditCommit((data) -> {
-         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());
-            }
-         }
+         data.getRowValue().setNachname(data.getNewValue().trim());
 
          // Hack, aktualisiert TableView, da Observable beim setzen nicht neu
          // abgefragt wird
@@ -115,26 +93,14 @@ public class AnsprechpartnerListeController
       });
 
       organisationsNameTableColumn.setCellValueFactory((data) -> {
-         data.getValue().organisationProperty().addListener((listener) -> {
-            // Hack, aktualisiert TableView, da Organisation kein
-            // Observable
+         data.getValue().getOrganisation().nameProperty()
+               .addListener((listener) -> {
+            // Hack, aktualisiert TableView, da Organisation kein Observable
             // ist
             organisationsNameTableColumn.setVisible(false);
             organisationsNameTableColumn.setVisible(true);
          });
-         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());
+         return new SimpleObjectProperty<>(data.getValue().getOrganisation());
       });
       organisationsNameTableColumn.setOnEditCommit((data) -> {
          data.getRowValue().setOrganisation(data.getNewValue());
@@ -185,10 +151,7 @@ public class AnsprechpartnerListeController
       {
          Organisation organisation = new Organisation();
          organisation.setName(result.get().trim());
-         if (!main.getOrganisation().contains(organisation))
-         {
-            main.getOrganisation().add(organisation);
-         }
+         main.getOrganisation().add(organisation);
       }
    }
 
@@ -211,12 +174,7 @@ public class AnsprechpartnerListeController
       Optional<String> result = dialog.showAndWait();
       if (result.isPresent() && !result.get().trim().isEmpty())
       {
-         Organisation tmpOrg = new Organisation();
-         tmpOrg.setName(result.get().trim());
-         if (!main.getOrganisation().contains(tmpOrg))
-         {
-            organisation.setName(tmpOrg.getName());
-         }
+         organisation.setName(result.get().trim());
       }
    }
 
@@ -246,14 +204,8 @@ public class AnsprechpartnerListeController
 
          if (result == null || result.get() == ButtonType.OK)
          {
-            Ansprechpartner[] ansArray = selectedItem.getAnsprechpartner()
-                  .toArray(new Ansprechpartner[selectedItem
-                        .getAnsprechpartner().size()]);
-            for (Ansprechpartner ans : ansArray)
-            {
-               ans.setOrganisation(null);
-            }
-
+            main.getAnsprechpartner()
+                  .removeAll(selectedItem.getAnsprechpartner());
             main.getOrganisation().remove(organisationComboBox.getValue());
          }
       }
@@ -271,7 +223,7 @@ public class AnsprechpartnerListeController
 
       if (selectedItem != null)
       {
-         selectedItem.setOrganisation(null);
+         selectedItem.getOrganisation().removeAnsprechpartner(selectedItem);
          main.getAnsprechpartner().remove(selectedItem);
       }
    }
@@ -302,10 +254,7 @@ public class AnsprechpartnerListeController
             ansprechpartner.setVorname(result.get().getKey().trim());
             ansprechpartner.setNachname(result.get().getValue().trim());
 
-            if (!main.getAnsprechpartner().contains(ansprechpartner))
-            {
-               main.getAnsprechpartner().add(ansprechpartner);
-            }
+            main.getAnsprechpartner().add(ansprechpartner);
          }
       }
    }