[webpack v4.0.0-alpha.0] this.context => this.rootContext (Breaking Change)

[drwpow]

(from webpack#6064)

Running webpack with 4.0.0-alpha.0 results in the following error, one per file:

ERROR in ./assets/images/icons/search.svg
Module build failed: TypeError: Cannot read property 'context' of undefined
    at Object.loader (/Users/drew/Sites/my-site/node_modules/file-loader/dist/index.js:34:49)
 @ ./styles.sass 6:40863-40906

[michael-ciniawsky]

Could you please try the suggestion in webpack/webpack#6064 (comment) using rootContextinstead of context and provide the results/errors here ?

[drwpow]

@michael-ciniawsky Not sure if it’s what you asked, but I tried changing context: to rootContext: in the webpack config but that obviously resulted in a configuration error.

So then I assumed you meant file-loader, and I went to index.js:34 and made the following change:

-  const context = options.context || this.options.context;
+  const context = options.context;

And it successfully built and emitted files.

At least in my instance, this.options wasn’t defined, and options was just an empty object ({}).

[michael-ciniawsky]

Not sure if it’s what you asked, but I tried changing context: to rootContext: in the webpack config but that obviously resulted in a configuration error.

You mean aValidationError in particular here ? :) That's to be expected then, but the change in question (this.options.context => this.rootContext) needs to be made elsewhere. Could you try

- const context = options.context || this.options.context;
+ const context = options.context || this.rootContext;

[drwpow]

@michael-ciniawsky Yup! That works for me—this.rootContext correctly returns the context. @TheLarkInn’s change in #234 seems to work for me.

[michael-ciniawsky]

@DangoDev Thx for testing that out :)

[Wisedemic]

- var context = options.context || this.options.context;
+ const context = options.context || this.rootContext;

This error is still happening! The above code fixed it for me.