Skip to content

Commit 25e05b8

Browse files
committed
Merge pull request #321 from dapetcu21/master
Fix crash on incomplete class property declaration (fixes #317)
2 parents e31b735 + c223c2d commit 25e05b8

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

lib/rules/jsx-sort-prop-types.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,8 @@ module.exports = function(context) {
2424
// (babel-eslint does not expose property name so we have to rely on tokens)
2525
if (node.type === 'ClassProperty') {
2626
var tokens = context.getFirstTokens(node, 2);
27-
if (tokens[0].value === 'propTypes' || tokens[1].value === 'propTypes') {
28-
return true;
29-
}
30-
return false;
27+
return (tokens[0] && tokens[0].value === 'propTypes') ||
28+
(tokens[1] && tokens[1].value === 'propTypes');
3129
}
3230

3331
return Boolean(

lib/rules/sort-comp.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ module.exports = Components.detect(function(context, components) {
132132
// (babel-eslint does not expose property name so we have to rely on tokens)
133133
if (node.type === 'ClassProperty') {
134134
var tokens = context.getFirstTokens(node, 2);
135-
return tokens[1].type === 'Identifier' ? tokens[1].value : tokens[0].value;
135+
return tokens[1] && tokens[1].type === 'Identifier' ? tokens[1].value : tokens[0].value;
136136
}
137137

138138
return node.key.name;

0 commit comments

Comments
 (0)