Changeset 17313 in osm for applications/editors/josm

Timestamp:

2009-08-28T11:22:08+02:00 (15 years ago)

Author:

dmuecke

Message:

[o3059] - Added way type building into crossing way test.

File:

• applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/CrossingWays.java (modified) (3 diffs)

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/tests/CrossingWays.java

@@ -44 +44 @@
     {
         super(tr("Crossing ways."),
-              tr("This test checks if two roads, railways or waterways crosses in the same layer, but are not connected by a node."));
+              tr("This test checks if two roads, railways, waterways or buildings crosses in the same layer, but are not connected by a node."));
     }
 
@@ -74 +74 @@
         String railway1 = w.get("railway");
         boolean isSubway1 = railway1 != null && railway1.equals("subway");
-        if( w.get("highway") == null && w.get("waterway") == null && (railway1 == null || isSubway1)  && !isCoastline1)
+        boolean isBuilding = false;
+        if(w.get("building") != null)
+            isBuilding = true;
+        
+        if( w.get("highway") == null && w.get("waterway") == null && (railway1 == null || isSubway1)  && !isCoastline1 && !isBuilding)
             return;
 
@@ -116 +120 @@
 
                         errors.add(new TestError(this, Severity.WARNING,
-                        tr("Crossing ways"), CROSSING_WAYS, prims, highlight));
+                        isBuilding ? tr("Crossing buildings") : tr("Crossing ways"), CROSSING_WAYS, prims, highlight));
                         ways_seen.put(prims, highlight);
                     }