Changeset 18323 in josm for trunk/src/org

Timestamp:

2021-11-12T11:23:59+01:00 (3 years ago)

Author:

GerdP

Message:

fix #21541: false positive: crossing railways

• treat railway=halt similar to railway=yard

File:

• trunk/src/org/openstreetmap/josm/data/validation/tests/CrossingWays.java (modified) (1 diff)

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

@@ -126 +126 @@
             if (isWaterArea(w1) && isWaterArea(w2))
                 return true; // handled by mapcss tests
-            if (w1.hasKey(RAILWAY) && w2.hasKey(RAILWAY) && w1.hasTag(RAILWAY, "yard") != w2.hasTag(RAILWAY, "yard")) {
-                return true; // see #20089
-            }
+            if (w1.hasKey(RAILWAY) && w2.hasKey(RAILWAY) && (w1.hasTag(RAILWAY, "yard") != w2.hasTag(RAILWAY, "yard")
+                    || w1.hasTag(RAILWAY, "halt") != w2.hasTag(RAILWAY, "halt")))
+                return true;  // see #20089, #21541
             return (w1.hasTag(WATERWAY, "river", "stream", "canal", "drain", "ditch") && isWaterArea(w2))
                     || (w2.hasTag(WATERWAY, "river", "stream", "canal", "drain", "ditch") && isWaterArea(w1));