Changeset 5459 in josm for trunk/src/org/openstreetmap/josm/io/FileExporter.java

Timestamp:

2012-08-20T01:11:45+02:00 (14 years ago)

Author:

Don-vip

Message:

fix #2961 - Improve usability of WMS Layer Saving/Loading

• Replaced the unconventional method of creating a blank layer, then loading a .wms file to a standard File->Open approach
• Fixed memory leaks with some actions registered as listeners but never destroyed
• Layer interface modified to allow a generic approach of layer saving in SaveActionBase rather than the previous one restricted to OSM and GPX data
• FileImporters and FileExporters can now be enabled/disabled at runtime, for example when the active layer changes

File:

• trunk/src/org/openstreetmap/josm/io/FileExporter.java (modified) (2 diffs)

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

@@ -8 +8 @@
 
 import org.openstreetmap.josm.actions.ExtensionFileFilter;
+import org.openstreetmap.josm.gui.MapView.LayerChangeListener;
 import org.openstreetmap.josm.gui.layer.Layer;
 
-public abstract class FileExporter {
-    public ExtensionFileFilter filter;
+public abstract class FileExporter implements LayerChangeListener {
+    
+    public final ExtensionFileFilter filter;
+
+    private boolean enabled;
 
     public FileExporter(ExtensionFileFilter filter) {
         this.filter = filter;
+        this.enabled = true;
     }
 
@@ -24 +29 @@
         throw new IOException(tr("Could not export ''{0}''.", file.getName()));
     }
+
+    /**
+     * Returns the enabled state of this {@code FileExporter}. When enabled, it is listed and usable in "File->Save" dialogs. 
+     * @return true if this {@code FileExporter} is enabled
+     * @since 5459
+     */
+    public final boolean isEnabled() {
+        return enabled;
+    }
+
+    /**
+     * Sets the enabled state of the {@code FileExporter}. When enabled, it is listed and usable in "File->Save" dialogs.
+     * @param enabled true to enable this {@code FileExporter}, false to disable it
+     * @since 5459
+     */
+    public final void setEnabled(boolean enabled) {
+        this.enabled = enabled;
+    }
+
+    @Override
+    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
+        // To be overriden by subclasses if their enabled state depends of the active layer nature
+    }
+
+    @Override
+    public void layerAdded(Layer newLayer) {
+        // To be overriden by subclasses if needed
+    }
+
+    @Override
+    public void layerRemoved(Layer oldLayer) {
+        // To be overriden by subclasses if needed
+    }
 }