Changeset 32365 in osm

Timestamp:

2016-06-23T00:20:45+02:00 (8 years ago)

Author:

floscher

Message:

Use ActiveLayerChangeListener instead of EditLayerChangeListener and LayerChangeListener

See #11838, #12924, #12937

File:

• applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryLayer.java (modified) (9 diffs)

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryLayer.java

@@ -1 +1 @@
 // License: GPL. For details, see LICENSE file.
 package org.openstreetmap.josm.plugins.mapillary;
-
-import static org.openstreetmap.josm.tools.I18n.marktr;
-import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.awt.AlphaComposite;
@@ -43 +40 @@
 import org.openstreetmap.josm.data.osm.visitor.paint.PaintColors;
 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.NavigatableComponent;
 import org.openstreetmap.josm.gui.dialogs.LayerListDialog;
@@ -50 +45 @@
 import org.openstreetmap.josm.gui.layer.AbstractModifiableLayer;
 import org.openstreetmap.josm.gui.layer.Layer;
+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.mapillary.cache.CacheUtils;
@@ -63 +60 @@
 import org.openstreetmap.josm.plugins.mapillary.utils.MapillaryUtils;
 
+import static org.openstreetmap.josm.tools.I18n.marktr;
+import static org.openstreetmap.josm.tools.I18n.tr;
+
 /**
  * This class represents the layer shown in JOSM. There can only exist one
@@ -71 +71 @@
  */
 public final class MapillaryLayer extends AbstractModifiableLayer implements
-    DataSetListener, EditLayerChangeListener, LayerChangeListener {
+    DataSetListener, ActiveLayerChangeListener {
 
   /** Maximum distance for the red/blue lines. */
@@ -108 +108 @@
     if (Main.main != null && Main.map.mapView != null) {
       setMode(new SelectMode());
-      Main.map.mapView.addLayer(this);
-      MapView.addEditLayerChangeListener(this, false);
-      MapView.addLayerChangeListener(this);
+      Main.getLayerManager().addLayer(this);
+      Main.getLayerManager().addActiveLayerChangeListener(this, false);
       if (Main.getLayerManager().getEditLayer() != null)
         Main.getLayerManager().getEditLayer().data.addDataSetListener(this);
@@ -222 +221 @@
     Main.map.mapView.removeMouseListener(this.mode);
     Main.map.mapView.removeMouseMotionListener(this.mode);
-    MapView.removeEditLayerChangeListener(this);
+    Main.getLayerManager().removeActiveLayerChangeListener(this);
     if (Main.getLayerManager().getEditLayer() != null)
       Main.getLayerManager().getEditLayer().data.removeDataSetListener(this);
@@ -498 +497 @@
 
   @Override
-  public void editLayerChanged(OsmDataLayer oldLayer, OsmDataLayer newLayer) {
-    if (oldLayer == null && newLayer != null) {
-      newLayer.data.addDataSetListener(this);
-    } else if (oldLayer != null && newLayer == null) {
-      oldLayer.data.removeDataSetListener(this);
+  public void activeOrEditLayerChanged(ActiveLayerChangeEvent e) {
+    if (Main.getLayerManager().getActiveLayer() == this) {
+      MapillaryUtils.updateHelpText();
+    }
+    MapillaryPlugin.setMenuEnabled(MapillaryPlugin.getJoinMenu(), Main.getLayerManager().getActiveLayer() == this);
+
+    if (Main.getLayerManager().getEditLayer() != e.getPreviousEditLayer()) {
+      if (Main.getLayerManager().getEditLayer() != null) {
+        Main.getLayerManager().getEditLayer().data.addDataSetListener(this);
+      }
+      if (e.getPreviousEditLayer() != null) {
+        e.getPreviousEditLayer().data.removeDataSetListener(this);
+      }
     }
   }
@@ -550 +557 @@
   @Override
   public void visitBoundingBox(BoundingXYVisitor v) {
-  }
-
-  @Override
-  public void activeLayerChange(Layer oldLayer, Layer newLayer) {
-    if (newLayer == this) {
-      MapillaryUtils.updateHelpText();
-      MapillaryPlugin.setMenuEnabled(MapillaryPlugin.getJoinMenu(), true);
-    } else
-      MapillaryPlugin.setMenuEnabled(MapillaryPlugin.getJoinMenu(), false);
-  }
-
-  @Override
-  public void layerAdded(Layer newLayer) {
-    // Do nothing, we're only interested in layer change, not addition
-  }
-
-  @Override
-  public void layerRemoved(Layer oldLayer) {
-    // Do nothing, we're only interested in layer change, not removal
   }