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

Timestamp:

2020-08-26T20:40:25+02:00 (4 years ago)

Author:

simon04

Message:

fix #19632 - GeoJSONWriter: write key={value} as JSON object (patch by taylor.smock, modified)

File:

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

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

@@ -2 +2 @@
 package org.openstreetmap.josm.io;
 
+import java.io.StringReader;
 import java.io.StringWriter;
 import java.math.BigDecimal;
@@ -22 +23 @@
 import javax.json.JsonWriter;
 import javax.json.stream.JsonGenerator;
+import javax.json.stream.JsonParser;
+import javax.json.stream.JsonParsingException;
 
 import org.openstreetmap.josm.data.Bounds;
@@ -55 +58 @@
 
     /**
+     * This is used to determine that a tag should be interpreted as a json
+     * object or array. The tag should have both {@link #JSON_VALUE_START_MARKER}
+     * and {@link #JSON_VALUE_END_MARKER}.
+     */
+    static final String JSON_VALUE_START_MARKER = "{";
+    /**
+     * This is used to determine that a tag should be interpreted as a json
+     * object or array. The tag should have both {@link #JSON_VALUE_START_MARKER}
+     * and {@link #JSON_VALUE_END_MARKER}.
+     */
+    static final String JSON_VALUE_END_MARKER = "}";
+
+    /**
      * Constructs a new {@code GeoJSONWriter}.
      * @param ds The OSM data set to save
@@ -183 +199 @@
         final JsonObjectBuilder propObj = Json.createObjectBuilder();
         for (Entry<String, String> t : p.getKeys().entrySet()) {
-            propObj.add(t.getKey(), t.getValue());
+            propObj.add(t.getKey(), convertValueToJson(t.getValue()));
         }
         final JsonObject prop = propObj.build();
@@ -199 +215 @@
                     .add("geometry", geom.isEmpty() ? JsonValue.NULL : geom));
         }
+    }
+
+    private static JsonValue convertValueToJson(String value) {
+        if (value.startsWith(JSON_VALUE_START_MARKER) && value.endsWith(JSON_VALUE_END_MARKER)) {
+            try (JsonParser parser = Json.createParser(new StringReader(value))) {
+                if (parser.hasNext() && parser.next() != null) {
+                    return parser.getValue();
+                }
+            } catch (JsonParsingException e) {
+                Logging.warn(e);
+            }
+        }
+        return Json.createValue(value);
     }