feat: DeprecationWarning: Chunk.modules in Webpack 3.x

[tleunen]

Webpack 3.0-rc was released and this plugin currently doesn't support it.

I haven't tested it, it might be just a question of updating the peerDependency.

[michael-ciniawsky]

It doesn't work at all or does it just displays a warning for an unmet peerDepedency?

[gpoitch]

It's working with webpack 3.0. It does emit this warning however:

DeprecationWarning: Chunk.modules is deprecated. Use Chunk.getNumberOfModules/mapModules/forEachModule/containsModule instead.
    at /node_modules/extract-text-webpack-plugin/index.js:271:24

[MirrorBytes]

Change line 271 to this:
async.forEach(Array.from(chunk.mapModules(function(c) { return c; })), function(module, callback) {

It solved the issue for me (ran 3 tests, might need a bit more testing). The Array.from() is specifically to counter an issue I have had in the past with several linux distros not liking "array-like" structures (such as arrays built from mapping).

[MirrorBytes]

@d3viant0ne Should a put a pull request in? It's not exactly urgent, but this is the most resource independent way to avoid this warning and making it work more effectively for Webpack 3.

[michael-ciniawsky]

Yes feel free to send PR :) for discussion && finishing details