Skip to content

Commit 87a42c9

Browse files
committed
Fix A Crash
In standard (latest `master`) when I enable `experimentalObjectRestSpread` in eslint. We run into this issue when using eslint 1.1.0 and eslint-plugin-react 3.2.1. ```js ~/dev/standard (master) $ node bin/cmd.js test.js standard: Unexpected linter output: TypeError: Cannot read property 'type' of undefined at isPropTypesDeclaration (/Users/jamuferguson/dev/standard/node_modules/eslint-plugin-react/lib/rules/prop-types.js:49:13) at /Users/jamuferguson/dev/standard/node_modules/eslint-plugin-react/lib/rules/prop-types.js:589:14 at Array.forEach (native) at EventEmitter.ObjectExpression (/Users/jamuferguson/dev/standard/node_modules/eslint-plugin-react/lib/rules/prop-types.js:588:23) at EventEmitter.emit (events.js:129:20) at Controller.controller.traverse.enter (/Users/jamuferguson/dev/standard/node_modules/eslint/lib/eslint.js:824:25) at Controller.__execute (/Users/jamuferguson/dev/standard/node_modules/estraverse/estraverse.js:397:31) at Controller.traverse (/Users/jamuferguson/dev/standard/node_modules/estraverse/estraverse.js:495:28) at EventEmitter.module.exports.api.verify (/Users/jamuferguson/dev/standard/node_modules/eslint/lib/eslint.js:817:24) at processText (/Users/jamuferguson/dev/standard/node_modules/eslint/lib/cli-engine.js:199:27) ``` By all means tell me to file an issue first or whatever the correct approach is :)
1 parent 13fa214 commit 87a42c9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/rules/prop-types.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ module.exports = function(context) {
4646

4747
// Special case for class properties
4848
// (babel-eslint does not expose property name so we have to rely on tokens)
49-
if (node.type === 'ClassProperty') {
49+
if (node && node.type === 'ClassProperty') {
5050
var tokens = context.getFirstTokens(node, 2);
5151
if (
5252
tokens[0].value === 'propTypes' ||

0 commit comments

Comments
 (0)