This repository has been archived by the owner on Sep 9, 2021. It is now read-only.
Cannot read property 'context' of undefined #125
Comments
|
context: options.context || this.rootContext || this.contextshould work. PR welcome :) |
odama626
pushed a commit
to odama626/service-worker-loader
that referenced
this issue
Mar 15, 2018
timneutkens
pushed a commit
to vercel/next.js
that referenced
this issue
Aug 7, 2018
## Context
When upgrading to Next.js 6.1.1-canary.4 and using the `emit-file-loader` included in Next.js, the following error is thrown:
```bash
error in ./src/graveyard/pages/_app.scss
Module build failed (from ../node_modules/next/dist/build/webpack/loaders/emit-file-loader.js):
TypeError: Cannot read property 'context' of undefined
at Object.module.exports (~/project-root/node_modules/next/dist/build/webpack/load
ers/emit-file-loader.js:27:68)
@ ./src/pages/_app.js 35:0-53 156:17-26 157:13-22
@ multi ./pages/_app.js
```
`next.config.js` (shortened):
```js
module.exports = {
webpack: (config, { dev }) => {
config.module.rules.push({
test: /\.scss$/,
use: [
{
loader: 'emit-file-loader',
options: {
name: 'dist/[path][name].[ext].js'
}
},
{
loader: 'babel-loader',
options: {
babelrc: false,
extends: path.resolve(__dirname, './src/.babelrc')
}
},
'styled-jsx-css-loader',
{ loader: 'postcss-loader', options: { sourceMap: dev } },
{
loader: 'sass-loader',
options: {
sourceMap: dev
}
}
]
});
return config;
}
};
```
## Suggested Fix
A quick Google search brought me to a [related issue in `webpack-loader`](webpack-contrib/worker-loader#125). As pointed out in the [Webpack docs](https://webpack.js.org/api/loaders/#this-rootcontext):
> Starting with webpack 4, the formerly `this.options.context` is provided as `this.rootContext`.
This PR makes this change while maintaining backward compatibility.
## System information
Next.js: 6.1.1-canary.4
Node: v9.3.0
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
When I use webpack (version: 4.0.0-beta.2) and worker-loader (version: next), it has an error:
I found the error here: https://github.com/webpack-contrib/worker-loader/blob/next/src/index.js#L40
I found that the code changed from
to
It is no problem.
The text was updated successfully, but these errors were encountered: