Changeset 32344 in osm

Timestamp:

2016-06-20T23:39:51+02:00 (8 years ago)

Author:

donvip

Message:

remove calls to deprecated methods

Location:

applications/editors/josm/plugins/graphview

Files:

• build.xml (modified) (1 diff)
• src/org/openstreetmap/josm/plugins/graphview/plugin/GraphViewPlugin.java (modified) (8 diffs)
• src/org/openstreetmap/josm/plugins/graphview/plugin/data/JOSMDataSource.java (modified) (1 diff)

applications/editors/josm/plugins/graphview/build.xml

@@ -2 +2 @@
 <project name="graphview" default="dist" basedir=".">
     <property name="commit.message" value="option to change graph colors; closes ticket 5523 in JOSM Trac"/>
-    <property name="plugin.main.version" value="10031"/>
+    <property name="plugin.main.version" value="10279"/>
 
     <!-- Configure these properties (replace "..." accordingly).

applications/editors/josm/plugins/graphview/src/org/openstreetmap/josm/plugins/graphview/plugin/GraphViewPlugin.java

@@ -18 +18 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.gui.MapFrame;
-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.preferences.PreferenceSetting;
 import org.openstreetmap.josm.plugins.Plugin;
@@ -81 +82 @@
         preferences = GraphViewPreferences.getInstance();
         this.preferences.addObserver(this);
-
     }
 
@@ -122 +122 @@
                     graphViewLayer.setNodePositioner(new DefaultNodePositioner());
 
-                    Main.main.addLayer(graphViewLayer);
-
+                    Main.getLayerManager().addLayer(graphViewLayer);
                 }
-
-            }
-
+            }
         } catch (AccessRulesetSyntaxException e) {
             JOptionPane.showMessageDialog(Main.parent, tr("Syntax exception in access ruleset:\n{0}", e));
-            e.printStackTrace();
+            Main.error(e);
         } catch (FileNotFoundException e) {
             JOptionPane.showMessageDialog(Main.parent, tr("File not found:\n{0}", e));
-            e.printStackTrace();
+            Main.error(e);
         } catch (IOException e) {
             JOptionPane.showMessageDialog(Main.parent, tr("Problem when accessing a file:\n{0}", e));
-            e.printStackTrace();
-        }
-
+            Main.error(e);
+        }
     }
 
     /** allows update of GraphViewLayer */
     public void updateGraphViewLayer() {
-
         try {
-
             if (graphViewLayer != null) {
 
@@ -158 +152 @@
                     transitionStructure.forceUpdate();
                 }
-
-            }
-
+            }
         } catch (AccessRulesetSyntaxException e) {
             JOptionPane.showMessageDialog(Main.parent, tr("Syntax exception in access ruleset:\n{0}", e));
-            e.printStackTrace();
+            Main.error(e);
         } catch (FileNotFoundException e) {
             JOptionPane.showMessageDialog(Main.parent, tr("File not found:\n", e));
-            e.printStackTrace();
+            Main.error(e);
         } catch (IOException e) {
             JOptionPane.showMessageDialog(Main.parent, tr("Problem when accessing a file:\n{0}", e));
-            e.printStackTrace();
-        }
-
+            Main.error(e);
+        }
     }
 
     /** repaints the GraphViewLayer without recalculating the graph (visual update) */
     public void repaintGraphViewLayer() {
-
         if (graphViewLayer != null) {
             graphViewLayer.invalidate();
         }
-
     }
 
@@ -211 +200 @@
                 throw new FileNotFoundException(tr("Couldn''t find built-in ruleset {0}", ruleset));
             }
-
         } else {
 
@@ -221 +209 @@
 
             rulesetInputStream = new FileInputStream(rulesetFile);
-
         }
 
         return AccessRulesetReader.readAccessRuleset(rulesetInputStream);
-
     }
 
@@ -241 +227 @@
                 newFrame.addToggleDialog(laneDialog);
             }
-            MapView.addLayerChangeListener(this);
+            Main.getLayerManager().addLayerChangeListener(this);
         } else {
-            MapView.removeLayerChangeListener(this);
-        }
-    }
-
-    public void layerRemoved(Layer oldLayer) {
-        if (oldLayer == graphViewLayer) {
+            Main.getLayerManager().removeLayerChangeListener(this);
+        }
+    }
+
+    @Override
+    public void layerRemoving(LayerRemoveEvent e) {
+        if (e.getRemovedLayer() == graphViewLayer) {
             graphViewLayer = null;
-        } else if (oldLayer == Main.main.getEditLayer()) { //data layer removed
+        } else if (e.getRemovedLayer() == Main.getLayerManager().getEditLayer()) { //data layer removed
             if (graphViewLayer != null) {
-                Main.main.removeLayer(graphViewLayer);
+                Main.getLayerManager().removeLayer(graphViewLayer);
                 graphViewLayer = null;
             }
@@ -258 +245 @@
     }
 
-    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
+    @Override
+    public void layerOrderChanged(LayerOrderChangeEvent e) {
         //do nothing
     }
 
-    public void layerAdded(Layer newLayer) {
+    @Override
+    public void layerAdded(LayerAddEvent e) {
         //do nothing
     }
 
+    @Override
     public void update(Observable arg0, Object arg1) {
-        if (arg0 == preferences) {
-            if (graphViewLayer != null) {
-                graphViewLayer.setColorScheme(preferences.getCurrentColorScheme());
-                graphViewLayer.setArrowheadPlacement(preferences.getArrowheadPlacement());
-            }
-        }
-    }
-
+        if (arg0 == preferences && graphViewLayer != null) {
+            graphViewLayer.setColorScheme(preferences.getCurrentColorScheme());
+            graphViewLayer.setArrowheadPlacement(preferences.getArrowheadPlacement());
+        }
+    }
 }

applications/editors/josm/plugins/graphview/src/org/openstreetmap/josm/plugins/graphview/plugin/data/JOSMDataSource.java

@@ -43 +43 @@
 
     public Iterable<Node> getNodes() {
-        return new FilteredOsmPrimitiveIterable<>(Main.main.getCurrentDataSet().getNodes());
+        return new FilteredOsmPrimitiveIterable<>(Main.getLayerManager().getEditDataSet().getNodes());
     }
 
     public Iterable<Relation> getRelations() {
-        return new FilteredRelationIterable(Main.main.getCurrentDataSet().getRelations());
+        return new FilteredRelationIterable(Main.getLayerManager().getEditDataSet().getRelations());
     }
 
     public Iterable<Way> getWays() {
-        return new FilteredOsmPrimitiveIterable<>(Main.main.getCurrentDataSet().getWays());
+        return new FilteredOsmPrimitiveIterable<>(Main.getLayerManager().getEditDataSet().getWays());
     }