Changeset 14305 in josm for trunk/src/org/openstreetmap

Timestamp:

2018-10-07T19:42:42+02:00 (6 years ago)

Author:

Don-vip

Message:

fix #16810 - Disable 'Way end node near other highway' warnings for railway=platform_edge

File:

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

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

@@ -181 +181 @@
                     return map;
                 }
-                for (Node en : s.nearbyNodes(mindist)) {
-                    if (en == null || !s.highway || !endnodesHighway.contains(en)) {
-                        continue;
+                if (s.highway) {
+                    for (Node en : s.nearbyNodes(mindist)) {
+                        if (en == null || !endnodesHighway.contains(en)) {
+                            continue;
+                        }
+                        if (en.hasTag(HIGHWAY, "turning_circle", "bus_stop")
+                                || en.hasTag("amenity", "parking_entrance")
+                                || en.hasTag(RAILWAY, "buffer_stop")
+                                || en.isKeyTrue("noexit")
+                                || en.hasKey("entrance", "barrier")) {
+                            continue;
+                        }
+                        // to handle intersections of 't' shapes and similar
+                        if (en.isConnectedTo(s.w.getNodes(), 3 /* hops */, null)) {
+                            continue;
+                        }
+                        map.put(en, s.w);
                     }
-                    if (en.hasTag(HIGHWAY, "turning_circle", "bus_stop")
-                            || en.hasTag("amenity", "parking_entrance")
-                            || en.hasTag(RAILWAY, "buffer_stop")
-                            || en.isKeyTrue("noexit")
-                            || en.hasKey("entrance", "barrier")) {
-                        continue;
-                    }
-                    // to handle intersections of 't' shapes and similar
-                    if (en.isConnectedTo(s.w.getNodes(), 3 /* hops */, null)) {
-                        continue;
-                    }
-                    map.put(en, s.w);
                 }
             }
@@ -453 +455 @@
                 && !w.hasKey("addr:interpolation")
                 // similarly for public transport platforms, tree rows
-                && !w.hasTag(HIGHWAY, "platform") && !w.hasTag(RAILWAY, "platform") && !w.hasTag("natural", "tree_row")
+                && !w.hasTag(HIGHWAY, "platform") && !w.hasTag(RAILWAY, "platform", "platform_edge") && !w.hasTag("natural", "tree_row")
                 ) {
             ways.addAll(getWaySegments(w));