trafficcue/planetiler-openmaptiles
1
0
Fork 0
mirror of https://github.com/cfpwastaken/planetiler-openmaptiles.git synced 2026-02-04 12:31:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix service/man_made npe (#72)

Browse Source
This commit is contained in:
Michael Barry
2022-02-07 20:41:02 -05:00
committed by GitHub
parent 60c39b8d8a
commit 262d510fef
2 changed files with 19 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/com/onthegomap/planetiler/basemap/layers/Transportation.java
View File

@@ -222,7 +222,7 @@ public class Transportation implements
"highway", coalesce(highway, ""),
"public_transport", coalesce(publicTransport, ""),
"construction", coalesce(construction, "")
), null) : manMade;
), null) : isBridgeOrPier(manMade) ? manMade : null;
}
static String highwaySubclass(String highwayClass, String publicTransport, String highway) {

18
src/test/java/com/onthegomap/planetiler/basemap/layers/TransportationTest.java
View File

@@ -497,6 +497,24 @@ public class TransportationTest extends AbstractLayerTest {
))));
}
@Test
public void testIgnoreManMadeWhenNotBridgeOrPier() {
// https://github.com/onthegomap/planetiler/issues/69
assertFeatures(14, List.of(), process(lineFeature(Map.of(
"man_made", "storage_tank",
"service", "driveway"
))));
assertFeatures(14, List.of(), process(lineFeature(Map.of(
"man_made", "courtyard",
"service", "driveway"
))));
assertFeatures(14, List.of(), process(lineFeature(Map.of(
"man_made", "courtyard",
"service", "driveway",
"name", "Named Driveway"
))));
}
@Test
public void testRaceway() {
assertFeatures(13, List.of(Map.of(