Changeset 6518 in josm for trunk/src

Timestamp:

2013-12-24T10:16:30+01:00 (10 years ago)

Author:

simon04

Message:

see #3531 - validator: unconnected ways: ignore highway/railway=platform

File:

• trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java

@@ -127 +127 @@
                         continue;
                     }
-                    if ("turning_circle".equals(en.get("highway"))
-                            || "bus_stop".equals(en.get("highway"))
-                            || "buffer_stop".equals(en.get("railway"))
+                    if (en.hasTag("highway", "turning_circle", "bus_stop")
+                            || en.hasTag("railway", "buffer_stop")
                             || OsmUtils.isTrue(en.get("noexit"))
                             || en.hasKey("barrier")) {
@@ -350 +349 @@
         if (!w.isUsable()
                 || w.hasKey("barrier")
-                || "cliff".equals(w.get("natural")))
+                || w.hasTag("natural", "cliff"))
             return ret;
 
@@ -375 +374 @@
     @Override
     public void visit(Way w) {
-        // Do not consider empty ways and addr:interpolation ways as they are not physical features and most of the time
-        // very near the associated highway, which is perfectly normal, see #9332
-        if (w.getNodesCount() > 0 && !w.hasKey("addr:interpolation")) {
+        if (w.getNodesCount() > 0 // do not consider empty ways
+                && !w.hasKey("addr:interpolation") // ignore addr:interpolation ways as they are not physical features and most of the time very near the associated highway, which is perfectly normal, see #9332
+                && !w.hasTag("highway", "platform") && !w.hasTag("railway", "platform") // similarly for public transport platforms
+                ) {
             ways.addAll(getWaySegments(w));
             QuadBuckets<Node> set = endnodes;