Merge pull request webpack#470 from markmarijnissen/hmr-accept-multiple-deps-bugfix

sokra · sokra · commit 4891225efc03 · 2014-09-16T20:11:42.000+02:00
bugfix HMR module.accept: resolve multiple deps to module ids
diff --git a/lib/HotModuleReplacementPlugin.js b/lib/HotModuleReplacementPlugin.js
@@ -175,12 +175,19 @@ HotModuleReplacementPlugin.prototype.apply = function(compiler) {
 	compiler.parser.plugin("call module.hot.accept", function(expr) {
 		if(!this.state.compilation.hotUpdateChunkTemplate) return false;
 		if(expr.arguments.length > 1) {
-			var param = this.evaluateExpression(expr.arguments[0]);
-			if(param.isString()) {
+			var arg = this.evaluateExpression(expr.arguments[0]);
+			var params = [];
+			if(arg.isString()) {
+				params = [arg];
+			}
+			if(arg.isArray()){
+				params = arg.items;
+			}
+			params.forEach(function(param){
 				var dep = new ModuleHotAcceptDependency(param.string, param.range);
 				dep.optional = true;
 				this.state.module.addDependency(dep);
-			}
+			}.bind(this));
 		}
 	});
 	compiler.parser.plugin("call module.hot.decline", function(expr) {
