Skip to content

Commit ea29e9a

Browse files
authored
[Fix] no-arrow-function-lifecycle: prevent crashing on ClassProperty
Fixes #3093
1 parent 210865e commit ea29e9a

File tree

2 files changed

+14
-0
lines changed

2 files changed

+14
-0
lines changed

lib/rules/no-arrow-function-lifecycle.js

+4
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,10 @@ module.exports = {
4444
*/
4545
function reportNoArrowFunctionLifecycle(properties) {
4646
properties.forEach((node) => {
47+
if (!node || !node.value) {
48+
return;
49+
}
50+
4751
const propertyName = astUtil.getPropertyName(node);
4852
const nodeType = node.value.type;
4953
const isLifecycleMethod = (

tests/lib/rules/no-arrow-function-lifecycle.js

+10
Original file line numberDiff line numberDiff line change
@@ -347,6 +347,16 @@ ruleTester.run('no-arrow-function-lifecycle', rule, {
347347
render: function() { return <div />; }
348348
});
349349
`
350+
},
351+
{
352+
code: `
353+
import * as React from "react";
354+
355+
export class MyComponent extends React.Component {
356+
onChange: Function;
357+
}
358+
`,
359+
parser: parsers['@TYPESCRIPT_ESLINT']
350360
}
351361
],
352362

0 commit comments

Comments
 (0)