vuejs
diff --git a/‎lib/rules/attribute-hyphenation.js
Lines changed: 43 additions & 45 deletions b/‎lib/rules/attribute-hyphenation.js
Lines changed: 43 additions & 45 deletions
diff --git a/‎lib/rules/comment-directive.js
Lines changed: 12 additions & 18 deletions b/‎lib/rules/comment-directive.js
Lines changed: 12 additions & 18 deletions
diff --git a/‎lib/rules/html-closing-bracket-newline.js
Lines changed: 39 additions & 46 deletions b/‎lib/rules/html-closing-bracket-newline.js
Lines changed: 39 additions & 46 deletions
diff --git a/‎lib/rules/html-end-tags.js
Lines changed: 31 additions & 42 deletions b/‎lib/rules/html-end-tags.js
Lines changed: 31 additions & 42 deletions
@@ -11,50 +11,6 @@ const casing = require('../utils/casing')
 // Rule Definition
 // ------------------------------------------------------------------------------
 
-function create (context) {
-  const sourceCode = context.getSourceCode()
-  const options = context.options[0]
-  const useHyphenated = options !== 'never'
-
-  const caseConverter = casing.getConverter(useHyphenated ? 'kebab-case' : 'camelCase')
-
-  function reportIssue (node, name) {
-    const text = sourceCode.getText(node.key)
-
-    context.report({
-      node: node.key,
-      loc: node.loc,
-      message: useHyphenated ? "Attribute '{{text}}' must be hyphenated." : "Attribute '{{text}}' cann't be hyphenated.",
-      data: {
-        text
-      },
-      fix: fixer => fixer.replaceText(node.key, text.replace(name, caseConverter(name)))
-    })
-  }
-
-  function isIgnoredAttribute (value) {
-    if (value.indexOf('data-') !== -1 || value.indexOf('aria-') !== -1) {
-      return true
-    }
-    return useHyphenated ? value.toLowerCase() === value : !/-/.test(value)
-  }
-
-  // ----------------------------------------------------------------------
-  // Public
-  // ----------------------------------------------------------------------
-
-  return utils.defineTemplateBodyVisitor(context, {
-    VAttribute (node) {
-      if (!utils.isCustomComponent(node.parent.parent)) return
-
-      const name = !node.directive ? node.key.rawName : node.key.name === 'bind' ? node.key.raw.argument : false
-      if (!name || isIgnoredAttribute(name)) return
-
-      reportIssue(node, name)
-    }
-  })
-}
-
 module.exports = {
   meta: {
     docs: {
@@ -70,5 +26,47 @@ module.exports = {
     ]
   },
 
-  create
+  create (context) {
+    const sourceCode = context.getSourceCode()
+    const options = context.options[0]
+    const useHyphenated = options !== 'never'
+
+    const caseConverter = casing.getConverter(useHyphenated ? 'kebab-case' : 'camelCase')
+
+    function reportIssue (node, name) {
+      const text = sourceCode.getText(node.key)
+
+      context.report({
+        node: node.key,
+        loc: node.loc,
+        message: useHyphenated ? "Attribute '{{text}}' must be hyphenated." : "Attribute '{{text}}' cann't be hyphenated.",
+        data: {
+          text
+        },
+        fix: fixer => fixer.replaceText(node.key, text.replace(name, caseConverter(name)))
+      })
+    }
+
+    function isIgnoredAttribute (value) {
+      if (value.indexOf('data-') !== -1 || value.indexOf('aria-') !== -1) {
+        return true
+      }
+      return useHyphenated ? value.toLowerCase() === value : !/-/.test(value)
+    }
+
+    // ----------------------------------------------------------------------
+    // Public
+    // ----------------------------------------------------------------------
+
+    return utils.defineTemplateBodyVisitor(context, {
+      VAttribute (node) {
+        if (!utils.isCustomComponent(node.parent.parent)) return
+
+        const name = !node.directive ? node.key.rawName : node.key.name === 'bind' ? node.key.raw.argument : false
+        if (!name || isIgnoredAttribute(name)) return
+
+        reportIssue(node, name)
+      }
+    })
+  }
 }
@@ -99,23 +99,6 @@ function processLine (context, comment) {
   }
 }
 
-/**
- * The implementation of `vue/comment-directive` rule.
- * @param {Program} node The program node to parse.
- * @returns {Object} The visitor of this rule.
- */
-function create (context) {
-  return {
-    Program (node) {
-      const comments = (node.templateBody && node.templateBody.comments) || []
-      for (const comment of comments) {
-        processBlock(context, comment)
-        processLine(context, comment)
-      }
-    }
-  }
-}
-
 // -----------------------------------------------------------------------------
 // Rule Definition
 // -----------------------------------------------------------------------------
@@ -129,5 +112,16 @@ module.exports = {
     },
     schema: []
   },
-  create
+
+  create (context) {
+    return {
+      Program (node) {
+        const comments = (node.templateBody && node.templateBody.comments) || []
+        for (const comment of comments) {
+          processBlock(context, comment)
+          processLine(context, comment)
+        }
+      }
+    }
+  }
 }
@@ -23,57 +23,11 @@ function getPhrase (lineBreaks) {
   }
 }
 
