From 0c1dc4d18c321421265bc9a360710747ad27ebae Mon Sep 17 00:00:00 2001 From: Karl von Randow Date: Tue, 13 Nov 2018 17:36:11 +1300 Subject: [PATCH] Fix mini-css-extract-plugin 0.4.3 issue Second attempt. Based on https://github.com/webdeveric/webpack-assets-manifest/pull/40 Resolves https://github.com/danethurber/webpack-manifest-plugin/issues/167 mini-css-extract-plugin reports additional, incorrect information for files that are refenced in CSS. The first time we see the file the `module.userRequest` is correct, and we add to `moduleAssets` correctly. However mini-css-extract-plugin then also reports the same `file` but with `module.userRequest` set to the CSS file that references it, which caused us to overwrite the good value in `moduleAssets`. See the change in mini-css-extract-plugin that caused this https://github.com/webpack-contrib/mini-css-extract-plugin/pull/177 --- lib/plugin.js | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/lib/plugin.js b/lib/plugin.js index de17947..a76f910 100644 --- a/lib/plugin.js +++ b/lib/plugin.js @@ -60,7 +60,7 @@ ManifestPlugin.prototype.apply = function(compiler) { var outputName = path.relative(outputFolder, outputFile); var moduleAsset = function (module, file) { - if (module.userRequest && !moduleAssets[file]) { + if (module.userRequest) { moduleAssets[file] = path.join( path.dirname(file), path.basename(module.userRequest) @@ -68,6 +68,20 @@ ManifestPlugin.prototype.apply = function(compiler) { } }; + var normalModuleLoader = function (loaderContext, module) { + const { emitFile } = loaderContext; + + loaderContext.emitFile = (file, content, sourceMap) => { + if (module.userRequest && !moduleAssets[file]) { + moduleAssets[file] = path.join( + path.dirname(file), + path.basename(module.userRequest) + ); + } + + return emitFile.call(module, file, content, sourceMap); + }; + }; var emit = function(compilation, compileCallback) { const emitCount = emitCountMap.get(outputFile) - 1 @@ -251,7 +265,7 @@ ManifestPlugin.prototype.apply = function(compiler) { } compiler.hooks.compilation.tap(pluginOptions, function (compilation) { - compilation.hooks.moduleAsset.tap(pluginOptions, moduleAsset); + compilation.hooks.normalModuleLoader.tap(pluginOptions, normalModuleLoader); }); compiler.hooks.emit.tap(pluginOptions, emit);