[Fix] destructuring-assignment: handle nested props usage

Author: alexzherdev

lib/rules/destructuring-assignment.js

@@ -82,6 +82,17 @@ module.exports = {
       }
     }
 
+    function isInClassProperty(node) {
+      let curNode = node.parent;
+      while (curNode) {
+        if (curNode.type === 'ClassProperty') {
+          return true;
+        }
+        curNode = curNode.parent;
+      }
+      return false;
+    }
+
     function handleClassUsage(node) {
       // this.props.Aprop || this.context.aProp || this.state.aState
       const isPropUsed = (
@@ -92,7 +103,7 @@ module.exports = {
 
       if (
         isPropUsed && configuration === 'always' &&
-        !(ignoreClassFields && node.parent.type === 'ClassProperty')
+        !(ignoreClassFields && isInClassProperty(node))
       ) {
         context.report({
           node: node,

tests/lib/rules/destructuring-assignment.js

@@ -173,6 +173,17 @@ ruleTester.run('destructuring-assignment', rule, {
     `,
     options: ['always', {ignoreClassFields: true}],
     parser: 'babel-eslint'
+  }, {
+    code: [
+      'class Input extends React.Component {',
+      '  id = `${this.props.name}`;',
+      '  render() {',
+      '    return <div />;',
+      '  }',
+      '}'
+    ].join('\n'),
+    options: ['always', {ignoreClassFields: true}],
+    parser: 'babel-eslint'
   }],
 
   invalid: [{