Fix no-global-function-names end positions (#918)

stylelint-scss · Dec 13, 2023 · 9716542 · 9716542
1 parent 29e1da3
commit 9716542
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/rules/no-global-function-names/__tests__/index.js b/src/rules/no-global-function-names/__tests__/index.js
@@ -302,6 +302,8 @@ testRule({
     `,
       line: 3,
       column: 21,
+      endLine: 3,
+      endColumn: 24,
       message: messages.rejected("red"),
       description: "red"
     },
@@ -471,7 +473,9 @@ testRule({
       }
     `,
       line: 3,
-      column: 21,
+      column: 23,
+      endLine: 3,
+      endColumn: 35,
       message: messages.rejected("change-color"),
       description: "change-color"
     },

diff --git a/src/rules/no-global-function-names/index.js b/src/rules/no-global-function-names/index.js
@@ -2,7 +2,6 @@
 
 const valueParser = require("postcss-value-parser");
 const { utils } = require("stylelint");
-const declarationValueIndex = require("../../utils/declarationValueIndex");
 const namespace = require("../../utils/namespace");
 const ruleUrl = require("../../utils/ruleUrl");
 
@@ -182,11 +181,11 @@ function rule(value) {
           return;
         }
 
-        if (Object.keys(rules).includes(cleanValue)) {
+        if (rules[cleanValue]) {
           utils.report({
             message: messages.rejected(cleanValue),
             node: decl,
-            index: declarationValueIndex(decl) + node.sourceIndex,
+            word: cleanValue,
             result,
             ruleName
           });