eslint-plugin-reblend
Version:
Reblend specific linting rules for ESLint
64 lines (49 loc) ⢠1.52 kB
Markdown
# Disallow direct mutation of this.state (`reblend/no-direct-mutation-state`)
š¼ This rule is enabled in the āļø `recommended` [config](https://github.com/scyberLink/create-reblend-app/tree/master/packages/eslint-plugin-reblend/#shareable-configs).
<!-- end auto-generated rule header -->
NEVER mutate `this.state` directly, as calling `setState()` afterwards may replace
the mutation you made. Treat `this.state` as if it were immutable.
The only place that's acceptable to assign this.state is in a ES6 `class` component constructor.
## Rule Details
This rule is aimed to forbid the use of mutating `this.state` directly.
Examples of **incorrect** code for this rule:
```jsx
var Hello = createReblendClass({
componentDidMount: function () {
this.state.name = this.props.name.toUpperCase();
},
render: function () {
return <div>Hello {this.state.name}</div>;
},
});
class Hello extends Reblend.Component {
constructor(props) {
super(props);
// Assign at instance creation time, not on a callback
doSomethingAsync(() => {
this.state = 'bad';
});
}
}
```
Examples of **correct** code for this rule:
```jsx
var Hello = createReblendClass({
componentDidMount: function() {
this.setState({
name: this.props.name.toUpperCase();
});
},
render: function() {
return <div>Hello {this.state.name}</div>;
}
});
class Hello extends Reblend.Component {
constructor(props) {
super(props)
this.state = {
foo: 'bar',
}
}
}
```