fix(eslint-plugin): [class-methods-use-this] detect a problematic cas…

…e for private/protected members if `ignoreClassesThatImplementAnInterface` is `true`
typescript-eslint · Sep 30, 2023 · 4d4e714 · 4d4e714
1 parent 0545fc6
commit 4d4e714
Show file tree

Hide file tree

Showing 2 changed files with 441 additions and 1 deletion.
diff --git a/packages/eslint-plugin/src/rules/class-methods-use-this.ts b/packages/eslint-plugin/src/rules/class-methods-use-this.ts
@@ -140,6 +140,16 @@ export default util.createRule<Options, MessageIds>({
       return oldStack;
     }
 
+    function isPublicMethod(
+      accessibility: TSESTree.Accessibility | undefined,
+    ): boolean {
+      if (!accessibility || accessibility === 'public') {
+        return true;
+      }
+
+      return false;
+    }
+
     /**
      * Check if the node is an instance method not excluded by config
      */
@@ -185,7 +195,8 @@ export default util.createRule<Options, MessageIds>({
         stackContext.usesThis ||
         (ignoreOverrideMethods && stackContext.member.override) ||
         (ignoreClassesThatImplementAnInterface &&
-          stackContext.class.implements != null)
+          stackContext.class.implements != null &&
+          isPublicMethod(stackContext.member.accessibility))
       ) {
         return;
       }