Fix multiple var and destructuring handling in props-types (fixes #159)

Author: yannickcr

lib/rules/prop-types.js

@@ -352,12 +352,12 @@ module.exports = function(context) {
           // Do not mark computed props as used.
           type = name !== '__COMPUTED_PROP__' ? 'direct' : null;
         } else if (
-          node.parent.parent.declarations &&
-          node.parent.parent.declarations[0].id.properties &&
-          getKeyValue(node.parent.parent.declarations[0].id.properties[0])
+          node.parent.id &&
+          node.parent.id.properties &&
+          getKeyValue(node.parent.id.properties[0])
         ) {
           type = 'destructuring';
-          properties = node.parent.parent.declarations[0].id.properties;
+          properties = node.parent.id.properties;
         }
         break;
       case 'VariableDeclarator':

tests/lib/rules/prop-types.js

@@ -214,6 +214,20 @@ eslintTester.addRuleTest('lib/rules/prop-types', {
         destructuring: true,
         jsx: true
       }
+    }, {
+      code: [
+        'class Hello extends React.Component {',
+        '  render() {',
+        '    var {firstname, lastname} = this.state, something = this.props;',
+        '    return <div>Hello {firstname}</div>;',
+        '  }',
+        '}'
+      ].join('\n'),
+      ecmaFeatures: {
+        classes: true,
+        destructuring: true,
+        jsx: true
+      }
     }, {
       code: [
         'class Hello extends React.Component {',