Merge pull request #5231 from webpack/bugfix/concat-hash

add hash method to concatenated module
webpack · Jul 7, 2017 · ba610cc · ba610cc
2 parents 3dc0d4e + cd7ec76
commit ba610cc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/optimize/ConcatenatedModule.js b/lib/optimize/ConcatenatedModule.js
@@ -451,6 +451,13 @@ class ConcatenatedModule extends Module {
 		return name;
 	}
 
+	updateHash(hash) {
+		for(const m of this.modules) {
+			m.updateHash(hash);
+		}
+		super.updateHash(hash);
+	}
+
 }
 
 class HarmonyImportSpecifierDependencyConcatenatedTemplate {

diff --git a/test/statsCases/scope-hoisting-multi/expected.txt b/test/statsCases/scope-hoisting-multi/expected.txt
@@ -1,4 +1,4 @@
-Hash: 731069e082cf620521ce1e4286719458d8076cfc
+Hash: 731069e082cf620521ce637b388d1abe7f9228e5
 Child
     Hash: 731069e082cf620521ce
     Time: Xms
@@ -14,7 +14,7 @@ Child
        [9] (webpack)/test/statsCases/scope-hoisting-multi/second.js 177 bytes {4} [built]
       [10] (webpack)/test/statsCases/scope-hoisting-multi/lazy_second.js 55 bytes {1} [built]
 Child
-    Hash: 1e4286719458d8076cfc
+    Hash: 637b388d1abe7f9228e5
     Time: Xms
        [0] (webpack)/test/statsCases/scope-hoisting-multi/common_lazy_shared.js 25 bytes {0} {1} {2} [built]
        [1] (webpack)/test/statsCases/scope-hoisting-multi/vendor.js 25 bytes {5} [built]