Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Commit 28e3360

Browse files
committed
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
1 parent 10721f5 commit 28e3360

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/index.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ class ExtractTextPlugin {
6464
}, this);
6565
} else if (checkedChunks.indexOf(chunk) < 0) {
6666
checkedChunks.push(chunk);
67-
chunk.modules.slice().forEach((module) => {
67+
chunk.forEachModule((module) => {
6868
intoChunk.addModule(module);
6969
module.addChunk(intoChunk);
7070
});
@@ -77,7 +77,7 @@ class ExtractTextPlugin {
7777

7878
renderExtractedChunk(chunk) {
7979
const source = new ConcatSource();
80-
chunk.modules.forEach((module) => {
80+
chunk.forEachModule((module) => {
8181
const moduleSource = module.source();
8282
source.add(this.applyAdditionalInformation(moduleSource, module.additionalInformation));
8383
}, this);
@@ -144,7 +144,7 @@ class ExtractTextPlugin {
144144
async.forEach(chunks, (chunk, callback) => { // eslint-disable-line no-shadow
145145
const extractedChunk = extractedChunks[chunks.indexOf(chunk)];
146146
const shouldExtract = !!(options.allChunks || isInitialOrHasNoParents(chunk));
147-
async.forEach(chunk.mapModules((c) => { return c; }), (module, callback) => { // eslint-disable-line no-shadow, arrow-body-style
147+
async.forEach(chunk.mapModules(c => c), (module, callback) => { // eslint-disable-line no-shadow
148148
let meta = module[NS];
149149
if (meta && (!meta.options.id || meta.options.id === id)) {
150150
const wasExtracted = Array.isArray(meta.content);
@@ -181,7 +181,7 @@ class ExtractTextPlugin {
181181
}, this);
182182
extractedChunks.forEach((extractedChunk) => {
183183
if (!isInitialOrHasNoParents(extractedChunk)) {
184-
extractedChunk.modules.slice().forEach((module) => {
184+
extractedChunk.forEachModule((module) => {
185185
extractedChunk.removeModule(module);
186186
});
187187
}
@@ -192,7 +192,7 @@ class ExtractTextPlugin {
192192
});
193193
compilation.plugin('additional-assets', (callback) => {
194194
extractedChunks.forEach((extractedChunk) => {
195-
if (extractedChunk.modules.length) {
195+
if (extractedChunk.getNumberOfModules()) {
196196
extractedChunk.modules.sort((a, b) => {
197197
if (!options.ignoreOrder && isInvalidOrder(a, b)) {
198198
compilation.errors.push(new OrderUndefinedError(a.getOriginalModule()));

0 commit comments

Comments
 (0)