Changeset 8043 in osm for applications/editors/josm/plugins/validator/src

Timestamp:

2008-06-02T11:27:13+02:00 (16 years ago)

Author:

stoecker

Message:

applied patch for bug #774

File:

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

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

@@ -3 +3 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
-import java.util.List;
-import java.util.ArrayList;
+import java.util.*;
 
 import org.openstreetmap.josm.data.coor.LatLon;
@@ -48 +47 @@
         public void endTest() 
         {
+                Map<List<Way>, List<WaySegment>> ways_seen = 
+                        new HashMap<List<Way>, List<WaySegment>>(500);
+
                 for (List<WaySegment> duplicated : nodePairs.values())
                 {
+                        int highway = 0;
+                        int railway = 0;
+
                         if (duplicated.size() > 1)
                         {
                                 List<OsmPrimitive> prims = new ArrayList<OsmPrimitive>();
-                                for (WaySegment ws : duplicated) prims.add(ws.way);
-                                errors.add(new TestError(this, Severity.OTHER,
-                                        tr("Overlapping ways"), prims, duplicated));
+                                List<Way> current_ways = new ArrayList<Way>();
+                                List<WaySegment> highlight;
+
+                                for (WaySegment ws : duplicated) 
+                                {
+                                        if (ws.way.get("highway") != null)
+                                                highway++;
+                                        else if (ws.way.get("railway") != null)
+                                                railway++;
+
+                                        prims.add(ws.way);
+                                        current_ways.add(ws.way);
+                                }
+                                /* These ways not seen before 
+                                 * If two or more of the overlapping ways are
+                                 * highways or railways mark a seperate error   
+                                */
+                                if ((highlight = ways_seen.get(current_ways)) == null)
+                                {
+                                    String errortype = tr("Overlapping ways");
+
+                                     if (highway >1)
+                                        errortype = tr("Overlapping highways");
+                                     else if (railway >1)
+                                        errortype = tr("Overlapping railways");
+
+                                     errors.add(new TestError(this, Severity.OTHER,
+                                        tr(errortype), prims, duplicated));
+                                     ways_seen.put(current_ways, duplicated);
+                                }
+                                else    /* way seen, mark highlight layer only */
+                                {
+                                        for (WaySegment ws : duplicated) 
+                                            highlight.add(ws);
+                                }
                         }
                 }