Merge pull request #1094 from ethereum/fixImportGithub

src/app.js

@@ -217,6 +217,21 @@ This instance of Remix you are visiting WILL NOT BE UPDATED.\n
 Please make a backup of your contracts and start using http://remix.ethereum.org`)
   }
 
+  function importExternal (url, cb) {
+    handleImports.import(url,
+      (loadingMsg) => {
+        toolTip(loadingMsg)
+      },
+      (error, content, cleanUrl, type, url) => {
+        if (!error) {
+          filesProviders[type].addReadOnly(cleanUrl, content, url)
+          cb(null, content)
+        } else {
+          cb(error)
+        }
+      })
+  }
+
   // ----------------- Compiler -----------------
   var compiler = new Compiler((url, cb) => {
     var provider = fileManager.fileProviderOf(url)
@@ -226,22 +241,11 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
         if (exist) {
           return provider.get(url, cb)
         } else {
-          return cb('Unable to import "' + url + '": File not found')
+          importExternal(url, cb)
         }
       })
     } else {
-      handleImports.import(url,
-      (loadingMsg) => {
-        $('#output').append($('<div/>').append($('<pre/>').text(loadingMsg)))
-      },
-      (error, content, cleanUrl, type, url) => {
-        if (!error) {
-          filesProviders[type].addReadOnly(cleanUrl, content, url)
-          cb(null, content)
-        } else {
-          cb(error)
-        }
-      })
+      importExternal(url, cb)
     }
   })
   var offsetToLineColumnConverter = new OffsetToLineColumnConverter(compiler.event)

src/app/compiler/compiler-imports.js

@@ -4,6 +4,7 @@ var swarmgw = require('swarmgw')
 var request = require('request')
 
 module.exports = {
+  previouslyHandled: {}, // cache import so we don't make the request at each compilation.
   handleGithubCall: function (root, path, cb) {
     return request.get(
       {
@@ -61,6 +62,11 @@ module.exports = {
   },
 
   import: function (url, loadingCb, cb) {
+    var self = this
+    var imported = this.previouslyHandled[url]
+    if (imported) {
+      return cb(null, imported.content, imported.cleanUrl, imported.type, url)
+    }
     var handlers = this.handlers()
 
     var found = false
@@ -79,7 +85,11 @@ module.exports = {
             cb('Unable to import "' + cleanUrl + '": ' + err)
             return
           }
-
+          self.previouslyHandled[url] = {
+            content: content,
+            cleanUrl: cleanUrl,
+            type: handler.type
+          }
           cb(null, content, cleanUrl, handler.type, url)
         })
       }