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

Timestamp:

2016-10-26T13:09:07+02:00 (8 years ago)

Author:

simon04

Message:

fix #13827 - Exporting invalid multipolygon as GeoJSON crashes JOSM

File:

• trunk/src/org/openstreetmap/josm/io/GeoJSONWriter.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/io/GeoJSONWriter.java

@@ -18 +18 @@
 import javax.json.stream.JsonGenerator;
 
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.data.coor.EastNorth;
@@ -119 +120 @@
         @Override
         public void visit(Relation r) {
-            if (r != null && r.isMultipolygon() && !r.hasIncompleteMembers()) {
+            if (r == null || !r.isMultipolygon() || r.hasIncompleteMembers()) {
+                return;
+            }
+            try {
                 final Pair<List<JoinedPolygon>, List<JoinedPolygon>> mp = MultipolygonBuilder.joinWays(r);
                 final JsonArrayBuilder polygon = Json.createArrayBuilder();
@@ -130 +134 @@
                 final JsonArrayBuilder multiPolygon = Json.createArrayBuilder().add(polygon);
                 geomObj.add("coordinates", multiPolygon);
+            } catch (MultipolygonBuilder.JoinedPolygonCreationException ex) {
+                Main.warn("GeoJSON: Failed to export multipolygon " + r.getUniqueId());
+                Main.warn(ex);
             }
         }