Changeset 8475 in josm for trunk/src/org/openstreetmap/josm/actions/InfoWebAction.java

Timestamp:

2015-06-07T15:46:39+02:00 (9 years ago)

Author:

Don-vip

Message:

fix #11219 - Open notes in browser with Ctrl-Shift-I: "Advanced info (web)"

File:

• trunk/src/org/openstreetmap/josm/actions/InfoWebAction.java (modified) (2 diffs)

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

@@ -6 +6 @@
 
 import java.awt.event.KeyEvent;
+import java.util.Collection;
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.data.notes.Note;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
@@ -31 +33 @@
 
     @Override
-    protected  String createInfoUrl(Object infoObject) {
-        OsmPrimitive primitive = (OsmPrimitive)infoObject;
-        return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId();
+    protected String createInfoUrl(Object infoObject) {
+        if (infoObject instanceof OsmPrimitive) {
+            OsmPrimitive primitive = (OsmPrimitive)infoObject;
+            return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId();
+        } else if (infoObject instanceof Note) {
+            Note note = (Note)infoObject;
+            return Main.getBaseBrowseUrl() + "/note/" + note.getId();
+        } else {
+            return null;
+        }
+    }
+
+    @Override
+    protected void updateEnabledState() {
+        super.updateEnabledState();
+        updateEnabledStateWithNotes();
+    }
+
+    @Override
+    protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
+        super.updateEnabledState(selection);
+        updateEnabledStateWithNotes();
+    }
+
+    private void updateEnabledStateWithNotes() {
+        // Allows enabling if a note is selected, even if no OSM object is selected
+        if (!isEnabled() && Main.isDisplayingMapView()) {
+            if (Main.map.noteDialog.getSelectedNote() != null) {
+                setEnabled(true);
+            }
+        }
+    }
+
+    /**
+     * Called when the note selection has changed.
+     * TODO: make a proper listener mechanism to handle change of note selection
+     * @since 8475
+     */
+    public final void noteSelectionChanged() {
+        updateEnabledState();
     }
 }