Fix: multiline-html-element-content-newline case sensitivity

armano2 · armano2 · commit a4b40a21cade · 2018-11-14T01:34:23.000+01:00
fix issue #662
diff --git a/lib/rules/multiline-html-element-content-newline.js b/lib/rules/multiline-html-element-content-newline.js
@@ -9,6 +9,7 @@
 // ------------------------------------------------------------------------------
 
 const utils = require('../utils')
+const casing = require('../utils/casing')
 
 // ------------------------------------------------------------------------------
 // Helpers
@@ -20,7 +21,7 @@ function isMultilineElement (element) {
 
 function parseOptions (options) {
   return Object.assign({
-    'ignores': ['pre', 'textarea']
+    ignores: ['pre', 'textarea']
   }, options)
 }
 
@@ -80,12 +81,18 @@ module.exports = {
 
     let inIgnoreElement
 
+    function isIgnoreElement (node) {
+      return ignores.includes(node.name) ||
+        ignores.includes(casing.pascalCase(node.rawName)) ||
+        ignores.includes(casing.kebabCase(node.rawName))
+    }
+
     return utils.defineTemplateBodyVisitor(context, {
       'VElement' (node) {
         if (inIgnoreElement) {
           return
         }
-        if (ignores.indexOf(node.name) >= 0) {
+        if (isIgnoreElement(node)) {
           // ignore element name
           inIgnoreElement = node
           return
@@ -95,6 +102,11 @@ module.exports = {
           return
         }
 
+        if (node.name === 'ignoretag') {
+          console.log(node)
+          debugger
+        }
+
         if (!isMultilineElement(node)) {
           return
         }
@@ -114,7 +126,7 @@ module.exports = {
             },
             messageId: 'unexpectedAfterClosingBracket',
             data: {
-              name: node.name,
+              name: node.rawName,
               actual: getPhrase(beforeLineBreaks)
             },
             fix (fixer) {
diff --git a/tests/lib/rules/multiline-html-element-content-newline.js b/tests/lib/rules/multiline-html-element-content-newline.js
@@ -98,6 +98,38 @@ tester.run('multiline-html-element-content-newline', rule, {
         ignores: ['ignore-tag']
       }]
     },
+    {
+      code: `
+        <template>
+          <IgnoreTag>content</IgnoreTag>
+          <IgnoreTag
+            id="test-pre"
+          >content</IgnoreTag>
+          <IgnoreTag><div
+            >content</div></IgnoreTag>
+          <IgnoreTag>>content
+            content</IgnoreTag>
+        </template>`,
+      options: [{
+        ignores: ['IgnoreTag']
+      }]
+    },
+    {
+      code: `
+        <template>
+          <ignore-tag>content</ignore-tag>
+          <ignore-tag
+            id="test-pre"
+          >content</ignore-tag>
+          <ignore-tag><div
+            >content</div></ignore-tag>
+          <ignore-tag>>content
+            content</ignore-tag>
+        </template>`,
+      options: [{
+        ignores: ['IgnoreTag']
+      }]
+    },
     // Ignore if no closing brackets
     `
       <template>
