Style: Remove completed TODOs

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/parser/CityGmlParser.java

@@ -169,7 +169,6 @@ public class CityGmlParser {
         }
         return context;
     }
-    //TODO: Implement resolving of library objects with database
 
     /**
      * Parses a given CityGML file and collects the contained CityObject Features in a {@link CityDoctorModel}.
@@ -601,7 +600,6 @@ public class CityGmlParser {
                     }else if (!cdModel.containsGmlId(id)){
                         cgmlModel.getCityObjectMembers().add(new AbstractCityObjectProperty(aco));
                     }
-                    // TODO: add check if aco was parsed successfully, add aco to cmodel if it wasnt
                 }
             }
 

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/BoundingBoxCalculator.java

@@ -97,7 +97,6 @@ public class BoundingBoxCalculator {
 	 * @return the bounding box of the model
 	 */
 	public static BoundingBox calculateBoundingBox(CityDoctorModel model) {
-		// TODO: Rework to Visitor
 		MinMaxExtentVisitor mima = new MinMaxExtentVisitor();
 		model.accept(mima);
 

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/BoundarySurfaceTest.java

@@ -51,8 +51,6 @@ import de.hft.stuttgart.citydoctor2.parser.ParserConfiguration;
 
 public class BoundarySurfaceTest {
 
-	//Todo: Check this class for rework potential with the new Visitors
-
 	@Test
 	public void testVisitorPropagation() {
 		BoundarySurface bs = new BoundarySurface(SurfaceFeatureType.BUILDING, BoundarySurfaceType.WALL, null);

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/BridgeObjectTest.java

@@ -51,7 +51,6 @@ import de.hft.stuttgart.citydoctor2.utils.visitors.UnsetGeometriesVisitor;
 
 public class BridgeObjectTest {
 
-	//TODO: Rework this class with the new Visitors
 
 	@Test
 	public void testAccept() {

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/VegetationTest.java

@@ -26,7 +26,6 @@ import static org.mockito.Mockito.mock;
 
 public class VegetationTest {
 
-    //TODO: Recreate geometries tests
 
     @Test
     public void testContainsError() {

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/CityDoctorController.java

@@ -1899,7 +1899,7 @@ public class CityDoctorController {
     public void delete(TreeItem<Renderable> selectedItem) {
         if (model == null) {
             return;
-        }/* TODO: Rework this properly via Renderable implementation
+        }/*  Old debug function, would first need a refactor due to the RenderNode and GeoDB reworks
         Renderable render = selectedItem.getValue();
         if (render instanceof BuildingNode node) {
             model.getBuildings().remove(node.getBuilding());

CityDoctorParent/Extensions/CityDoctorGUI/src/main/java/de/hft/stuttgart/citydoctor2/gui/MainWindow.java

@@ -801,10 +801,13 @@ public class MainWindow extends Application {
         });
         menu.getItems().add(copyToClipboardOption);
 
+    /* Was used for debugging the repair module, functionless since the RenderNode rework.
+    *   Commented out for now, might be needed later on again
         MenuItem deleteOption = new MenuItem(Localization.getText("MainWindow.delete"));
         deleteOption.setOnAction(ae -> controller.delete(featureView.getSelectionModel().getSelectedItem()));
         menu.getItems().add(deleteOption);
         featureView.setContextMenu(menu);
+     */
     }
 
     public void fetchCopyAction(Renderable node) {