Commit 4312b075 authored by Riegel's avatar Riegel
Browse files

Fix: Disable QualityADE for new top-level types

Disables writing to the QualityADE when parsing the newly added feature
types as a temporary fix. The QualityADE is only compatible with CityGML
Version 2.0, since the XML namespaces changed in Version 3.0.
Updating the ADE to be compatible with 2.0 and 3.0 is a bigger task,
a discussion if it should be updated vs. deprecation and removal is
required.
2 merge requests!28Version 3.17.0 Release,!26Add ZIP-archive support
Pipeline #10952 failed with stage
in 1 minute and 14 seconds
Showing with 9 additions and 2 deletions
+9 -2
......@@ -29,6 +29,9 @@ import java.util.Set;
import java.util.UUID;
import java.util.concurrent.atomic.AtomicInteger;
import de.hft.stuttgart.citydoctor2.datastructure.AbstractTunnel;
import de.hft.stuttgart.citydoctor2.datastructure.CityFurniture;
import de.hft.stuttgart.citydoctor2.datastructure.GenericCityObject;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.citygml4j.core.model.core.AbstractFeatureProperty;
......@@ -139,8 +142,12 @@ public class StreamCityGmlConsumer implements CityGmlConsumer {
pdfReporter.report(co);
}
// store quality ade datastructures in cityobject
QualityADEUtils.writeQualityAde(co, val);
// TODO: Update QualityADE to be compatible with CityGML 3.0 or deprecate it, unhandled top-level feature types are just skipped for now
if (!(co instanceof AbstractTunnel || co instanceof CityFurniture || co instanceof GenericCityObject)) {
// store quality ade datastructures in cityobject
QualityADEUtils.writeQualityAde(co, val);
}
// recreate geometry
co.reCreateGeometries(gmlFactory, config.getParserConfiguration());
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment