Merge pull request #37 from mpvue/fix-empty-script

fix: #562 添加默认 script 标签

Author: anchengjian

lib/loader.js

@@ -24,7 +24,7 @@ var hasBuble = !!tryRequire('buble-loader')
 
 // for mp js
 var { compileMP, compileMPScript } = require('./mp-compiler')
-var { defaultStylePart } = require('./mp-compiler/util')
+var { defaultPart } = require('./mp-compiler/util')
 
 var rewriterInjectRE = /\b(css(?:-loader)?(?:\?[^!]+)?)(?:!|$)/
 
@@ -103,7 +103,11 @@ module.exports = function (content) {
 
   // fix #153: 根组件没有 style 模块，不生成页面的 wxss，补齐内容方便加载 vendor.wxss
   if (!parts.styles.length) {
-    parts.styles.push(defaultStylePart)
+    parts.styles.push(defaultPart('style'))
+  }
+  // fix #562: 组件没有 script 模块，会阻塞编译
+  if (!parts.script) {
+    parts.script = defaultPart('script')
   }
 
   var hasScoped = parts.styles.some(function (s) { return s.scoped })

lib/mp-compiler/util.js

@@ -121,23 +121,24 @@ function getBabelrc (src) {
   return ''
 }
 
-const defaultStylePart = {
-  type: 'style',
-  content: '\n',
-  start: 0,
-  attrs: {},
-  end: 1,
-  map: {
-    version: 3,
-    sources: [],
-    names: [],
-    mappings: '',
-    sourcesContent: []
+const defaultPart = type => {
+  return {
+    type,
+    content: '\n',
+    start: 0,
+    attrs: {},
+    end: 1,
+    map: {
+      version: 3,
+      sources: [],
+      names: [],
+      mappings: '',
+      sourcesContent: []
   }
 }
 
 module.exports = {
-  defaultStylePart,
+  defaultPart,
   cacheFileInfo,
   getFileInfo,
   getCompNameAndSrc,

lib/selector.js

@@ -5,7 +5,7 @@
 var path = require('path')
 var parse = require('./parser')
 var loaderUtils = require('loader-utils')
-var { defaultStylePart } = require('./mp-compiler/util')
+var { defaultPart } = require('./mp-compiler/util')
 
 module.exports = function (content) {
   this.cacheable()
@@ -17,6 +17,6 @@ module.exports = function (content) {
   if (Array.isArray(part)) {
     part = part[query.index]
   }
-  part = part || defaultStylePart
+  part = part || defaultPart('style')
   this.callback(null, part.content, part.map)
 }