Changeset 0c6b92a for imaps-frontend/node_modules/eslint-plugin-react/lib/rules/jsx-no-bind.js

Timestamp:

12/12/24 17:06:06 (5 weeks ago)

Author:

stefan toskovski <stefantoska84@…>

Branches:

main

Parents:

d565449

Message:

Pred finalna verzija

File:

• imaps-frontend/node_modules/eslint-plugin-react/lib/rules/jsx-no-bind.js (modified) (6 diffs)

imaps-frontend/node_modules/eslint-plugin-react/lib/rules/jsx-no-bind.js

@@ -10 +10 @@
 const propName = require('jsx-ast-utils/propName');
 const docsUrl = require('../util/docsUrl');
+const astUtil = require('../util/ast');
 const jsxUtil = require('../util/jsx');
 const report = require('../util/report');
@@ -25 +26 @@
 };
 
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
@@ -84 +86 @@
 
     function getNodeViolationType(node) {
-      const nodeType = node.type;
       if (
         !configuration.allowBind
-        && nodeType === 'CallExpression'
+        && astUtil.isCallExpression(node)
         && node.callee.type === 'MemberExpression'
         && node.callee.property.type === 'Identifier'
@@ -94 +95 @@
         return 'bindCall';
       }
-      if (nodeType === 'ConditionalExpression') {
+      if (node.type === 'ConditionalExpression') {
         return getNodeViolationType(node.test)
                || getNodeViolationType(node.consequent)
                || getNodeViolationType(node.alternate);
       }
-      if (!configuration.allowArrowFunctions && nodeType === 'ArrowFunctionExpression') {
+      if (!configuration.allowArrowFunctions && node.type === 'ArrowFunctionExpression') {
         return 'arrowFunc';
       }
       if (
         !configuration.allowFunctions
-        && (nodeType === 'FunctionExpression' || nodeType === 'FunctionDeclaration')
+        && (node.type === 'FunctionExpression' || node.type === 'FunctionDeclaration')
       ) {
         return 'func';
       }
-      if (!configuration.allowBind && nodeType === 'BindExpression') {
+      if (!configuration.allowBind && node.type === 'BindExpression') {
         return 'bindExpression';
       }
@@ -117 +118 @@
     /**
      * @param {string | number} violationType
-     * @param {any} variableName
+     * @param {unknown} variableName
      * @param {string | number} blockStart
      */
@@ -176 +177 @@
           blockAncestors.length > 0
           && variableViolationType
+          && 'kind' in node.parent
           && node.parent.kind === 'const' // only support const right now
         ) {
-          addVariableNameToSet(
-            variableViolationType, node.id.name, blockAncestors[0].range[0]
-          );
+          addVariableNameToSet(variableViolationType, 'name' in node.id ? node.id.name : undefined, blockAncestors[0].range[0]);
         }
       },