Fix regression in jsx-uses-vars (fixes #73)

Author: yannickcr

lib/rules/jsx-uses-vars.js

@@ -20,8 +20,11 @@ module.exports = function(context) {
       variableUtil.markVariableAsUsed(context, node.expression.name);
     },
 
-    JSXOpeningElement: function(node) {
-      variableUtil.markVariableAsUsed(context, node.name.name);
+    JSXIdentifier: function(node) {
+      if (node.parent.type === 'JSXAttribute') {
+        return;
+      }
+      variableUtil.markVariableAsUsed(context, node.name);
     }
 
   };

tests/lib/rules/jsx-uses-vars.js

@@ -60,6 +60,14 @@ eslintTester.addRuleTest('node_modules/eslint/lib/rules/no-unused-vars', {
       ecmaFeatures: {
         jsx: true
       }
+    }, {
+      code: '\
+        /*eslint jsx-uses-vars:1*/\
+        var App;\
+        <App.Hello />',
+      ecmaFeatures: {
+        jsx: true
+      }
     }
   ],
   invalid: [