Merge pull request #13431 from Shinyaigeek/fix/overwritten_webpack__e…

…xports_onChunksLoaded Fix: broken library output by webpackPrefetch
webpack · Jul 16, 2021 · e15752c · e15752c
2 parents e5570ab + 3967571
commit e15752c
Show file tree

Hide file tree

Showing 8 changed files with 234 additions and 198 deletions.
diff --git a/lib/javascript/JavascriptModulesPlugin.js b/lib/javascript/JavascriptModulesPlugin.js
@@ -834,7 +834,9 @@ class JavascriptModulesPlugin {
 				}
 			}
 			if (runtimeRequirements.has(RuntimeGlobals.onChunksLoaded)) {
-				startupSource.add(`${RuntimeGlobals.onChunksLoaded}();\n`);
+				startupSource.add(
+					`__webpack_exports__ = ${RuntimeGlobals.onChunksLoaded}(__webpack_exports__);\n`
+				);
 			}
 			source.add(
 				hooks.renderStartup.call(startupSource, lastInlinedModule, {

diff --git a/lib/prefetch/ChunkPrefetchStartupRuntimeModule.js b/lib/prefetch/ChunkPrefetchStartupRuntimeModule.js
@@ -32,16 +32,17 @@ class ChunkPrefetchStartupRuntimeModule extends RuntimeModule {
 					`${RuntimeGlobals.onChunksLoaded}(0, ${JSON.stringify(
 						// This need to include itself to delay execution after this chunk has been fully loaded
 						onChunks.filter(c => c === chunk).map(c => c.id)
-					)}, ${runtimeTemplate.expressionFunction(
+					)}, ${runtimeTemplate.basicFunction(
+						"",
 						chunks.size < 3
 							? Array.from(
 									chunks,
 									c =>
-										`${RuntimeGlobals.prefetchChunk}(${JSON.stringify(c.id)})`
-							  ).join(", ")
+										`${RuntimeGlobals.prefetchChunk}(${JSON.stringify(c.id)});`
+							  )
 							: `${JSON.stringify(Array.from(chunks, c => c.id))}.map(${
 									RuntimeGlobals.prefetchChunk
-							  })`
+							  });`
 					)}, 5);`
 			)
 		);

diff --git a/lib/runtime/OnChunksLoadedRuntimeModule.js b/lib/runtime/OnChunksLoadedRuntimeModule.js
@@ -58,7 +58,11 @@ class OnChunksLoadedRuntimeModule extends RuntimeModule {
 						]),
 						"}",
 						"if(fulfilled) {",
-						Template.indent(["deferred.splice(i--, 1)", "result = fn();"]),
+						Template.indent([
+							"deferred.splice(i--, 1)",
+							"var r = fn();",
+							"if (r !== undefined) result = r;"
+						]),
 						"}"
 					]),
 					"}",

diff --git a/test/__snapshots__/StatsTestCases.basictest.js.snap b/test/__snapshots__/StatsTestCases.basictest.js.snap
diff --git a/test/configCases/issues/issue-12993/dynamic.js b/test/configCases/issues/issue-12993/dynamic.js
@@ -0,0 +1 @@
+export default "dynamic";
diff --git a/test/configCases/issues/issue-12993/index.js b/test/configCases/issues/issue-12993/index.js
@@ -0,0 +1,6 @@
+export const main = "main";
+
+it("library output should be accurate value", async () => {
+	expect(global.lib).toEqual(nsObj({ main: "main" }));
+	await import(/* webpackPrefetch: true */ "./dynamic.js");
+});
diff --git a/test/configCases/issues/issue-12993/test.config.js b/test/configCases/issues/issue-12993/test.config.js
@@ -0,0 +1,5 @@
+module.exports = {
+	afterExecute() {
+		delete global.lib;
+	}
+};
diff --git a/test/configCases/issues/issue-12993/webpack.config.js b/test/configCases/issues/issue-12993/webpack.config.js
@@ -0,0 +1,17 @@
+module.exports = [
+	{
+		mode: "development",
+		output: {
+			library: "lib",
+			libraryTarget: "global"
+		}
+	},
+	{
+		mode: "development",
+		devtool: false,
+		output: {
+			library: "lib",
+			libraryTarget: "global"
+		}
+	}
+];