Merge pull request #1319 from marcelmokos/#969-jsx-no-duplicate-props…

…-type-error issue #969 typeError: name.toLowerCase is not a function…
jsx-eslint · Jul 25, 2017 · 8139569 · 8139569
2 parents 2d2cf65 + 10e4f0c
commit 8139569
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/rules/jsx-no-duplicate-props.js b/lib/rules/jsx-no-duplicate-props.js
@@ -45,6 +45,10 @@ module.exports = {
 
           let name = decl.name.name;
 
+          if (typeof name !== 'string') {
+            return;
+          }
+
           if (ignoreCase) {
             name = name.toLowerCase();
           }

diff --git a/tests/lib/rules/jsx-no-duplicate-props.js b/tests/lib/rules/jsx-no-duplicate-props.js
@@ -49,7 +49,8 @@ ruleTester.run('jsx-no-duplicate-props', rule, {
     {code: '<App c="a" {...this.props} a="c" b="b" />;'},
     {code: '<App A a />;'},
     {code: '<App A b a />;'},
-    {code: '<App A="a" b="b" B="B" />;'}
+    {code: '<App A="a" b="b" B="B" />;'},
+    {code: '<App a:b="c" />;', options: ignoreCaseArgs}
   ],
   invalid: [
     {code: '<App a a />;', errors: [expectedError]},