Changeset 14395 in osm for applications/editors/josm/plugins/routing

Timestamp:

2009-04-03T13:19:41+02:00 (15 years ago)

Author:

juangui

Message:

Menu items are enabled or disabled according to availability of a routing layer.

Location:

applications/editors/josm/plugins/routing/src/com/innovant/josm/plugin/routing

Files:

• RoutingPlugin.java (modified) (5 diffs)
• gui/RoutingMenu.java (modified) (7 diffs)

applications/editors/josm/plugins/routing/src/com/innovant/josm/plugin/routing/RoutingPlugin.java

@@ -113 +113 @@
      */
     private IconToggleButton moveRouteNodeButton;
+
+    /**
+     * IconToggleButton for moving route nodes, we use this field to show or hide the button.
+     */
+    private RoutingMenu menu;
 
     /**
@@ -133 +138 @@
         // Add routing layer
         routingLayer = new RoutingLayer(tr("Navigation"));
-        Main.main.menu.add(new RoutingMenu(tr("Routing")));
+        // Add menu
+        menu = new RoutingMenu(tr("Routing"));
+        Main.main.menu.add(menu);
         // Register this class as LayerChangeListener
         Layer.listeners.add(this);
@@ -188 +195 @@
                         removeRouteNodeButton.setVisible(false);
                         moveRouteNodeButton.setVisible(false);
+                        // Enable menu
+                        menu.enableStartItem();
             newFrame.addToggleDialog(routingDialog);
         }
@@ -209 +218 @@
                         removeRouteNodeButton.setVisible(true);
                         moveRouteNodeButton.setVisible(true);
+                        menu.enableRestOfItems();
                         logger.debug("Added routing layer.");
                 }
@@ -223 +233 @@
                         removeRouteNodeButton.setVisible(false);
                         moveRouteNodeButton.setVisible(false);
+                        menu.disableRestOfItems();
                 logger.debug("Removed routing layer.");
                 }

applications/editors/josm/plugins/routing/src/com/innovant/josm/plugin/routing/gui/RoutingMenu.java

@@ -60 +60 @@
         private static final long serialVersionUID = 3559922048225708480L;
 
+        private JMenuItem startMI;
+        private JMenuItem reverseMI;
+        private JMenuItem clearMI;
+        private JMenu criteriaM;
+
         /**
          * @param s
@@ -71 +76 @@
                 JMenu m;
 
-                mi = new JMenuItem(tr("Start routing"));
-                mi.addActionListener(new ActionListener() {
+                startMI = new JMenuItem(tr("Start routing"));
+                startMI.addActionListener(new ActionListener() {
                         public void actionPerformed(ActionEvent e) {
                                 if(!routingLayer.isLayerAdded()) {
@@ -80 +85 @@
                         }
                 });
-                this.add(mi);
+                this.add(startMI);
 
                 this.addSeparator();
                 ButtonGroup group = new ButtonGroup();
 
-                m = new JMenu(tr("Criteria"));
+                criteriaM = new JMenu(tr("Criteria"));
 
                 JRadioButtonMenuItem rshorter = new JRadioButtonMenuItem(tr("Shortest"));
@@ -110 +115 @@
                 group.add(rshorter);
                 group.add(rfaster);
-                m.add(rshorter);
-                m.add(rfaster);
+                criteriaM.add(rshorter);
+                criteriaM.add(rfaster);
 
-                m.addSeparator();
+                criteriaM.addSeparator();
                 JCheckBoxMenuItem cbmi = new JCheckBoxMenuItem("Ignore oneways");
                 cbmi.addItemListener(new ItemListener() {
@@ -125 +130 @@
                         }
                 });
-                m.add(cbmi);
-                this.add(m);
+                criteriaM.add(cbmi);
+                this.add(criteriaM);
 
                 this.addSeparator();
-                mi = new JMenuItem(tr("Reverse route"));
-                mi.addActionListener(new ActionListener() {
+                reverseMI = new JMenuItem(tr("Reverse route"));
+                reverseMI.addActionListener(new ActionListener() {
                         public void actionPerformed(ActionEvent e) {
                                 routingModel.reverseNodes();
@@ -136 +141 @@
                         }
                 });
-                this.add(mi);
+                this.add(reverseMI);
 
-                mi = new JMenuItem(tr("Clear route"));
-                mi.addActionListener(new ActionListener() {
+                clearMI = new JMenuItem(tr("Clear route"));
+                clearMI.addActionListener(new ActionListener() {
                         public void actionPerformed(ActionEvent e) {
                                 // Reset routing nodes and paths
@@ -147 +152 @@
                         }
                 });
-                this.add(mi);
+                this.add(clearMI);
 
+                // Initially disabled
+                disableAllItems();
         }
 
+        public void disableAllItems() {
+                startMI.setEnabled(false);
+                reverseMI.setEnabled(false);
+                clearMI.setEnabled(false);
+                criteriaM.setEnabled(false);
+        }
 
+        public void enableStartItem() {
+                startMI.setEnabled(true);
+        }
+
+        public void enableRestOfItems() {
+                reverseMI.setEnabled(true);
+                clearMI.setEnabled(true);
+                criteriaM.setEnabled(true);
+        }
+
+        public void disableRestOfItems() {
+                reverseMI.setEnabled(false);
+                clearMI.setEnabled(false);
+                criteriaM.setEnabled(false);
+        }
 }