Changeset 12134 in josm for trunk/src/org/openstreetmap/josm/io/session

Timestamp:

2017-05-13T12:58:12+02:00 (7 years ago)

Author:

bastiK

Message:

see #14773 - improve session export/import for imagery layer offset data

Location:

trunk/src/org/openstreetmap/josm/io/session

Files:

• ImagerySessionExporter.java (modified) (2 diffs)
• ImagerySessionImporter.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/io/session/ImagerySessionExporter.java

@@ -14 +14 @@
 import org.openstreetmap.josm.data.Preferences;
 import org.openstreetmap.josm.data.imagery.ImageryInfo.ImageryPreferenceEntry;
+import org.openstreetmap.josm.data.imagery.OffsetBookmark;
 import org.openstreetmap.josm.gui.layer.AbstractTileSourceLayer;
 import org.openstreetmap.josm.gui.layer.ImageryLayer;
@@ -82 +83 @@
         Map<String, String> data = new LinkedHashMap<>(Preferences.serializeStruct(e, ImageryPreferenceEntry.class));
         if (layer instanceof AbstractTileSourceLayer) {
-            AbstractTileSourceLayer<?> tsLayer = (AbstractTileSourceLayer<?>) layer;
-            tsLayer.getDisplaySettings().storeTo(data);
+            ((AbstractTileSourceLayer<?>) layer).getDisplaySettings().storeTo(data);
         }
-        for (Map.Entry<String, String> entry : data.entrySet()) {
-            Element attrElem = support.createElement(entry.getKey());
-            layerElem.appendChild(attrElem);
-            attrElem.appendChild(support.createTextNode(entry.getValue()));
+        addAttributes(layerElem, data, support);
+        if (layer instanceof AbstractTileSourceLayer) {
+            OffsetBookmark offset = ((AbstractTileSourceLayer<?>) layer).getDisplaySettings().getOffsetBookmark();
+            if (offset != null) {
+                Map<String, String> offsetProps = offset.toPropertiesMap();
+                Element offsetEl = support.createElement("offset");
+                layerElem.appendChild(offsetEl);
+                addAttributes(offsetEl, offsetProps, support);
+            }
         }
         return layerElem;
     }
+
+    private void addAttributes(Element element, Map<String, String> props, ExportSupport support) {
+        for (Map.Entry<String, String> entry : props.entrySet()) {
+            Element attrElem = support.createElement(entry.getKey());
+            element.appendChild(attrElem);
+            attrElem.appendChild(support.createTextNode(entry.getValue()));
+        }
+    }
 }

trunk/src/org/openstreetmap/josm/io/session/ImagerySessionImporter.java

@@ -11 +11 @@
 import org.openstreetmap.josm.data.imagery.ImageryInfo;
 import org.openstreetmap.josm.data.imagery.ImageryInfo.ImageryPreferenceEntry;
+import org.openstreetmap.josm.data.imagery.OffsetBookmark;
 import org.openstreetmap.josm.gui.layer.AbstractTileSourceLayer;
 import org.openstreetmap.josm.gui.layer.ImageryLayer;
@@ -36 +37 @@
             throw new IllegalDataException(tr("Version ''{0}'' of meta data for imagery layer is not supported. Expected: 0.1", version));
         }
+        Map<String, String> attributes = readProperties(elem);
+
+        ImageryPreferenceEntry prefEntry = Preferences.deserializeStruct(attributes, ImageryPreferenceEntry.class);
+        ImageryInfo info = new ImageryInfo(prefEntry);
+        ImageryLayer layer = ImageryLayer.create(info);
+        if (layer instanceof AbstractTileSourceLayer) {
+            AbstractTileSourceLayer<?> tsLayer = (AbstractTileSourceLayer<?>) layer;
+            tsLayer.getDisplaySettings().loadFrom(attributes);
+            NodeList nodes = elem.getChildNodes();
+            for (int i = 0; i < nodes.getLength(); ++i) {
+                Node node = nodes.item(i);
+                if (node.getNodeType() == Node.ELEMENT_NODE && "offset".equals(node.getLocalName())) {
+                    Map<String, String> offsetAttributes = readProperties((Element) node);
+                    OffsetBookmark offset = OffsetBookmark.fromPropertiesMap(offsetAttributes);
+                    tsLayer.getDisplaySettings().setOffsetBookmark(offset);
+                    break;
+                }
+            }
+        }
+        return layer;
+    }
+
+    private Map<String, String> readProperties(Element elem) {
         Map<String, String> attributes = new HashMap<>();
-
         NodeList nodes = elem.getChildNodes();
-
         for (int i = 0; i < nodes.getLength(); ++i) {
             Node node = nodes.item(i);
-            if (node.getNodeType() == Node.ELEMENT_NODE) {
+            if (node.getNodeType() == Node.ELEMENT_NODE && node.getChildNodes().getLength() <= 1) {
                 Element e = (Element) node;
                 attributes.put(e.getTagName(), e.getTextContent());
             }
         }
-        ImageryPreferenceEntry prefEntry = Preferences.deserializeStruct(attributes, ImageryPreferenceEntry.class);
-        ImageryInfo i = new ImageryInfo(prefEntry);
-        ImageryLayer layer = ImageryLayer.create(i);
-        if (layer instanceof AbstractTileSourceLayer) {
-            AbstractTileSourceLayer<?> tsLayer = (AbstractTileSourceLayer<?>) layer;
-            tsLayer.getDisplaySettings().loadFrom(attributes);
-        }
-        return layer;
+        return attributes;
     }
 }