Don't remove all Ansprechpartner on Organisation delete

4 files changed, 85 insertions, 34 deletions

diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java
index fe144e7..e786f78 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/Main.java
@@ -175,15 +175,13 @@ public class Main extends Application
          {
             Organisation organisation = orga
                   .get(rs.getString("organisation"));
-            if (organisation != null)
-            {
-               Ansprechpartner ansprechpartner = new Ansprechpartner(
-                     organisation);
-               ansprechpartner.setVorname(rs.getString("vorname"));
-               ansprechpartner.setNachname(rs.getString("nachname"));
 
-               this.ansprechpartner.add(ansprechpartner);
-            }
+            Ansprechpartner ansprechpartner = new Ansprechpartner(
+                  organisation);
+            ansprechpartner.setVorname(rs.getString("vorname"));
+            ansprechpartner.setNachname(rs.getString("nachname"));
+
+            this.ansprechpartner.add(ansprechpartner);
          }
 
          organisation.addAll(orga.values());
@@ -223,7 +221,14 @@ public class Main extends Application
          {
             ps.setString(1, ans.getVorname());
             ps.setString(2, ans.getNachname());
-            ps.setString(3, ans.getOrganisation().getName());
+            if (ans.getOrganisation() != null)
+            {
+               ps.setString(3, ans.getOrganisation().getName());
+            }
+            else
+            {
+               ps.setString(3, null);
+            }
 
             ps.execute();
          }
diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java
index 11873e0..66b6ab5 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Ansprechpartner.java
@@ -1,6 +1,11 @@
 package de.fhswf.in.inf.se.projektthemenvergabe.model;
 
+import java.util.ArrayList;
+import java.util.List;
+
 import javafx.beans.Observable;
+import javafx.beans.property.ObjectProperty;
+import javafx.beans.property.SimpleObjectProperty;
 import javafx.beans.property.SimpleStringProperty;
 import javafx.beans.property.StringProperty;
 import javafx.util.Callback;
@@ -11,7 +16,7 @@ public class Ansprechpartner
 
    private StringProperty nachname = new SimpleStringProperty();
 
-   private Organisation organisation;
+   private ObjectProperty<Organisation> organisation = new SimpleObjectProperty<>();
 
    /**
     * Generates an extractor that fires when a property of a list value
@@ -21,10 +26,21 @@ public class Ansprechpartner
     */
    public static Callback<Ansprechpartner, Observable[]> extractor()
    {
-      return ansprechpartner -> new Observable[] {
-            ansprechpartner.vornameProperty(),
-            ansprechpartner.nachnameProperty(),
-            ansprechpartner.getOrganisation().nameProperty() };
+      return ansprechpartner -> {
+         List<Observable> extractorList = new ArrayList<>();
+
+         extractorList.add(ansprechpartner.vornameProperty());
+         extractorList.add(ansprechpartner.nachnameProperty());
+         extractorList.add(ansprechpartner.organisationProperty());
+
+         if (ansprechpartner.getOrganisation() != null)
+         {
+            extractorList
+                  .add(ansprechpartner.getOrganisation().nameProperty());
+         }
+
+         return extractorList.toArray(new Observable[extractorList.size()]);
+      };
    }
 
    /**
@@ -34,23 +50,32 @@ public class Ansprechpartner
     */
    public Ansprechpartner(Organisation organisation)
    {
-      if (organisation == null)
+      this.organisation.set(organisation);
+
+      if (organisation != null)
       {
-         throw new IllegalArgumentException(
-               "Ansprechpartner sollten nicht arbeitslos sein.");
+         this.organisation.get().addAnsprechpartner(this);
       }
-      this.organisation = organisation;
-      this.organisation.addAnsprechpartner(this);
    }
 
    /**
     * Getter for property organisation.
+    *
+    * @return Returns the organisation.
+    */
+   public ObjectProperty<Organisation> organisationProperty()
+   {
+      return organisation;
+   }
+
+   /**
+    * Getter for organisation.
     * 
     * @return Returns the organisation.
     */
    public Organisation getOrganisation()
    {
-      return organisation;
+      return organisation.get();
    }
 
    /**
@@ -61,14 +86,17 @@ public class Ansprechpartner
     */
    public void setOrganisation(Organisation organisation)
    {
-      if (organisation == null)
+      if (this.organisation.get() != null)
+      {
+         this.organisation.get().removeAnsprechpartner(this);
+      }
+
+      this.organisation.set(organisation);
+
+      if (organisation != null)
       {
-         throw new IllegalArgumentException(
-               "Ansprechpartner sollten nicht arbeitslos sein.");
+         organisation.addAnsprechpartner(this);
       }
-      this.organisation.removeAnsprechpartner(this);
-      this.organisation = organisation;
-      this.organisation.addAnsprechpartner(this);
    }
 
    /**
diff --git a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java
index 8f04979..dca4511 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/model/Organisation.java
@@ -11,7 +11,7 @@ import javafx.util.Callback;
 
 public class Organisation
 {
-   private StringProperty name = new SimpleStringProperty();
+   private StringProperty name = new SimpleStringProperty("");
 
    private List<Ansprechpartner> ansprechpartner = new ArrayList<>();
 
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 2ae8405..27f85c9 100644
--- a/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
+++ b/src/de/fhswf/in/inf/se/projektthemenvergabe/view/AnsprechpartnerListeController.java
@@ -92,14 +92,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());
@@ -203,8 +215,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());
          }
       }
@@ -222,7 +240,7 @@ public class AnsprechpartnerListeController
 
       if (selectedItem != null)
       {
-         selectedItem.getOrganisation().removeAnsprechpartner(selectedItem);
+         selectedItem.setOrganisation(null);
          main.getAnsprechpartner().remove(selectedItem);
       }
    }