Changeset 8475 in josm for trunk/src/org/openstreetmap/josm/actions

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)"

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• AbstractInfoAction.java (modified) (5 diffs)
• HistoryInfoWebAction.java (modified) (1 diff)
• InfoWebAction.java (modified) (2 diffs)

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

@@ -14 +14 @@
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.data.notes.Note;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.gui.HelpAwareOptionPane;
@@ -68 +69 @@
     }
 
-    protected void launchInfoBrowsersForSelectedPrimitives() {
-        List<OsmPrimitive> primitivesToShow = new ArrayList<>(getCurrentDataSet().getAllSelected());
+    protected void launchInfoBrowsersForSelectedPrimitivesAndNote() {
+        List<OsmPrimitive> primitivesToShow = new ArrayList<>();
+        if (getCurrentDataSet() != null) {
+            primitivesToShow.addAll(getCurrentDataSet().getAllSelected());
+        }
+
+        Note noteToShow = Main.isDisplayingMapView() ? Main.map.noteDialog.getSelectedNote() : null;
 
         // filter out new primitives which are not yet uploaded to the server
@@ -80 +86 @@
         }
 
-        if (primitivesToShow.isEmpty()) {
+        if (primitivesToShow.isEmpty() && noteToShow == null) {
             JOptionPane.showMessageDialog(
                     Main.parent,
@@ -95 +101 @@
         if (primitivesToShow.size() > max && !confirmLaunchMultiple(primitivesToShow.size()))
             return;
-        for(int i = 0; i < max; i++) {
-            OpenBrowser.displayUrl(createInfoUrl(primitivesToShow.get(i)));
+        for (int i = 0; i < max; i++) {
+            launchInfoBrowser(primitivesToShow.get(i));
+        }
+
+        if (noteToShow != null) {
+            launchInfoBrowser(noteToShow);
+        }
+    }
+
+    protected final void launchInfoBrowser(Object o) {
+        String url = createInfoUrl(o);
+        if (url != null) {
+            String result = OpenBrowser.displayUrl(url);
+            if (result != null) {
+                Main.warn(result);
+            }
         }
     }
@@ -102 +122 @@
     @Override
     public void actionPerformed(ActionEvent e) {
-        launchInfoBrowsersForSelectedPrimitives();
+        launchInfoBrowsersForSelectedPrimitivesAndNote();
     }
 

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

@@ -28 +28 @@
     @Override
     protected  String createInfoUrl(Object infoObject) {
-        OsmPrimitive primitive = (OsmPrimitive) infoObject;
-        return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId() + "/history";
+        if (infoObject instanceof OsmPrimitive) {
+            OsmPrimitive primitive = (OsmPrimitive) infoObject;
+            return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId() + "/history";
+        } else {
+            return null;
+        }
     }
 }

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();
     }
 }