Fix webpack#5806, TypeError on ExternalModule hash.update

STRML · STRML · commit c9bad176fd29 · 2017-10-11T14:02:46.000-05:00
undefined must be coerced to `false`
diff --git a/lib/ExternalModule.js b/lib/ExternalModule.js
@@ -120,7 +120,7 @@ class ExternalModule extends Module {
 	updateHash(hash) {
 		hash.update(this.type);
 		hash.update(JSON.stringify(this.request));
-		hash.update(JSON.stringify(this.optional));
+		hash.update(JSON.stringify(Boolean(this.optional)));
 		super.updateHash(hash);
 	}
 }
diff --git a/test/ExternalModule.test.js b/test/ExternalModule.test.js
@@ -340,4 +340,32 @@ module.exports = some/request;`;
 			hashedText.should.containEql("12345678");
 		});
 	});
+
+	describe("#updateHash without optional", function() {
+		let hashedText;
+		let hash;
+		beforeEach(function() {
+			hashedText = "";
+			hash = {
+				update: (text) => {
+					hashedText += text;
+				}
+			};
+			// Note no set of `externalModule.optional`, which crashed externals in 3.7.0
+			externalModule.id = 12345678;
+			externalModule.updateHash(hash);
+		});
+		it("updates hash with request", function() {
+			hashedText.should.containEql("some/request");
+		});
+		it("updates hash with type", function() {
+			hashedText.should.containEql("some-type");
+		});
+		it("updates hash with optional flag", function() {
+			hashedText.should.containEql("false");
+		});
+		it("updates hash with module id", function() {
+			hashedText.should.containEql("12345678");
+		});
+	});
 });

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ class ExternalModule extends Module {`
`120`	`120`	`updateHash(hash) {`
`121`	`121`	`hash.update(this.type);`
`122`	`122`	`hash.update(JSON.stringify(this.request));`
`123`		`- hash.update(JSON.stringify(this.optional));`
	`123`	`+ hash.update(JSON.stringify(Boolean(this.optional)));`
`124`	`124`	`super.updateHash(hash);`
`125`	`125`	`}`
`126`	`126`	`}`