Rewrite the observable config model base class

pom.xml

@@ -378,7 +378,7 @@
         <!-- LICENSE INFORMATION
         FontawesomeFX:8.8
         Apache 2 License
-        http://logging.apache.org/log4j/2.x/license.html
+        http://www.apache.org/licenses/LICENSE-2.0.txt
         2015-12-04
         emifor
         -->

src/main/java/com/speedment/internal/ui/config/AbstractRootDocumentProperty.java

@@ -26,15 +26,15 @@
  * @author        Emil Forslund
  * @param <THIS>  this class type
  */
-public abstract class AbstractRootDocumentProperty<THIS extends AbstractRootDocumentProperty<? super THIS>> 
+public abstract class AbstractRootDocumentProperty<THIS extends AbstractRootDocumentProperty<? super THIS>>
     extends AbstractDocumentProperty<THIS> {
 
     public AbstractRootDocumentProperty(Map<String, Object> data) {
         super(data);
     }
 
     @Override
-    public final Optional<? extends Document> getParent() {
+    public final Optional<? extends DocumentProperty> getParent() {
         return Optional.empty();
     }
 

src/main/java/com/speedment/internal/ui/config/ColumnProperty.java

@@ -167,7 +167,4 @@ public String toString(TypeMapper<?, ?> typeMapper) {
             }
         }
     };
-
-    @Override
-    public void prepare() { /* Do nothing since this document has no known children. */ }
 }

src/main/java/com/speedment/internal/ui/config/DbmsProperty.java

@@ -145,12 +145,6 @@ public SchemaProperty addNewSchema() {
         return created;
     }
 
-
-    @Override
-    public void prepare() {
-        schemas().forEach(DocumentProperty::prepare);
-    }
-
     @Override
     public String toString() {
         return toStringHelper(this);

src/main/java/com/speedment/internal/ui/config/DefaultDocumentProperty.java

@@ -50,7 +50,4 @@ public Stream<PropertySheet.Item> getUiVisibleProperties(Speedment speedment) {
     public Class<? extends Document> mainInterface() {
         return Document.class;
     }
-
-    @Override
-    public void prepare() { /* Do nothing since this document has no known children. */ }
 }

src/main/java/com/speedment/internal/ui/config/DocumentProperty.java

@@ -19,6 +19,7 @@
 import com.speedment.config.Document;
 import com.speedment.config.db.trait.HasMainInterface;
 import com.speedment.exception.SpeedmentException;
+import com.speedment.internal.ui.config.trait.HasExpandedProperty;
 import com.speedment.internal.ui.config.trait.HasUiVisibleProperties;
 import java.util.Map;
 import java.util.function.BiFunction;
@@ -62,7 +63,10 @@
  * 
  * @author Emil Forslund
  */
-public interface DocumentProperty extends Document, HasUiVisibleProperties, HasMainInterface, Observable {
+public interface DocumentProperty extends Document, 
+        HasUiVisibleProperties, 
+        HasMainInterface,
+        Observable {
 
     /**
      * Wraps the specified String value in a property so that changes to it can
@@ -145,10 +149,9 @@ public interface DocumentProperty extends Document, HasUiVisibleProperties, HasM
      * 
      * @param <T>  result type
      * @param key  the key to look at
-     * @param type the expected result type
      * @return     an observable list of the documents under that key
      */
-    <T extends DocumentProperty> ObservableList<T> observableListOf(String key/*, Class<T> type*/);
+    <T extends DocumentProperty> ObservableList<T> observableListOf(String key);
 
     /**
      * Returns a list of all children instantiated using 
@@ -181,10 +184,4 @@ public interface DocumentProperty extends Document, HasUiVisibleProperties, HasM
      * any events observing the tree will know the state has changed.
      */
     void invalidate();
-
-    /**
-     * Goes through all {@code ObservableList ObservableLists} in the tree and
-     * make sure every one has been initalized.
-     */
-    void prepare();
 }

src/main/java/com/speedment/internal/ui/config/ForeignKeyColumnProperty.java

@@ -105,7 +105,4 @@ public Column findForeignColumn() throws SpeedmentException {
     public String toString() {
         return toStringHelper(this);
     }
-
-    @Override
-    public void prepare() { /* Do nothing since this document has no known children. */ }
 }

src/main/java/com/speedment/internal/ui/config/ForeignKeyProperty.java

@@ -88,12 +88,6 @@ public boolean isExpandedByDefault() {
         return false;
     }
 
-
-    @Override
-    public void prepare() {
-        foreignKeyColumns().forEach(DocumentProperty::prepare);
-    }
-
     @Override
     public String toString() {
         return toStringHelper(this);

src/main/java/com/speedment/internal/ui/config/IndexColumnProperty.java

@@ -52,7 +52,4 @@ public Stream<PropertySheet.Item> getUiVisibleProperties(Speedment speedment) {
     public String toString() {
         return toStringHelper(this);
     }
-
-    @Override
-    public void prepare() { /* Do nothing since this document has no known children. */ }
 }

src/main/java/com/speedment/internal/ui/config/IndexProperty.java

@@ -101,12 +101,6 @@ public boolean isExpandedByDefault() {
         return false;
     }
 
-
-    @Override
-    public void prepare() {
-        indexColumns().forEach(DocumentProperty::prepare);
-    }
-
     @Override
     public String toString() {
         return toStringHelper(this);

src/main/java/com/speedment/internal/ui/config/PrimaryKeyColumnProperty.java

@@ -48,7 +48,4 @@ public Stream<PropertySheet.Item> getUiVisibleProperties(Speedment speedment) {
     public String toString() {
         return toStringHelper(this);
     }
-
-    @Override
-    public void prepare() { /* Do nothing since this document has no known children. */ }
 }

src/main/java/com/speedment/internal/ui/config/ProjectProperty.java

@@ -175,9 +175,4 @@ public String getName() throws SpeedmentException {
     public String toString() {
         return toStringHelper(this);
     }
-
-    @Override
-    public void prepare() {
-        dbmses().forEach(DocumentProperty::prepare);
-    }
 }

src/main/java/com/speedment/internal/ui/config/SchemaProperty.java

@@ -103,12 +103,6 @@ public TableProperty addNewTable() {
         return created;
     }
 
-
-    @Override
-    public void prepare() {
-        tables().forEach(DocumentProperty::prepare);
-    }
-
     @Override
     public String toString() {
         return toStringHelper(this);

src/main/java/com/speedment/internal/ui/config/TableProperty.java

@@ -151,15 +151,6 @@ public PrimaryKeyColumnProperty addNewPrimaryKeyColumn() {
         return created;
     }
 
-
-    @Override
-    public void prepare() {
-        columns().forEach(DocumentProperty::prepare);
-        indexes().forEach(DocumentProperty::prepare);
-        foreignKeys().forEach(DocumentProperty::prepare);
-        primaryKeyColumns().forEach(DocumentProperty::prepare);
-    }
-
     @Override
     public String toString() {
         return toStringHelper(this);

src/main/java/com/speedment/internal/ui/controller/ProjectTreeController.java

@@ -99,7 +99,6 @@ private void prepareTree(ProjectProperty project) {
         hierarchy.setCellFactory(view -> new DocumentPropertyCell(ui));
         hierarchy.getSelectionModel().setSelectionMode(SINGLE);
 
-        project.prepare();
         populateTree(project);
     }