diff --git a/src/main/java/com/onthegomap/planetiler/basemap/layers/Transportation.java b/src/main/java/com/onthegomap/planetiler/basemap/layers/Transportation.java index 2d9da5e..b89470a 100644 --- a/src/main/java/com/onthegomap/planetiler/basemap/layers/Transportation.java +++ b/src/main/java/com/onthegomap/planetiler/basemap/layers/Transportation.java @@ -222,7 +222,7 @@ public class Transportation implements "highway", coalesce(highway, ""), "public_transport", coalesce(publicTransport, ""), "construction", coalesce(construction, "") - ), manMade) : manMade; + ), null) : manMade; } static String highwaySubclass(String highwayClass, String publicTransport, String highway) { diff --git a/src/test/java/com/onthegomap/planetiler/basemap/layers/TransportationTest.java b/src/test/java/com/onthegomap/planetiler/basemap/layers/TransportationTest.java index 28d24fd..e2cee75 100644 --- a/src/test/java/com/onthegomap/planetiler/basemap/layers/TransportationTest.java +++ b/src/test/java/com/onthegomap/planetiler/basemap/layers/TransportationTest.java @@ -444,6 +444,17 @@ public class TransportationTest extends AbstractLayerTest { )))); } + @Test + public void testBridgeConstruction() { + // https://www.openstreetmap.org/way/658335918 was returning class=bridge, should just ignore + assertFeatures(13, List.of(), process(lineFeature(Map.of( + "highway", "construction", + "construction", "bridge", + "man_made", "bridge", + "layer", "1" + )))); + } + @Test public void testRaceway() { assertFeatures(13, List.of(Map.of(