Merge branch 'dev' into 'master'

Version 3.17.2

See merge request !32

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/OpeningNode.java → CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/node/OpeningNode.java

-package de.hft.stuttgart.citydoctor2.gui.tree;
+package de.hft.stuttgart.citydoctor2.gui.tree.node;
 
 import de.hft.stuttgart.citydoctor2.datastructure.Opening;
 import de.hft.stuttgart.citydoctor2.gui.CheckStatus;
 import de.hft.stuttgart.citydoctor2.gui.Renderer;
+import de.hft.stuttgart.citydoctor2.gui.tree.Renderable;
 
 public class OpeningNode extends Renderable {
 

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/PolygonNode.java → CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/node/PolygonNode.java

-package de.hft.stuttgart.citydoctor2.gui.tree;
+package de.hft.stuttgart.citydoctor2.gui.tree.node;
 
 import de.hft.stuttgart.citydoctor2.datastructure.Polygon;
 import de.hft.stuttgart.citydoctor2.gui.CheckStatus;
 import de.hft.stuttgart.citydoctor2.gui.Renderer;
+import de.hft.stuttgart.citydoctor2.gui.tree.Renderable;
 
 public class PolygonNode extends Renderable {
 

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/TextNode.java → CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/node/TextNode.java

-package de.hft.stuttgart.citydoctor2.gui.tree;
+package de.hft.stuttgart.citydoctor2.gui.tree.node;
 
 import de.hft.stuttgart.citydoctor2.gui.Renderer;
+import de.hft.stuttgart.citydoctor2.gui.tree.Renderable;
 
 public class TextNode extends Renderable {
 	

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/node/TopLevelCityObjectNode.java

+package de.hft.stuttgart.citydoctor2.gui.tree.node;
+
+import de.hft.stuttgart.citydoctor2.datastructure.CityObject;
+import de.hft.stuttgart.citydoctor2.gui.MainWindow;
+import de.hft.stuttgart.citydoctor2.utils.Localization;
+import javafx.scene.input.Clipboard;
+import javafx.scene.input.ClipboardContent;
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
+
+import static javafx.scene.input.Clipboard.getSystemClipboard;
+
+
+public class TopLevelCityObjectNode extends CityObjectNode {
+
+    private static final Logger logger = LogManager.getLogger(TopLevelCityObjectNode.class);
+
+    public TopLevelCityObjectNode(CityObject co) {
+        super(co);
+    }
+
+    @Override
+    public void getClipboardAction(MainWindow mainWindow) {
+        Clipboard clipboard = getSystemClipboard();
+        ClipboardContent content = new ClipboardContent();
+        content.putString(this.getFeature().getGmlId().toString());
+        if (!clipboard.setContent(content)) {
+            if (logger.isWarnEnabled()) {
+                logger.warn(Localization.getText("MainWindow.warnCopyIdFailed"));
+            }
+
+        }
+    }
+
+    @Override
+    public void getFeatureExportAction(MainWindow mainWindow) {
+        try {
+            mainWindow.getController().export(this.getFeature());
+            if (logger.isWarnEnabled()) {
+                logger.info(Localization.getText("CityDoctorController.exportSuccess"));
+            }
+        } catch (Exception e) {
+            logger.error("Export failed: Unexpected error occurred");
+            logger.error(e);
+        }
+    }
+}

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/VertexNode.java → CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/node/VertexNode.java

-package de.hft.stuttgart.citydoctor2.gui.tree;
+package de.hft.stuttgart.citydoctor2.gui.tree.node;
 
 import java.text.DecimalFormat;
 import java.text.DecimalFormatSymbols;
 
 import de.hft.stuttgart.citydoctor2.datastructure.Vertex;
 import de.hft.stuttgart.citydoctor2.gui.Renderer;
+import de.hft.stuttgart.citydoctor2.gui.tree.Renderable;
 
 public class VertexNode extends Renderable {
 	

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/ZipEntryNode.java → CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/tree/node/ZipEntryNode.java

-package de.hft.stuttgart.citydoctor2.gui.tree;
+package de.hft.stuttgart.citydoctor2.gui.tree.node;
 
 import de.hft.stuttgart.citydoctor2.zip.CityGmlZipEntry;
 import javafx.scene.paint.Color;

CityDoctorParent/Extensions/CityDoctorHealer/pom.xml

@@ -6,7 +6,7 @@
 	<parent>
 		<groupId>de.hft.stuttgart</groupId>
 		<artifactId>CityDoctorParent</artifactId>
-		<version>3.17.1</version>
+		<version>3.17.2</version>
 		<relativePath>../../pom.xml</relativePath>
 	</parent>
 	<artifactId>CityDoctorHealer</artifactId>

CityDoctorParent/Extensions/CityDoctorHealer/src/main/java/de/hft/stuttgart/citydoctor2/healer/Healer.java

@@ -263,7 +263,7 @@ public class Healer {
 			co.reCreateGeometries(factory, checker.getConfig().getParserConfiguration());
 			byte[] gml = writeCityGml(co.getGmlObject());
 			// remove them again to save memory
-			co.unsetGmlGeometries();
+			co.clearGmlGeometries();
 			SvrlContentHandler handler = Checker.executeSchematronValidationIfAvailable(config,
 					new ByteArrayInputStream(gml));
 			if (handler != null) {

CityDoctorParent/Extensions/CityDoctorHealer/src/test/java/de/hft/stuttgart/citydoctor2/healing/HealHoleOutsideErrorTest.java

@@ -2,6 +2,8 @@ package de.hft.stuttgart.citydoctor2.healing;
 
 import java.io.IOException;
 
+import de.hft.stuttgart.citydoctor2.datastructure.BoundarySurface;
+import de.hft.stuttgart.citydoctor2.datastructure.Building;
 import org.junit.Assert;
 import org.junit.Test;
 
@@ -24,14 +26,14 @@ public class HealHoleOutsideErrorTest {
 
 		Checker c = new Checker(config, model);
 		c.runChecks();
-		
-		Geometry geom = model.getBuildings().get(0).getGeometries().get(0);
-		Assert.assertTrue(geom.containsError(CheckId.C_GE_P_HOLE_OUTSIDE));
+
+		Building building = model.getBuildings().get(0);
+		Assert.assertTrue(building.containsError(CheckId.C_GE_P_HOLE_OUTSIDE));
 		
 		Healer healer = new Healer(c);
 		healer.healCityObject(model.getBuildings().get(0));
 
-		Assert.assertFalse(geom.containsError(CheckId.C_GE_P_HOLE_OUTSIDE));
+		Assert.assertFalse(building.containsError(CheckId.C_GE_P_HOLE_OUTSIDE));
 	}
 
 }

CityDoctorParent/Extensions/CityDoctorHealerGUI/pom.xml

@@ -7,7 +7,7 @@
 	<parent>
 		<groupId>de.hft.stuttgart</groupId>
 		<artifactId>CityDoctorParent</artifactId>
-		<version>3.17.1</version>
+		<version>3.17.2</version>
 		<relativePath>../../pom.xml</relativePath>
 	</parent>
 	<artifactId>CityDoctorHealerGUI</artifactId>
@@ -26,14 +26,14 @@
 			<groupId>de.hft.stuttgart</groupId>
 			<artifactId>CityDoctorAutoPro</artifactId>
 		</dependency>
-					<dependency>
-				<groupId>org.openjfx</groupId>
-				<artifactId>javafx-controls</artifactId>
-			</dependency>
-			<dependency>
-				<groupId>org.openjfx</groupId>
-				<artifactId>javafx-fxml</artifactId>
-			</dependency>
+		<dependency>
+			<groupId>org.openjfx</groupId>
+			<artifactId>javafx-controls</artifactId>
+		</dependency>
+		<dependency>
+			<groupId>org.openjfx</groupId>
+			<artifactId>javafx-fxml</artifactId>
+		</dependency>
 	</dependencies>
 	<build>
 		<plugins>

CityDoctorParent/Extensions/CityDoctorHealerGenetic/pom.xml

@@ -4,7 +4,7 @@
 	<parent>
 		<groupId>de.hft.stuttgart</groupId>
 		<artifactId>CityDoctorParent</artifactId>
-		<version>3.17.1</version>
+		<version>3.17.2</version>
 		<relativePath>../../pom.xml</relativePath>
 	</parent>
 	<artifactId>CityDoctorHealerGenetic</artifactId>

CityDoctorParent/Extensions/CityDoctorHealerGenetic/src/test/java/de/hft/stuttgart/citydoctor2/healer/genetic/NonPlanarGenetic.java

@@ -66,9 +66,9 @@ public class NonPlanarGenetic {
 	private static final int ERROR_PENALTY = 100000;
 	private Random r = new Random();
 	
-	private Map<Integer, List<Long>> durationMap = new HashMap<>(); 
+	private Map<Integer, List<Long>> durationMap = new HashMap<>();
 
-	@Test
+	//@Test
 	public void findVerticesForNonPlanarTest() throws CityGmlParseException, InvalidGmlFileException, IOException {
 		ValidationConfiguration config = ValidationConfiguration.loadStandardValidationConfig();
 		config.setNumberOfRoundingPlacesInGlobalParameters(8);

CityDoctorParent/pom.xml

@@ -5,7 +5,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>de.hft.stuttgart</groupId>
     <artifactId>CityDoctorParent</artifactId>
-    <version>3.17.1</version>
+    <version>3.17.2</version>
     <packaging>pom</packaging>
     <name>CityDoctorParent</name>
     <properties>
@@ -115,6 +115,12 @@
                 <artifactId>CityDoctorHealer</artifactId>
                 <version>${revision}</version>
             </dependency>
+			<dependency>
+				<groupId>commons-io</groupId>
+				<artifactId>commons-io</artifactId>
+				<version>2.16.1</version>
+				<scope>compile</scope>
+			</dependency>
             <dependency>
                 <groupId>net.sf.saxon</groupId>
                 <artifactId>Saxon-HE</artifactId>
@@ -143,7 +149,7 @@
             <dependency>
                 <groupId>org.apache.xmlgraphics</groupId>
                 <artifactId>fop</artifactId>
-                <version>2.7</version>
+                <version>2.10</version>
             </dependency>
             <dependency>
                 <groupId>org.jdom</groupId>
@@ -248,22 +254,18 @@
         <dependency>
             <groupId>org.openjfx</groupId>
             <artifactId>javafx-controls</artifactId>
-            <version>${jfx-version}</version>
         </dependency>
         <dependency>
             <groupId>org.openjfx</groupId>
             <artifactId>javafx-fxml</artifactId>
-            <version>${jfx-version}</version>
         </dependency>
         <dependency>
             <groupId>org.openjfx</groupId>
             <artifactId>javafx-swing</artifactId>
-            <version>${jfx-version}</version>
         </dependency>
         <dependency>
             <groupId>info.picocli</groupId>
             <artifactId>picocli</artifactId>
-            <version>4.7.6</version>
         </dependency>
     </dependencies>
     <modules>