Changeset 32289 in osm

Timestamp:

2016-06-16T23:22:00+02:00 (9 years ago)

Author:

donvip

Message:

fix warnings

Location:

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core

Files:

• gui/OdDialog.java (modified) (5 diffs)
• layers/OdDataLayer.java (modified) (7 diffs)

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/OdDialog.java

@@ -19 +19 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.JosmAction;
-import org.openstreetmap.josm.gui.MapView;
-import org.openstreetmap.josm.gui.MapView.EditLayerChangeListener;
-import org.openstreetmap.josm.gui.MapView.LayerChangeListener;
 import org.openstreetmap.josm.gui.SideButton;
 import org.openstreetmap.josm.gui.dialogs.ToggleDialog;
 import org.openstreetmap.josm.gui.layer.Layer;
-import org.openstreetmap.josm.gui.layer.OsmDataLayer;
+import org.openstreetmap.josm.gui.layer.MainLayerManager.ActiveLayerChangeEvent;
+import org.openstreetmap.josm.gui.layer.MainLayerManager.ActiveLayerChangeListener;
 import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
 import org.openstreetmap.josm.plugins.opendata.core.layers.OdDataLayer;
@@ -32 +30 @@
 
 @SuppressWarnings("serial")
-public class OdDialog extends ToggleDialog implements LayerChangeListener, EditLayerChangeListener {
+public class OdDialog extends ToggleDialog implements ActiveLayerChangeListener {
 
     //private final SideButton selectButton;
@@ -122 +120 @@
         createLayout(new JTree(treeModel), true, buttons);
         
-        MapView.addEditLayerChangeListener(this);
+        Main.getLayerManager().addActiveLayerChangeListener(this);
     }
-    
+
     private void disableAllButtons() {
         for (SideButton button : buttons) {
@@ -132 +130 @@
 
     @Override
-    public void editLayerChanged(OsmDataLayer oldLayer, OsmDataLayer newLayer) {
-        activeLayerChange(oldLayer, newLayer);
-    }
-
-    @Override
-    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
+    public void activeOrEditLayerChanged(ActiveLayerChangeEvent e) {
+        Layer newLayer = Main.getLayerManager().getActiveLayer();
         if (newLayer instanceof OdLayer) {
             dataLayer = ((OdLayer) newLayer).getDataLayer();
@@ -157 +151 @@
 
     @Override
-    public void layerAdded(Layer newLayer) {
-    }
-
-    @Override
-    public void layerRemoved(Layer oldLayer) {
-    }
-
-    @Override
     public void destroy() {
         super.destroy();
-        MapView.removeLayerChangeListener(this);
-        MapView.removeEditLayerChangeListener(this);
+        Main.getLayerManager().removeActiveLayerChangeListener(this);
     }
 

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/layers/OdDataLayer.java

@@ -16 +16 @@
 import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.Node;
-import org.openstreetmap.josm.gui.MapView;
-import org.openstreetmap.josm.gui.MapView.LayerChangeListener;
-import org.openstreetmap.josm.gui.layer.Layer;
+import org.openstreetmap.josm.gui.layer.LayerManager.LayerAddEvent;
+import org.openstreetmap.josm.gui.layer.LayerManager.LayerChangeListener;
+import org.openstreetmap.josm.gui.layer.LayerManager.LayerOrderChangeEvent;
+import org.openstreetmap.josm.gui.layer.LayerManager.LayerRemoveEvent;
+import org.openstreetmap.josm.gui.layer.MainLayerManager.ActiveLayerChangeEvent;
+import org.openstreetmap.josm.gui.layer.MainLayerManager.ActiveLayerChangeListener;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
 import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
@@ -29 +32 @@
 import org.openstreetmap.josm.tools.ImageProvider;
 
-public class OdDataLayer extends OsmDataLayer implements OdLayer, LayerChangeListener {
+public class OdDataLayer extends OsmDataLayer implements OdLayer, LayerChangeListener, ActiveLayerChangeListener {
 
     public OdDiffLayer diffLayer;
@@ -52 +55 @@
             }
         }
-        MapView.addLayerChangeListener(this);
+        Main.getLayerManager().addLayerChangeListener(this);
     }
     
@@ -92 +95 @@
             final OdOsmDataLayer layer = new OdOsmDataLayer(this, dataSet, getName()+"/OSM");
             addOsmLayer(layer);
-            Main.map.mapView.setActiveLayer(osmLayer);
+            Main.getLayerManager().setActiveLayer(osmLayer);
             if (oapiReq != null) {
                 OsmDownloader.downloadOapi(oapiReq);
@@ -104 +107 @@
 
     @Override
-    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
-        if (newLayer == this && this.handler != null) {
+    public void activeOrEditLayerChanged(ActiveLayerChangeEvent e) {
+        if (Main.getLayerManager().getActiveLayer() == this && this.handler != null) {
             this.handler.notifyActive();
         }
@@ -111 +114 @@
 
     @Override
-    public void layerAdded(Layer newLayer) {
+    public void layerAdded(LayerAddEvent e) {
     }
 
     @Override
-    public void layerRemoved(Layer oldLayer) {
-        if (oldLayer == this) {
+    public void layerRemoving(LayerRemoveEvent e) {
+        if (e.getRemovedLayer() == this) {
             removeOsmLayer();
             removeDiffLayer();
-        } else if (oldLayer == osmLayer) {
+        } else if (e.getRemovedLayer() == osmLayer) {
             osmLayer = null;
-        } else if (oldLayer == diffLayer) {
+        } else if (e.getRemovedLayer() == diffLayer) {
             diffLayer = null;
         }
+    }
+
+    @Override
+    public void layerOrderChanged(LayerOrderChangeEvent e) {
     }
 
@@ -174 +181 @@
         final OdDiffLayer layer = new OdDiffLayer(this, getName()+"/Diff");
         addDiffLayer(layer);
-        Main.map.mapView.setActiveLayer(diffLayer);
+        Main.getLayerManager().setActiveLayer(diffLayer);
     }
 }