bug fix

Author: haledeng

lib/page.js

@@ -92,18 +92,18 @@ _.assign(module.exports.prototype, {
         // 生成resourceMap
         resourceMap = self.generateSourceMap(analysis);
 
-        // 插入resourceMap
+        // // 插入resourceMap
         content = self.injectResourceMap(content, resourceMap);
 
-        // process loadUrl
-        if (conf.autoPack) {
-            content = self.processLoadUrl(content, analysis.depMap);
-        }
+        // // process loadUrl
+        // if (conf.autoPack) {
+        //     content = self.processLoadUrl(content, analysis.depMap);
+        // }
 
-        if (conf.autoPack) {
-            content = self.removePlaceholder(content);
-            content = self.mergeInlineAssets(content);
-        }
+        // if (conf.autoPack) {
+        //     content = self.removePlaceholder(content);
+        //     content = self.mergeInlineAssets(content);
+        // }
 
         file.setContent(content);
 
@@ -413,7 +413,7 @@ _.assign(module.exports.prototype, {
     },
     processLoadUrl: function(content, depMap) {
         var self = this;
-        var reg = /require\.loadUrl\s*\(\s*("(?:[^\\"\r\n\f]|\\[\s\S])*"|'(?:[^\\'\n\r\f]|\\[\s\S])*'|\[[\s\S]*?\])\s*,\s*function\(([\s\S]*?)\)\s*{\s*/g;
+        var reg = /require\.loadUrl\s*\(\s*("(?:[^\\"\r\n\f]|\\[\s\S])*"|'(?:[^\\'\n\r\f]|\\[\s\S])*'|\[[\s\S]*?\])\s*,\s*function\s*\(([\s\S]*?)\)\s*{\s*/g;
 
         content = content.replace(reg, function(m, value, arg) {
             var hasBrackets = false;
@@ -423,16 +423,16 @@ _.assign(module.exports.prototype, {
             var info;
             var ret;
 
+
             value = value.trim().replace(/(^\[|\]$)/g, function(m, v) {
                 if (v) hasBrackets = true;
                 return '';
             });
             values = value.split(/\s*,\s*/);
             args = arg.split(/\s*,\s*/);
+
             for (var i = 0, l = values.length; i < l; ++i) {
                 info = fis.project.lookup(values[i], self.file);
-
-                // console.log('processLoadUrl', info.id, self.ret.map.pkg[info.id + '.min']);
                 if (!self.ret.map.pkg[info.id + '.min']) {
                     console.log('[ERROR] there has not the pkg:', info.id);
                     return m;
@@ -725,11 +725,9 @@ _.assign(module.exports.prototype, {
             ret = this.ret,
             html,
             subpath,
-            mainScriptFile,
-            oldUrl, newUrl;
+            mainScriptFile;
 
-        // console.log(conf);
-        if (conf.outputResourceMapToMainJs) {
+        if (conf.autoPack && conf.outputResourceMapToMainJs) {
             if (!this.mainScriptId) {
                 this.getMainJs();
             }
@@ -738,42 +736,7 @@ _.assign(module.exports.prototype, {
                 if ((mainScriptFile = this.ret.pkg[subpath])) {
                     mainScriptFile.setContent('require.resourceMap(' + JSON.stringify(resourceMap, null, this.conf.autoPack ? null : 4) + ')' + LINE_BREAK + mainScriptFile.getContent());
                 }
-
-                // //////////////////// hack fis.file mark ////////////////////////////////////////////
-                //
-                // 因为md5固定在了第一次获取的文件内容（详见file.getHash）
-                // 所以这里出现了死循环：
-                // 生成resourceMap的时候需要获取打包文件的url，这个时候md5就会固化
-                // 现在又要把resourceMap放到打包文件里面，内容改变了，但是md5不会变！
-                // 因此这里需要对md5进行一次hack
-                // by lqlongli on 2015/11/27
-                // oldUrl = mainScriptFile.getUrl();
-                // mainScriptFile.useHash = true;
-                // newUrl = mainScriptFile.getUrl();
-                // // console.log('>>>', oldUrl, newUrl);
-
-                // // restore 只要使用过这个file.getUrl的地方都需要替换【重要】
-                // // html文件替换掉url
-                // this.file.setContent(this.file.getContent().replace(new RegExp(fis.util.escapeReg(oldUrl), 'g'), newUrl));
-                // // mainScript文件里面替换掉url
-                // mainScriptFile.setContent(mainScriptFile.getContent().replace(new RegExp(fis.util.escapeReg(oldUrl), 'g'), newUrl));
-
-
-                // oldUrl = mainScriptFile.getUrl();
-                // mainScriptFile.useHash = true;
-                // newUrl = mainScriptFile.getUrl();
-
-                // // restore 只要使用过这个file.getUrl的地方都需要替换【重要】
-                // // html文件替换掉url
-                // content = content.replace(new RegExp(fis.util.escapeReg(oldUrl), 'g'), newUrl);
-                // // mainScript文件里面替换掉url
-                // mainScriptFile.setContent(mainScriptFile.getContent().replace(new RegExp(fis.util.escapeReg(oldUrl), 'g'), newUrl));
-                // // map表替换掉url
-                // this.ret.map.pkg[this.mainScriptId + '.min'].uri = newUrl;
-                // //////////////////// hack fis.file mark ////////////////////////////////////////////
             }
-
-            // console.log(this.file.extras.loadUrls);
             (this.file.extras.loadUrls || []).forEach(function(id) {
                 var subpath = conf.output.replace('${id}', id),
                     _file = fis.file(fis.project.getProjectPath(), subpath);
@@ -788,7 +751,7 @@ _.assign(module.exports.prototype, {
                     type: 'js'
                 };
                 // 替换md5
-                file.setContent(file.getContent().replace(new RegExp(fis.util.escapeReg(beforeMd5Url), 'g'), file.getUrl()))
+                file.setContent(file.getContent().replace(new RegExp(fis.util.escapeReg(beforeMd5Url), 'g'), file.getUrl()));
             });
         } else {
             html = this.modJsCodeGen(resourceMap);