[Fix] jsx-handler-names: Skip inline handlers when checkInlineFunct…

…ion=false. Fixes #2832
jsx-eslint · Oct 16, 2020 · 42e2b0e · 42e2b0e
1 parent aeff5ea
commit 42e2b0e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/lib/rules/jsx-handler-names.js b/lib/rules/jsx-handler-names.js
@@ -106,6 +106,7 @@ module.exports = {
         if (
           !node.value
           || !node.value.expression
+          || (!checkInlineFunction && isInlineHandler(node))
           || (
             !checkLocal
             && (isInlineHandler(node)

diff --git a/tests/lib/rules/jsx-handler-names.js b/tests/lib/rules/jsx-handler-names.js
@@ -59,6 +59,11 @@ ruleTester.run('jsx-handler-names', rule, {
     options: [{
       checkLocalVariables: false
     }]
+  }, {
+    code: '<TestComponent onChange={event => window.alert(event.target.value)} />',
+    options: [{
+      checkInlineFunction: false
+    }]
   }, {
     code: '<TestComponent onChange={() => handleChange()} />',
     options: [{