Changeset 19138 in osm for applications/editors

Timestamp:

2009-12-19T01:37:20+01:00 (15 years ago)

Author:

bilbo

Message:

Fixed #4197

Multipolygon menu entry is now enabled only if layer is loaded and something is selected

Location:

applications/editors/josm/plugins/multipoly/src/multipoly

Files:

• MultipolyAction.java (modified) (3 diffs)
• MultipolyPlugin.java (modified) (1 diff)

applications/editors/josm/plugins/multipoly/src/multipoly/MultipolyAction.java

@@ -33 +33 @@
  * New relation with type=multipolygon is created
  *
- * If one or more of ways is already in relation with type=multipolygon or the way os not closed,
+ * If one or more of ways is already in relation with type=multipolygon or the way is not closed,
  * then error is reported and no relation is created
  *
@@ -49 +49 @@
   Shortcut.registerShortcut("tools:multipoly", tr("Tool: {0}", tr("Create multipolygon")),
   KeyEvent.VK_M, Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true);
-  setEnabled(true);
  }
 
@@ -144 +143 @@
  }
 
+ /** Enable this action only if something is selected */
+ @Override protected void updateEnabledState() {
+  if (getCurrentDataSet()==null) {
+   setEnabled(false);
+  } else {
+   updateEnabledState(getCurrentDataSet().getSelected());
+  }
+ }
+
+ /** Enable this action only if something is selected */
+ @Override protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
+  setEnabled(selection != null && !selection.isEmpty());
+ }
+
 }

applications/editors/josm/plugins/multipoly/src/multipoly/MultipolyPlugin.java

@@ -35 +35 @@
       toolsMenu.add(new JMenuItem(new MultipolyAction()));
     }
-    
+
   }