Assign parameters to variables defined within the jsx-no-bind.js file

jsx-eslint · Aug 12, 2022 · be4eb34 · be4eb34
1 parent a3ad878
commit be4eb34
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/lib/rules/jsx-closing-bracket-location.js b/lib/rules/jsx-closing-bracket-location.js
@@ -165,7 +165,7 @@ module.exports = {
      * @return {String} The characters used for indentation
      */
     function getIndentation(tokens, expectedLocation, correctColumn) {
-      correctColumn = correctColumn || 0;
+      const newColumn = correctColumn || 0;
       let indentation;
       let spaces = [];
       switch (expectedLocation) {
@@ -179,7 +179,7 @@ module.exports = {
         default:
           indentation = '';
       }
-      if (indentation.length + 1 < correctColumn) {
+      if (indentation.length + 1 < newColumn) {
         // Non-whitespace characters were included in the column offset
         spaces = new Array(+correctColumn + 1 - indentation.length);
       }

diff --git a/lib/rules/jsx-no-bind.js b/lib/rules/jsx-no-bind.js
@@ -69,6 +69,9 @@ module.exports = {
     // bind expression or an arrow function in different block statements
     const blockVariableNameSets = {};
 
+    /**
+     * @param {string | number} blockStart
+     */
     function setBlockVariableNameSet(blockStart) {
       blockVariableNameSets[blockStart] = {
         arrowFunc: new Set(),
@@ -80,7 +83,6 @@ module.exports = {
 
     function getNodeViolationType(node) {
       const nodeType = node.type;
-
       if (
         !configuration.allowBind
         && nodeType === 'CallExpression'
@@ -111,6 +113,11 @@ module.exports = {
       return null;
     }
 
+    /**
+     * @param {string | number} violationType
+     * @param {any} variableName
+     * @param {string | number} blockStart
+     */
     function addVariableNameToSet(violationType, variableName, blockStart) {
       blockVariableNameSets[blockStart][violationType].add(variableName);
     }