Changeset 30372 in osm for applications/editors/josm/plugins/pointInfo/src

Timestamp:

2014-03-26T20:59:59+01:00 (11 years ago)

Author:

mkyral

Message:

PointInfo: Replace openWebPage function by josm.tools.OpenBrowser.

File:

• applications/editors/josm/plugins/pointInfo/src/org/openstreetmap/josm/plugins/pointinfo/PointInfoAction.java (modified) (5 diffs)

applications/editors/josm/plugins/pointInfo/src/org/openstreetmap/josm/plugins/pointinfo/PointInfoAction.java

@@ -22 +22 @@
 import java.awt.Cursor;
 import java.awt.Point;
-import java.awt.Desktop;
 import java.awt.event.ActionEvent;
 import java.awt.event.InputEvent;
@@ -39 +38 @@
 import javax.swing.event.HyperlinkEvent;
 import javax.swing.event.HyperlinkListener;
-import java.net.URI;
-import java.net.URISyntaxException;
 
 import org.openstreetmap.josm.Main;
@@ -55 +52 @@
 import org.openstreetmap.josm.tools.ImageProvider;
 import org.openstreetmap.josm.tools.Shortcut;
+import org.openstreetmap.josm.tools.OpenBrowser;
 import org.xml.sax.SAXException;
 
@@ -91 +89 @@
     private static Cursor getCursor() {
         return ImageProvider.getCursor("crosshair", "info-sml");
-    }
-
-    private static void openWebpage(URI uri) {
-        Desktop desktop = Desktop.isDesktopSupported() ? Desktop.getDesktop() : null;
-        if (desktop != null && desktop.isSupported(Desktop.Action.BROWSE)) {
-            try {
-                desktop.browse(uri);
-            } catch (Exception e) {
-                e.printStackTrace();
-            }
-        }
     }
 
@@ -137 +124 @@
                             mRuian.performAction(hle.getURL().toString());
                           } else {
-                            try {
-                                openWebpage(hle.getURL().toURI());
-                            } catch (URISyntaxException e) {
-                                e.printStackTrace();
+                            String ret = OpenBrowser.displayUrl(hle.getURL().toString());
+                            if (ret != null) {
+                              PointInfoUtils.showNotification(ret, "error");
                             }
                           }