Changeset 6271 in josm for trunk/src/org/openstreetmap/josm/io

Timestamp:

2013-09-30T02:51:34+02:00 (11 years ago)

Author:

Don-vip

Message:

fix #9116 - Remember active layer in session

Location:

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

Files:

• SessionReader.java (modified) (3 diffs)
• SessionWriter.java (modified) (2 diffs)

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

@@ -89 +89 @@
     private ZipFile zipFile;
     private List<Layer> layers = new ArrayList<Layer>();
+    private int active = -1;
     private List<Runnable> postLoadTasks = new ArrayList<Runnable>();
     private ViewportData viewport;
@@ -97 +98 @@
     public List<Layer> getLayers() {
         return layers;
+    }
+
+    /**
+     * @return active layer, or {@code null} if not set
+     * @since 6271
+     */
+    public Layer getActive() {
+        // layers is in reverse order because of the way TreeMap is built
+        return (active >= 0 && active < layers.size()) ? layers.get(layers.size()-1-active) : null;
     }
 
@@ -327 +337 @@
         if (layersEl == null) return;
 
+        String activeAtt = layersEl.getAttribute("active");
+        try {
+            active = (activeAtt != null && !activeAtt.isEmpty()) ? Integer.parseInt(activeAtt)-1 : -1;
+        } catch (NumberFormatException e) {
+            Main.warn("Unsupported value for 'active' layer attribute. Ignoring it. Error was: "+e.getMessage());
+            active = -1;
+        }
+        
         MultiMap<Integer, Integer> deps = new MultiMap<Integer, Integer>();
         Map<Integer, Element> elems = new HashMap<Integer, Element>();

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

@@ -80 +80 @@
     }
 
-    private List<Layer> layers;
-    private Map<Layer, SessionLayerExporter> exporters;
-    private MultiMap<Layer, Layer> dependencies;
-    private boolean zip;
+    private final List<Layer> layers;
+    private final int active; 
+    private final Map<Layer, SessionLayerExporter> exporters;
+    private final MultiMap<Layer, Layer> dependencies;
+    private final boolean zip;
 
     private ZipOutputStream zipOut;
 
-    public SessionWriter(List<Layer> layers, Map<Layer, SessionLayerExporter> exporters,
+    /**
+     * Constructs a new {@code SessionWriter}.
+     * @param layers The ordered list of layers to save
+     * @param active The index of active layer in {@code layers} (starts to 0). Ignored if set to -1
+     * @param exporters The exprters to use to save layers
+     * @param zip {@code true} if a joz archive has to be created, {@code false otherwise}
+     * @since 6271
+     */
+    public SessionWriter(List<Layer> layers, int active, Map<Layer, SessionLayerExporter> exporters,
                 MultiMap<Layer, Layer> dependencies, boolean zip) {
         this.layers = layers;
+        this.active = active;
         this.exporters = exporters;
         this.dependencies = dependencies;
@@ -182 +192 @@
 
         Element layersEl = doc.createElement("layers");
+        if (active >= 0) {
+            layersEl.setAttribute("active", Integer.toString(active+1));
+        }
         root.appendChild(layersEl);