By using syntax like file://./docs/index.html, editor will open file index.html stored in folder SKETCH_FOLDER/docs/. Fixes #224

Author: Federico Fissore

app/src/processing/app/Editor.java

@@ -1006,7 +1006,7 @@ private SketchTextArea createTextArea() throws IOException {
       @Override
       public void hyperlinkUpdate(HyperlinkEvent hyperlinkEvent) {
         try {
-          platform.openURL(hyperlinkEvent.getURL().toExternalForm());
+          platform.openURL(sketch.getFolder(), hyperlinkEvent.getURL().toExternalForm());
         } catch (Exception e) {
           Base.showWarning(e.getMessage(), e.getMessage(), e);
         }

arduino-core/src/processing/app/Platform.java

@@ -29,51 +29,55 @@
 import processing.app.legacy.PConstants;
 
 import javax.swing.*;
-import java.io.*;
-import java.util.*;
+import java.io.File;
+import java.io.IOException;
+import java.util.HashMap;
+import java.util.LinkedList;
+import java.util.List;
+import java.util.Map;
 
 import static processing.app.I18n._;
 
 
 /**
  * Used by Base for platform-specific tweaking, for instance finding the
  * sketchbook location using the Windows registry, or OS X event handling.
- * 
- * The methods in this implementation are used by default, and can be 
- * overridden by a subclass, if loaded by Base.main(). 
- * 
+ * <p/>
+ * The methods in this implementation are used by default, and can be
+ * overridden by a subclass, if loaded by Base.main().
+ * <p/>
  * These methods throw vanilla-flavored Exceptions, so that error handling
- * occurs inside Base. 
- * 
- * There is currently no mechanism for adding new platforms, as the setup is 
- * not automated. We could use getProperty("os.arch") perhaps, but that's 
- * debatable (could be upper/lowercase, have spaces, etc.. basically we don't 
+ * occurs inside Base.
+ * <p/>
+ * There is currently no mechanism for adding new platforms, as the setup is
+ * not automated. We could use getProperty("os.arch") perhaps, but that's
+ * debatable (could be upper/lowercase, have spaces, etc.. basically we don't
  * know if name is proper Java package syntax.)
  */
 public class Platform {
-  
-  
+
+
   /**
    * Set the default L & F. While I enjoy the bounty of the sixteen possible
-   * exception types that this UIManager method might throw, I feel that in 
+   * exception types that this UIManager method might throw, I feel that in
    * just this one particular case, I'm being spoiled by those engineers
    * at Sun, those Masters of the Abstractionverse. It leaves me feeling sad
    * and overweight. So instead, I'll pretend that I'm not offered eleven dozen
    * ways to report to the user exactly what went wrong, and I'll bundle them
-   * all into a single catch-all "Exception". Because in the end, all I really 
+   * all into a single catch-all "Exception". Because in the end, all I really
    * care about is whether things worked or not. And even then, I don't care.
-   * 
+   *
    * @throws Exception Just like I said.
    */
   public void setLookAndFeel() throws Exception {
     UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
   }
-  
-  
+
+
   public void init() throws IOException {
   }
-  
-  
+
+
   public File getSettingsFolder() throws Exception {
     // otherwise make a .processing directory int the user's home dir
     File home = new File(System.getProperty("user.home"));
@@ -95,37 +99,46 @@ public File getSettingsFolder() throws Exception {
     }
     */
   }
-  
+
 
   /**
-   * @return null if not overridden, which will cause a prompt to show instead. 
+   * @return null if not overridden, which will cause a prompt to show instead.
    * @throws Exception
    */
   public File getDefaultSketchbookFolder() throws Exception {
     return null;
   }
-  
-  
+
+  public void openURL(File folder, String url) throws Exception {
+    if (!url.startsWith("file://./")) {
+      openURL(url);
+      return;
+    }
+
+    url = url.replaceAll("file://./", folder.getCanonicalFile().toURI().toASCIIString());
+    openURL(url);
+  }
+
   public void openURL(String url) throws Exception {
     String launcher = PreferencesData.get("launcher");
     if (launcher != null) {
-      Runtime.getRuntime().exec(new String[] { launcher, url });
+      Runtime.getRuntime().exec(new String[]{launcher, url});
     } else {
       showLauncherWarning();
-    } 
+    }
   }
 
 
   public boolean openFolderAvailable() {
     return PreferencesData.get("launcher") != null;
   }
-  
-  
+
+
   public void openFolder(File file) throws Exception {
     String launcher = PreferencesData.get("launcher");
     if (launcher != null) {
       String folder = file.getAbsolutePath();
-      Runtime.getRuntime().exec(new String[] { launcher, folder });
+      Runtime.getRuntime().exec(new String[]{launcher, folder});
     } else {
       showLauncherWarning();
     }
@@ -184,14 +197,14 @@ public String getName() {
     return PConstants.platformNames[PConstants.OTHER];
   }
 
-  
+
   // . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . .  
 
 
   protected void showLauncherWarning() {
-    BaseNoGui.showWarning(_("No launcher available"), 
-                          _("Unspecified platform, no launcher available.\nTo enable opening URLs or folders, add a \n\"launcher=/path/to/app\" line to preferences.txt"),
-                          null);
+    BaseNoGui.showWarning(_("No launcher available"),
+      _("Unspecified platform, no launcher available.\nTo enable opening URLs or folders, add a \n\"launcher=/path/to/app\" line to preferences.txt"),
+      null);
   }
 
   public List<BoardPort> filterPorts(List<BoardPort> ports, boolean aBoolean) {

arduino-core/src/processing/app/macosx/Platform.java

@@ -34,11 +34,14 @@
 import processing.app.legacy.PConstants;
 
 import java.awt.*;
-import java.io.*;
-import java.lang.reflect.Method;
+import java.io.ByteArrayOutputStream;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
 import java.net.URI;
-import java.util.*;
+import java.util.LinkedList;
 import java.util.List;
+import java.util.Map;
 
 
 /**
@@ -98,45 +101,13 @@ public File getDefaultSketchbookFolder() throws Exception {
 
 
   public void openURL(String url) throws Exception {
-    if (PApplet.javaVersion < 1.6f) {
-      if (url.startsWith("http")) {
-        // formerly com.apple.eio.FileManager.openURL(url);
-        // but due to deprecation, instead loading dynamically
-        try {
-          Class<?> eieio = Class.forName("com.apple.eio.FileManager");
-          Method openMethod =
-            eieio.getMethod("openURL", new Class[] { String.class });
-          openMethod.invoke(null, new Object[] { url });
-        } catch (Exception e) {
-          e.printStackTrace();
-        }
-      } else {
-      // Assume this is a file instead, and just open it.
-      // Extension of http://dev.processing.org/bugs/show_bug.cgi?id=1010
-      PApplet.open(url);
-      }
+    Desktop desktop = Desktop.getDesktop();
+    if (url.startsWith("http") || url.startsWith("file:")) {
+      desktop.browse(new URI(url));
     } else {
-      try {
-        Class<?> desktopClass = Class.forName("java.awt.Desktop");
-        Method getMethod = desktopClass.getMethod("getDesktop");
-        Object desktop = getMethod.invoke(null, new Object[] { });
-
-        // for Java 1.6, replacing with java.awt.Desktop.browse() 
-        // and java.awt.Desktop.open()
-        if (url.startsWith("http")) {  // browse to a location
-          Method browseMethod =
-            desktopClass.getMethod("browse", new Class[] { URI.class });
-          browseMethod.invoke(desktop, new Object[] { new URI(url) });
-        } else {  // open a file
-          Method openMethod =
-            desktopClass.getMethod("open", new Class[] { File.class });
-          openMethod.invoke(desktop, new Object[] { new File(url) });
-          }
-      } catch (Exception e) {
-        e.printStackTrace();
-        }
-      }
+      desktop.open(new File(url));
     }
+  }
 
 
   public boolean openFolderAvailable() {

arduino-core/src/processing/app/windows/Platform.java

@@ -130,7 +130,7 @@ public void openURL(String url) throws Exception {
     // "Access is denied" in both cygwin and the "dos" prompt.
     //Runtime.getRuntime().exec("cmd /c " + currentDir + "\\reference\\" +
     //                    referenceFile + ".html");
-    if (url.startsWith("http")) {
+    if (url.startsWith("http") || url.startsWith("file:")) {
       // open dos prompt, give it 'start' command, which will
       // open the url properly. start by itself won't work since
       // it appears to need cmd