Changeset 6682 in josm for trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

Timestamp:

2014-01-13T19:29:33+01:00 (10 years ago)

Author:

simon04

Message:

fix #5629 #5108 #9568 - Make Unix web browsers configurable via browser.unix property

Default is ["xdg-open", "#DESKTOP#", "$BROWSER", "gnome-open", "kfmclient openURL", "firefox"].

File:

• trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
+import java.awt.Desktop;
 import java.awt.Dimension;
 import java.awt.GraphicsEnvironment;
@@ -12 +13 @@
 import java.io.IOException;
 import java.io.InputStreamReader;
+import java.net.URI;
+import java.net.URISyntaxException;
 import java.util.Arrays;
 
@@ -41 +44 @@
     @Override
     public void openUrl(String url) throws IOException {
-        String[] programs = {"gnome-open", "kfmclient openURL", "firefox"};
-        for (String program : programs) {
+        for (String program : Main.pref.getCollection("browser.unix",
+                Arrays.asList("xdg-open", "#DESKTOP#", "$BROWSER", "gnome-open", "kfmclient openURL", "firefox"))) {
             try {
-                Runtime.getRuntime().exec(program+" "+url);
+                if ("#DESKTOP#".equals(program)) {
+                    Desktop.getDesktop().browse(new URI(url));
+                } else if (program.startsWith("$")) {
+                    program = System.getenv().get(program.substring(1));
+                    Runtime.getRuntime().exec(new String[]{program, url});
+                } else {
+                    Runtime.getRuntime().exec(new String[]{program, url});
+                }
                 return;
             } catch (IOException e) {
+                Main.warn(e);
+            } catch (URISyntaxException e) {
                 Main.warn(e);
             }