track dependencies even if error occur

sokra · sokra · commit 4a52440ea804 · 2016-02-03T23:46:51.000+01:00
fixes webpack#1975 merged webpack#1985
diff --git a/lib/NormalModule.js b/lib/NormalModule.js
@@ -128,6 +128,11 @@ NormalModule.prototype.doBuild = function doBuild(options, compilation, resolver
 		context: loaderContext,
 		readResource: fs.readFile.bind(fs)
 	}, function(err, result) {
+		if(result) {
+			module.cacheable = result.cacheable;
+			module.fileDependencies = result.fileDependencies;
+			module.contextDependencies = result.contextDependencies;
+		}
 		if(err) {
 			module.error = err;
 			return callback(new ModuleBuildError(module, err));
@@ -136,9 +141,6 @@ NormalModule.prototype.doBuild = function doBuild(options, compilation, resolver
 		var resourceBuffer = result.resourceBuffer;
 		var source = result.result[0];
 		var sourceMap = result.result[1];
-		module.cacheable = result.cacheable;
-		module.fileDependencies = result.fileDependencies;
-		module.contextDependencies = result.contextDependencies;
 
 		if(!Buffer.isBuffer(source) && typeof source !== "string") {
 			module.error = new Error("Final loader didn't return a Buffer or String");
@@ -156,7 +158,7 @@ NormalModule.prototype.doBuild = function doBuild(options, compilation, resolver
 			module._source = new RawSource(source);
 		}
 		return callback();
-	})
+	});
 };
 
 NormalModule.prototype.disconnect = function disconnect() {
