From 1ec14f21ee89e66eec6e44cb071ee9eeb968c645 Mon Sep 17 00:00:00 2001 From: Joshua Wiens Date: Sat, 24 Jun 2017 01:04:42 -0500 Subject: [PATCH] refactor: Replace usage of chunk.modules BREAKING CHANGE: Updates to `Chunk.mapModules | forEachModule | getNumberOfModules`. This release is not backwards compatible with `Webpack 2.x` due to breaking changes in webpack/webpack#4764 --- src/index.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/index.js b/src/index.js index 161f67bd..02f061de 100644 --- a/src/index.js +++ b/src/index.js @@ -64,7 +64,7 @@ class ExtractTextPlugin { }, this); } else if (checkedChunks.indexOf(chunk) < 0) { checkedChunks.push(chunk); - chunk.modules.slice().forEach((module) => { + chunk.forEachModule((module) => { intoChunk.addModule(module); module.addChunk(intoChunk); }); @@ -77,7 +77,7 @@ class ExtractTextPlugin { renderExtractedChunk(chunk) { const source = new ConcatSource(); - chunk.modules.forEach((module) => { + chunk.forEachModule((module) => { const moduleSource = module.source(); source.add(this.applyAdditionalInformation(moduleSource, module.additionalInformation)); }, this); @@ -145,7 +145,7 @@ class ExtractTextPlugin { const extractedChunk = extractedChunks[chunks.indexOf(chunk)]; const shouldExtract = !!(options.allChunks || isInitialOrHasNoParents(chunk)); chunk.sortModules(); - async.forEach(chunk.mapModules((c) => { return c; }), (module, callback) => { // eslint-disable-line no-shadow, arrow-body-style + async.forEach(chunk.mapModules(c => c), (module, callback) => { // eslint-disable-line no-shadow let meta = module[NS]; if (meta && (!meta.options.id || meta.options.id === id)) { const wasExtracted = Array.isArray(meta.content); @@ -182,7 +182,7 @@ class ExtractTextPlugin { }, this); extractedChunks.forEach((extractedChunk) => { if (!isInitialOrHasNoParents(extractedChunk)) { - extractedChunk.modules.slice().forEach((module) => { + extractedChunk.forEachModule((module) => { extractedChunk.removeModule(module); }); } @@ -193,7 +193,7 @@ class ExtractTextPlugin { }); compilation.plugin('additional-assets', (callback) => { extractedChunks.forEach((extractedChunk) => { - if (extractedChunk.modules.length) { + if (extractedChunk.getNumberOfModules()) { extractedChunk.modules.sort((a, b) => { if (!options.ignoreOrder && isInvalidOrder(a, b)) { compilation.errors.push(new OrderUndefinedError(a.getOriginalModule()));