-/**
- * Creates AST event handlers for html-closing-bracket-newline.
- *
- * @param {RuleContext} context - The rule context.
- * @returns {object} AST event handlers.
- */
-function create (context) {
-  const options = context.options[0] || {}
-  const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
-
-  return utils.defineTemplateBodyVisitor(context, {
-    'VStartTag, VEndTag' (node) {
-      const closingBracketToken = template.getLastToken(node)
-      if (closingBracketToken.type !== 'HTMLSelfClosingTagClose' && closingBracketToken.type !== 'HTMLTagClose') {
-        return
-      }
-
-      const prevToken = template.getTokenBefore(closingBracketToken)
-      const type = (node.loc.start.line === prevToken.loc.end.line) ? 'singleline' : 'multiline'
-      const expectedLineBreaks = (options[type] === 'always') ? 1 : 0
-      const actualLineBreaks = (closingBracketToken.loc.start.line - prevToken.loc.end.line)
-
-      if (actualLineBreaks !== expectedLineBreaks) {
-        context.report({
-          node,
-          loc: {
-            start: prevToken.loc.end,
-            end: closingBracketToken.loc.start
-          },
-          message: 'Expected {{expected}} before closing bracket, but {{actual}} found.',
-          data: {
-            expected: getPhrase(expectedLineBreaks),
-            actual: getPhrase(actualLineBreaks)
-          },
-          fix (fixer) {
-            const range = [prevToken.range[1], closingBracketToken.range[0]]
-            const text = '\n'.repeat(expectedLineBreaks)
-            return fixer.replaceTextRange(range, text)
-          }
-        })
-      }
-    }
-  })
-}
-
 // ------------------------------------------------------------------------------
 // Rule Definition
 // ------------------------------------------------------------------------------
 
 module.exports = {
-  create,
   meta: {
     docs: {
       description: "require or disallow a line break before tag's closing brackets",
@@ -89,5 +43,44 @@ module.exports = {
       },
       additionalProperties: false
     }]
+  },
+
+  create (context) {
+    const options = context.options[0] || {}
+    const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
+
+    return utils.defineTemplateBodyVisitor(context, {
+      'VStartTag, VEndTag' (node) {
+        const closingBracketToken = template.getLastToken(node)
+        if (closingBracketToken.type !== 'HTMLSelfClosingTagClose' && closingBracketToken.type !== 'HTMLTagClose') {
+          return
+        }
+
+        const prevToken = template.getTokenBefore(closingBracketToken)
+        const type = (node.loc.start.line === prevToken.loc.end.line) ? 'singleline' : 'multiline'
+        const expectedLineBreaks = (options[type] === 'always') ? 1 : 0
+        const actualLineBreaks = (closingBracketToken.loc.start.line - prevToken.loc.end.line)
+
+        if (actualLineBreaks !== expectedLineBreaks) {
+          context.report({
+            node,
+            loc: {
+              start: prevToken.loc.end,
+              end: closingBracketToken.loc.start
+            },
+            message: 'Expected {{expected}} before closing bracket, but {{actual}} found.',
+            data: {
+              expected: getPhrase(expectedLineBreaks),
+              actual: getPhrase(actualLineBreaks)
+            },
+            fix (fixer) {
+              const range = [prevToken.range[1], closingBracketToken.range[0]]
+              const text = '\n'.repeat(expectedLineBreaks)
+              return fixer.replaceTextRange(range, text)
+            }
+          })
+        }
+      }
+    })
   }
 }
@@ -11,53 +11,11 @@
 
 const utils = require('../utils')
 
-// ------------------------------------------------------------------------------
-// Helpers
-// ------------------------------------------------------------------------------
-
-/**
- * Creates AST event handlers for html-end-tags.
- *
- * @param {RuleContext} context - The rule context.
- * @returns {Object} AST event handlers.
- */
-function create (context) {
-  let hasInvalidEOF = false
-
-  return utils.defineTemplateBodyVisitor(context, {
-    VElement (node) {
-      if (hasInvalidEOF) {
-        return
-      }
-
-      const name = node.name
-      const isVoid = utils.isHtmlVoidElementName(name)
-      const isSelfClosing = node.startTag.selfClosing
-      const hasEndTag = node.endTag != null
-
-      if (!isVoid && !hasEndTag && !isSelfClosing) {
-        context.report({
-          node: node.startTag,
-          loc: node.startTag.loc,
-          message: "'<{{name}}>' should have end tag.",
-          data: { name },
-          fix: (fixer) => fixer.insertTextAfter(node, `</${name}>`)
-        })
-      }
-    }
-  }, {
-    Program (node) {
-      hasInvalidEOF = utils.hasInvalidEOF(node)
-    }
-  })
-}
-
 // ------------------------------------------------------------------------------
 // Rule Definition
 // ------------------------------------------------------------------------------
 
 module.exports = {
-  create,
   meta: {
     docs: {
       description: 'enforce end tag style',
@@ -66,5 +24,36 @@ module.exports = {
     },
     fixable: 'code',
     schema: []
+  },
+
+  create (context) {
+    let hasInvalidEOF = false
+
+    return utils.defineTemplateBodyVisitor(context, {
+      VElement (node) {
+        if (hasInvalidEOF) {
+          return
+        }
+
+        const name = node.name
+        const isVoid = utils.isHtmlVoidElementName(name)
+        const isSelfClosing = node.startTag.selfClosing
+        const hasEndTag = node.endTag != null
+
+        if (!isVoid && !hasEndTag && !isSelfClosing) {
+          context.report({
+            node: node.startTag,
+            loc: node.startTag.loc,
+            message: "'<{{name}}>' should have end tag.",
+            data: { name },
+            fix: (fixer) => fixer.insertTextAfter(node, `</${name}>`)
+          })
+        }
+      }
+    }, {
+      Program (node) {
+        hasInvalidEOF = utils.hasInvalidEOF(node)
+      }
+    })
   }
 }