Fix array indexing for markdownlint-disable-next-line when front matt…

…er is present.
DavidAnson · Dec 23, 2021 · 5253669 · 5253669
1 parent 7a76f1d
commit 5253669
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 4 deletions.
diff --git a/demo/markdownlint-browser.js b/demo/markdownlint-browser.js
@@ -1305,8 +1305,9 @@ function getEnabledRulesPerLineNumber(ruleList, lines, frontMatterLines, noInlin
     // eslint-disable-next-line jsdoc/require-jsdoc
     function disableNextLine(action, parameter, lineNumber) {
         if (action === "DISABLE-NEXT-LINE") {
-            enabledRulesPerLineNumber[lineNumber + 1] =
-                applyEnableDisable(action, parameter, enabledRulesPerLineNumber[lineNumber + 1] || {});
+            var nextLineNumber = frontMatterLines.length + lineNumber + 1;
+            enabledRulesPerLineNumber[nextLineNumber] =
+                applyEnableDisable(action, parameter, enabledRulesPerLineNumber[nextLineNumber] || {});
         }
     }
     // Handle inline comments

diff --git a/lib/markdownlint.js b/lib/markdownlint.js
@@ -403,11 +403,12 @@ function getEnabledRulesPerLineNumber(
   // eslint-disable-next-line jsdoc/require-jsdoc
   function disableNextLine(action, parameter, lineNumber) {
     if (action === "DISABLE-NEXT-LINE") {
-      enabledRulesPerLineNumber[lineNumber + 1] =
+      const nextLineNumber = frontMatterLines.length + lineNumber + 1;
+      enabledRulesPerLineNumber[nextLineNumber] =
         applyEnableDisable(
           action,
           parameter,
-          enabledRulesPerLineNumber[lineNumber + 1] || {}
+          enabledRulesPerLineNumber[nextLineNumber] || {}
         );
     }
   }

diff --git a/test/front-matter-with-disable-next-line.md b/test/front-matter-with-disable-next-line.md
@@ -0,0 +1,20 @@
+---
+front: matter
+---
+
+# Front Matter with Disable-Next-Line
+
+<!-- markdownlint-disable-next-line no-inline-html -->
+<hr/>
+
+<hr/> {MD033}
+
+<!-- markdownlint-disable-next-line -->
+<hr/>
+
+<hr/> {MD033}
+<hr/> {MD033}
+<!-- markdownlint-disable-next-line -->
+<hr/>
+<hr/> {MD033}
+<hr/> {MD033}