Changeset 3092 in josm for trunk/src/org/openstreetmap/josm/actions/UpdateModifiedAction.java

Timestamp:

07.03.2010 14:34:27 (2 years ago)

Author:

Gubaer

Message:

fixed #4681: Open Street Bug Plugin Crashed

File:

• trunk/src/org/openstreetmap/josm/actions/UpdateModifiedAction.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/UpdateModifiedAction.java

@@ -32 +32 @@
     }
 
+    // FIXME: overrides the behaviour of UpdateSelectionAction. Doesn't update
+    // the enabled state based on the current selection because
+    // it doesn't depend on it.
+    // The action should be enabled/disabled based on whether there is a least
+    // one modified object in the current dataset. Unfortunately, there is no
+    // efficient way to find out here. getDataSet().allModifiedPrimitives() is
+    // too heavy weight because it loops over the whole dataset.
+    // Perhaps this action should  be a DataSetListener? Or it could listen to the
+    // REQUIRES_SAVE_TO_DISK_PROP and REQUIRES_UPLOAD_TO_SERVER_PROP properties
+    // in the OsmLayer?
+    //
     @Override
-    protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
-        super.updateEnabledState(getData());
+    protected void updateEnabledState() {
+        setEnabled(getCurrentDataSet() != null);
     }
 
     @Override
-    protected Collection<OsmPrimitive> getData() {
-        return getCurrentDataSet().allModifiedPrimitives();
+    protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
     }
 }