New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rule react/destructruring-assignement in a constructor #1728
Comments
I’m confused why you’d mutate the object after assigning it (it’s much slower and messier), but yes, this is a bug - specifically, it shouldn’t warn on LHS member chains off of state or props, only RHS. |
I need to condition which attribute I set in the state. |
@leohxj in that case, it’s correct. Do |
ok, but why "don’t do things in class property initializers that depend on props or state" ? |
@leohxj of course it will work; the reason not to do it is because it makes things much less clear when reading the code. |
This Pull Request updates dependency [eslint-plugin-react](https://github.com/yannickcr/eslint-plugin-react) from `v7.9.1` to `v7.10.0` <details> <summary>Release Notes</summary> ### [`v7.10.0`](https://github.com/yannickcr/eslint-plugin-react/blob/master/CHANGELOG.md#​7100---2018-06-24) [Compare Source](jsx-eslint/eslint-plugin-react@v7.9.1...v7.10.0) ##### Added * Allow eslint ^5 ([#​1843][] @​papandreou, @​ljharb) * [`no-unsafe`][] rule ([#​1831][], [#​1830][] @​sergei-startsev) * [`no-will-update-set-state`][]: Account for `UNSAFE_` methods ([#​1845][], [#​1844][] @​alexzherdev) ##### Fixed * [`no-typos`][]: Fix static propTypes handling ([#​1827][], [#​1677][] @​alexzherdev) * [`destructuring-assignment`][]: Allow LHS ([#​1825][], [#​1728][] @​alexzherdev) * [`no-unused-prop-types`][]: Fix crash when encountering mixed union and intersection flow types ([#​1806][] @​yannickcr) ##### Changed * Typo fixes in [`jsx-no-target-blank`][] ([#​1805][] @​ferhatelmas)) [#​1845]: `jsx-eslint/eslint-plugin-react#1845 [#​1844]: `jsx-eslint/eslint-plugin-react#1844 [#​1843]: `jsx-eslint/eslint-plugin-react#1843 [#​1831]: `jsx-eslint/eslint-plugin-react#1831 [#​1830]: `jsx-eslint/eslint-plugin-react#1830 [#​1827]: `jsx-eslint/eslint-plugin-react#1827 [#​1825]: `jsx-eslint/eslint-plugin-react#1825 [#​1806]: `jsx-eslint/eslint-plugin-react#1806 [#​1805]: `jsx-eslint/eslint-plugin-react#1805 [#​1728]: `jsx-eslint/eslint-plugin-react#1728 [#​1677]: `jsx-eslint/eslint-plugin-react#1677 --- </details> --- This PR has been generated by [Renovate Bot](https://renovatebot.com).
The rule
react/destructruring-assignement
should not throw in a component constructor.The text was updated successfully, but these errors were encountered: