Changeset 5459 in josm for trunk/src/org/openstreetmap/josm/gui/ImageryMenu.java

Timestamp:

2012-08-20T01:11:45+02:00 (12 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/gui/ImageryMenu.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/ImageryMenu.java

@@ -24 +24 @@
 import org.openstreetmap.josm.gui.layer.ImageryLayer;
 import org.openstreetmap.josm.gui.layer.Layer;
-import org.openstreetmap.josm.gui.layer.WMSLayer;
 import org.openstreetmap.josm.tools.ImageProvider;
 
@@ -69 +68 @@
     JMenuItem offsetMenuItem = singleOffset;
     Map_Rectifier_WMSmenuAction rectaction = new Map_Rectifier_WMSmenuAction();
-    JosmAction blankmenu = new JosmAction(
-            tr("Blank Layer"), /* ICON */"blankmenu", tr("Open a blank WMS layer to load data from a file"), null, false) {
-        @Override
-        public void actionPerformed(ActionEvent ev) {
-            if (!isEnabled()) return;
-            Main.main.addLayer(new WMSLayer());
-        }
-
-        @Override
-        protected void updateEnabledState() {
-            setEnabled(Main.map != null && Main.map.mapView != null && !Main.map.mapView.getAllLayers().isEmpty());
-        }
-    };
     int offsPos;
 
@@ -102 +88 @@
         offsPos = getMenuComponentCount();
         add(offsetMenuItem);
-        addSeparator();
-        add(new JMenuItem(blankmenu));
     }