Fix prop-types to not validate computed keys in destructuring (fixes #236)

Author: yannickcr

lib/rules/prop-types.js

@@ -405,7 +405,7 @@ module.exports = function(context) {
         break;
       case 'destructuring':
         for (var k = 0, l = properties.length; k < l; k++) {
-          if (hasSpreadOperator(properties[k])) {
+          if (hasSpreadOperator(properties[k]) || properties[k].computed) {
             continue;
           }
           var propName = getKeyValue(properties[k]);

tests/lib/rules/prop-types.js

@@ -771,6 +771,17 @@ ruleTester.run('prop-types', rule, {
       ecmaFeatures: {
         jsx: true
       }
+    }, {
+      code: [
+        'class Hello extends React.Component {',
+        '  render() {',
+        '    var foo = \'fullname\';',
+        '    var { [foo]: firstname } = this.props;',
+        '    return <div>Hello {firstname}</div>;',
+        '  }',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint'
     }
   ],