Changeset 0c6b92a for imaps-frontend/node_modules/eslint-plugin-react/lib/rules/no-unused-state.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/no-unused-state.js (modified) (19 diffs)

imaps-frontend/node_modules/eslint-plugin-react/lib/rules/no-unused-state.js

@@ -11 +11 @@
 
 const docsUrl = require('../util/docsUrl');
-const ast = require('../util/ast');
+const astUtil = require('../util/ast');
 const componentUtil = require('../util/componentUtil');
 const report = require('../util/report');
@@ -45 +45 @@
 
 function isThisExpression(node) {
-  return ast.unwrapTSAsExpression(uncast(node)).type === 'ThisExpression';
+  return astUtil.unwrapTSAsExpression(uncast(node)).type === 'ThisExpression';
 }
 
@@ -66 +66 @@
 
 function isSetStateCall(node) {
-  const unwrappedCalleeNode = ast.unwrapTSAsExpression(node.callee);
+  const unwrappedCalleeNode = astUtil.unwrapTSAsExpression(node.callee);
 
   return (
@@ -79 +79 @@
 };
 
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
@@ -175 +176 @@
     // destructures `this.state`.
     function handleStateDestructuring(node) {
-      for (const prop of node.properties) {
+      node.properties.forEach((prop) => {
         if (prop.type === 'Property') {
           addUsedStateField(prop.key);
@@ -184 +185 @@
           classInfo.aliases.add(getName(prop.argument));
         }
-      }
+      });
     }
 
@@ -190 +191 @@
     // AssignmentExpressions and VariableDeclarators.
     function handleAssignment(left, right) {
-      const unwrappedRight = ast.unwrapTSAsExpression(right);
+      const unwrappedRight = astUtil.unwrapTSAsExpression(right);
 
       switch (left.type) {
@@ -202 +203 @@
             handleStateDestructuring(left);
           } else if (isThisExpression(unwrappedRight) && classInfo.aliases) {
-            for (const prop of left.properties) {
+            left.properties.forEach((prop) => {
               if (prop.type === 'Property' && getName(prop.key) === 'state') {
                 const name = getName(prop.value);
@@ -211 +212 @@
                 }
               }
-            }
+            });
           }
           break;
@@ -221 +222 @@
     function reportUnusedFields() {
       // Report all unused state fields.
-      for (const node of classInfo.stateFields) {
+      classInfo.stateFields.forEach((node) => {
         const name = getName(node.key);
         if (!classInfo.usedStateFields.has(name)) {
@@ -231 +232 @@
           });
         }
-      }
+      });
     }
 
@@ -293 +294 @@
         }
 
-        const unwrappedNode = ast.unwrapTSAsExpression(node);
-        const unwrappedArgumentNode = ast.unwrapTSAsExpression(unwrappedNode.arguments[0]);
+        const unwrappedNode = astUtil.unwrapTSAsExpression(node);
+        const unwrappedArgumentNode = astUtil.unwrapTSAsExpression(unwrappedNode.arguments[0]);
 
         // If we're looking at a `this.setState({})` invocation, record all the
@@ -309 +310 @@
           && unwrappedArgumentNode.type === 'ArrowFunctionExpression'
         ) {
-          const unwrappedBodyNode = ast.unwrapTSAsExpression(unwrappedArgumentNode.body);
+          const unwrappedBodyNode = astUtil.unwrapTSAsExpression(unwrappedArgumentNode.body);
 
           if (unwrappedBodyNode.type === 'ObjectExpression') {
@@ -331 +332 @@
         // If we see state being assigned as a class property using an object
         // expression, record all the fields of that object as state fields.
-        const unwrappedValueNode = ast.unwrapTSAsExpression(node.value);
+        const unwrappedValueNode = astUtil.unwrapTSAsExpression(node.value);
 
         const name = getName(node.key);
@@ -432 +433 @@
         }
 
-        if (parent.key.name === 'getInitialState') {
+        if (
+          'key' in parent
+          && 'name' in parent.key
+          && parent.key.name === 'getInitialState'
+        ) {
           const body = node.body.body;
           const lastBodyNode = body[body.length - 1];
@@ -453 +458 @@
         }
 
-        const unwrappedLeft = ast.unwrapTSAsExpression(node.left);
-        const unwrappedRight = ast.unwrapTSAsExpression(node.right);
+        const unwrappedLeft = astUtil.unwrapTSAsExpression(node.left);
+        const unwrappedRight = astUtil.unwrapTSAsExpression(node.right);
 
         // Check for assignments like `this.state = {}`
@@ -464 +469 @@
         ) {
           // Find the nearest function expression containing this assignment.
+          /** @type {import("eslint").Rule.Node} */
           let fn = node;
           while (fn.type !== 'FunctionExpression' && fn.parent) {
@@ -494 +500 @@
           return;
         }
-        if (isStateReference(ast.unwrapTSAsExpression(node.object))) {
+        if (isStateReference(astUtil.unwrapTSAsExpression(node.object))) {
           // If we see this.state[foo] access, give up.
           if (node.computed && node.property.type !== 'Literal') {
@@ -503 +509 @@
           addUsedStateField(node.property);
         // If we see a `this.state` access in a CallExpression, give up.
-        } else if (isStateReference(node) && node.parent.type === 'CallExpression') {
+        } else if (isStateReference(node) && astUtil.isCallExpression(node.parent)) {
           classInfo = null;
         }