Merge pull request #98 from naumovs/master

Support "src" attribute for script tag

Author: yyx990803

lib/loader.js

@@ -153,10 +153,15 @@ module.exports = function (content) {
     })
 
     // add require for script
+    var script
     if (parts.script.length) {
+      script = parts.script[0]
       output +=
-        'module.exports = ' + getRequire('script', parts.script[0], 0) + '\n' +
-        'if (module.exports.__esModule) module.exports = module.exports.default\n'
+        'module.exports = ' + (
+          script.src
+            ? getRequireForImport('script', script, 0)
+            : getRequire('script', script, 0)
+        ) + '\nif (module.exports.__esModule) module.exports = module.exports.default\n'
     }
 
     // add require for template

lib/parser.js

@@ -32,11 +32,6 @@ module.exports = function (content) {
 
     // handle src imports
     if (src) {
-      if (type !== 'style' && type !== 'template') {
-        return cb(new Error(
-          '[vue-loader] src import is only supported for <template> and <style> tags.'
-        ))
-      }
       if (type === 'style') {
         output.styleImports.push({
           src: src,
@@ -48,6 +43,11 @@ module.exports = function (content) {
           src: src,
           lang: lang
         })
+      } else if (type === 'script') {
+        output.script.push({
+          src: src,
+          lang: lang
+        })
       }
       return
     }

test/fixtures/script-import-entry.js

@@ -0,0 +1 @@
+window.testModule = require('./script-import.vue')

test/fixtures/script-import.js

@@ -0,0 +1,7 @@
+export default {
+  data () {
+    return {
+      msg: 'Hello from Component A!'
+    }
+  }
+};

test/fixtures/script-import.vue

@@ -0,0 +1 @@
+<script src="./script-import.js"></script>

test/test.js

@@ -130,6 +130,16 @@ describe('vue-loader', function () {
     })
   })
 
+  it('script import', function (done) {
+    test({
+      entry: './test/fixtures/script-import-entry.js'
+    }, function (window) {
+      var module = window.testModule
+      expect(module.data().msg).to.contain('Hello from Component A!')
+      done()
+    })
+  })
+
   it('source map', function (done) {
     var config = Object.assign({}, globalConfig, {
       entry: './test/fixtures/basic.js',