Changeset 2282 in osm

Timestamp:

2007-03-19T12:20:44+01:00 (18 years ago)

Author:

frsantos

Message:

Shows a more friendly error if cannot start browser

File:

• utils/josm/plugins/ywms/src/org/openstreetmap/josm/plugins/ywms/GeckoSupport.java (modified) (3 diffs)

utils/josm/plugins/ywms/src/org/openstreetmap/josm/plugins/ywms/GeckoSupport.java

@@ -1 +1 @@
 package org.openstreetmap.josm.plugins.ywms;
+
+import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.io.IOException;
@@ -35 +37 @@
                 builder.environment().put("MOZ_NO_REMOTE", "1");
                 
-                return builder.start();
+        return startProcess(builder);
         }
+
+    /**
+     * Starts the process
+     * 
+     * @param builder The builder for the process
+     * @return The started process
+     * @throws IOException If coulnd't start the process
+     */
+    private static Process startProcess(ProcessBuilder builder) throws IOException
+    {
+        try
+        {
+            return builder.start();
+        }
+        catch(IOException ioe)
+        {
+            throw new IOException( tr("Could not start browser. Please check that the executable path is correct.")); 
+        }
+    }
         
         /**
@@ -70 +91 @@
                         environment.put("MOZ_FORCE_PAINT_AFTER_ONLOAD", System.getProperty("java.io.tmpdir") + "/ywms");
                 
-                return builder.start();
+                return startProcess(builder);
         }