Changeset 4814 in josm

Timestamp:

Jan 17, 2012 10:59:35 PM (16 months ago)

Author:

bastiK

Message:

fixed #7193 - Opening more than two gpx-Tracks fails (regression)

File:

• trunk/src/org/openstreetmap/josm/io/OsmImporter.java (modified) (6 diffs)

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

@@ -22 +22 @@
 public class OsmImporter extends FileImporter {
 
-    protected OsmDataLayer layer;
-    protected Runnable postLayerTask;
+    protected class OsmImporterData {
+        public OsmDataLayer layer;
+        public Runnable postLayerTask;
+    }
 
     public OsmImporter() {
@@ -45 +47 @@
 
     protected void importData(InputStream in, final File associatedFile) throws IllegalDataException {
-        loadLayer(in, associatedFile, associatedFile == null ? OsmDataLayer.createNewName() : associatedFile.getName(), NullProgressMonitor.INSTANCE);
-
-        final OsmDataLayer layer = this.layer;
-        final Runnable postLayerTask = this.postLayerTask;
+        final OsmImporterData data = loadLayer(in, associatedFile, 
+                associatedFile == null ? OsmDataLayer.createNewName() : associatedFile.getName(), NullProgressMonitor.INSTANCE);
 
         // FIXME: remove UI stuff from IO subsystem
@@ -54 +54 @@
             @Override
             public void run() {
-                Main.main.addLayer(layer);
-                postLayerTask.run();
-                layer.onPostLoadFromFile();
+                Main.main.addLayer(data.layer);
+                data.postLayerTask.run();
+                data.layer.onPostLoadFromFile();
             }
         });
@@ -65 +65 @@
      * associatedFile can be null if the stream does not come from a file.
      */
-    public void loadLayer(InputStream in, final File associatedFile, final String layerName, ProgressMonitor progressMonitor) throws IllegalDataException {
+    public OsmImporterData loadLayer(InputStream in, final File associatedFile, final String layerName, ProgressMonitor progressMonitor) throws IllegalDataException {
         final DataSet dataSet = parseDataSet(in, progressMonitor);
         if (dataSet == null) {
             throw new IllegalDataException(tr("Invalid dataset"));
         }
-        layer = createLayer(dataSet, associatedFile, layerName);
-        postLayerTask = createPostLayerTask(dataSet, associatedFile, layerName);
+        OsmImporterData data = new OsmImporterData();
+        data.layer = createLayer(dataSet, associatedFile, layerName);
+        data.postLayerTask = createPostLayerTask(dataSet, associatedFile, layerName, data.layer);
+        return data;
     }
     
@@ -82 +84 @@
     }
     
-    protected Runnable createPostLayerTask(final DataSet dataSet, final File associatedFile, final String layerName) {
+    protected Runnable createPostLayerTask(final DataSet dataSet, final File associatedFile, final String layerName, final OsmDataLayer layer) {
         return new Runnable() {
             @Override
@@ -103 +105 @@
         };
     }
-
-    public OsmDataLayer getLayer() {
-        return layer;
-    }
-
-    public Runnable getPostLayerTask() {
-        return postLayerTask;
-    }
 }