fix(eslint-plugin): [dot-notation] handle missing declarations (#1947)

also minor perf improvement
typescript-eslint · Apr 29, 2020 · 383f931 · 383f931
1 parent f7ec192
commit 383f931
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/packages/eslint-plugin/src/rules/dot-notation.ts b/packages/eslint-plugin/src/rules/dot-notation.ts
@@ -62,16 +62,17 @@ export default createRule<Options, MessageIds>({
 
     return {
       MemberExpression(node: TSESTree.MemberExpression): void {
-        const objectSymbol = typeChecker.getSymbolAtLocation(
-          parserServices.esTreeNodeToTSNodeMap.get(node.property),
-        );
-
-        if (
-          allowPrivateClassPropertyAccess &&
-          objectSymbol?.declarations[0]?.modifiers?.[0].kind ===
+        if (allowPrivateClassPropertyAccess && node.computed) {
+          // for perf reasons - only fetch the symbol if we have to
+          const objectSymbol = typeChecker.getSymbolAtLocation(
+            parserServices.esTreeNodeToTSNodeMap.get(node.property),
+          );
+          if (
+            objectSymbol?.getDeclarations()?.[0]?.modifiers?.[0].kind ===
             ts.SyntaxKind.PrivateKeyword
-        ) {
-          return;
+          ) {
+            return;
+          }
         }
         rules.MemberExpression(node);
       },