Merge pull request #15195 from webpack/fix-hash-function-usage

sokra · web-flow · commit f16e7468a295 · 2022-01-18T12:50:56.000+01:00
provide hashFunction parameter to DependencyTemplates
diff --git a/lib/Compilation.js b/lib/Compilation.js
@@ -1020,7 +1020,9 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		/** @type {Map<DepConstructor, ModuleFactory>} */
 		this.dependencyFactories = new Map();
 		/** @type {DependencyTemplates} */
-		this.dependencyTemplates = new DependencyTemplates();
+		this.dependencyTemplates = new DependencyTemplates(
+			this.outputOptions.hashFunction
+		);
 		this.childrenCounters = {};
 		/** @type {Set<number|string>} */
 		this.usedChunkIds = null;
diff --git a/lib/DependencyTemplates.js b/lib/DependencyTemplates.js
@@ -57,7 +57,7 @@ class DependencyTemplates {
 	}
 
 	clone() {
-		const newInstance = new DependencyTemplates();
+		const newInstance = new DependencyTemplates(this._hashFunction);
 		newInstance._map = new Map(this._map);
 		newInstance._hash = this._hash;
 		return newInstance;

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class DependencyTemplates {`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`clone() {`
`60`		`- const newInstance = new DependencyTemplates();`
	`60`	`+ const newInstance = new DependencyTemplates(this._hashFunction);`
`61`	`61`	`newInstance._map = new Map(this._map);`
`62`	`62`	`newInstance._hash = this._hash;`
`63`	`63`	`return newInstance;`