Merge pull request jsx-eslint#193 from mattyod/fix/check-index-is-not-negative

yannickcr · yannickcr · commit 94133a07f771 · 2015-08-20T21:59:41.000+02:00
Fix crash on propTypes declarations with an empty body
diff --git a/lib/rules/prop-types.js b/lib/rules/prop-types.js
@@ -580,7 +580,9 @@ module.exports = function(context) {
         }
       }
 
-      markPropTypesAsDeclared(node, node.value.body.body[i].argument);
+      if (i >= 0) {
+        markPropTypesAsDeclared(node, node.value.body.body[i].argument);
+      }
     },
 
     ObjectExpression: function(node) {

Original file line number	Diff line number	Diff line change
`@@ -580,7 +580,9 @@ module.exports = function(context) {`
`580`	`580`	`}`
`581`	`581`	`}`
`582`	`582`
`583`		`- markPropTypesAsDeclared(node, node.value.body.body[i].argument);`
	`583`	`+ if (i >= 0) {`
	`584`	`+ markPropTypesAsDeclared(node, node.value.body.body[i].argument);`
	`585`	`+ }`
`584`	`586`	`},`
`585`	`587`
`586`	`588`	`ObjectExpression: function(node) {`