Merge branch 'master' into patch-15-html-attributes-casing

Author: armano2

docs/rules/no-dupe-keys.md

@@ -0,0 +1,78 @@
+# Prevents duplication of field names (no-dupe-keys)
+
+This rule prevents to use duplicated names.
+
+## :book: Rule Details
+
+This rule is aimed at preventing duplicated property names.
+
+:-1: Examples of **incorrect** code for this rule:
+
+```js
+export default {
+  props: {
+    foo: String
+  },
+  computed: {
+    foo: {
+      get () {
+      }
+    }
+  },
+  data: {
+    foo: null
+  },
+  methods: {
+    foo () {
+    }
+  }
+}
+```
+
+:+1: Examples of **correct** code for this rule:
+
+```js
+export default {
+  props: ['foo'],
+  computed: {
+    bar () {
+    }
+  },
+  data () {
+    return {
+      dat: null
+    }
+  },
+  methods: {
+    test () {
+    }
+  }
+}
+```
+
+## :wrench: Options
+
+This rule has an object option:
+
+`"groups"`: [] (default) array of additional groups to search for duplicates.
+
+### Example:
+
+```
+vue/no-dupe-keys: [2, {
+  groups: ['asyncComputed']
+}]
+```
+
+:-1: Examples of **incorrect** code for this configuration
+
+```js
+export default {
+  computed: {
+    foo () {}
+  },
+  asyncComputed: {
+    foo () {}
+  }
+}
+```

docs/rules/no-parsing-error.md

@@ -1,6 +1,14 @@
 # Disallow parsing errors in `<template>` (no-parsing-error)
 
-This rule reports syntax errors in directives/mustaches of `<template>`.
+This rule reports syntax errors in `<template>`. For example:
+
+- Syntax errors of scripts in directives.
+- Syntax errors of scripts in mustaches.
+- Syntax errors of HTML.
+    - Invalid end tags.
+    - Attributes in end tags.
+    - ...
+    - See also: https://html.spec.whatwg.org/multipage/parsing.html#parse-errors
 
 ## :book: Rule Details
 
@@ -25,4 +33,56 @@ Then reports syntax errors if exist.
 
 ## :wrench: Options
 
-Nothing.
+```json
+{
+    "vue/no-parsing-error": ["error", {
+        "abrupt-closing-of-empty-comment": false,
+        "absence-of-digits-in-numeric-character-reference": false,
+        "cdata-in-html-content": false,
+        "character-reference-outside-unicode-range": false,
+        "control-character-in-input-stream": false,
+        "control-character-reference": false,
+        "eof-before-tag-name": false,
+        "eof-in-cdata": false,
+        "eof-in-comment": false,
+        "eof-in-tag": false,
+        "incorrectly-closed-comment": false,
+        "incorrectly-opened-comment": false,
+        "invalid-first-character-of-tag-name": false,
+        "missing-attribute-value": false,
+        "missing-end-tag-name": false,
+        "missing-semicolon-after-character-reference": false,
+        "missing-whitespace-between-attributes": false,
+        "nested-comment": false,
+        "noncharacter-character-reference": false,
+        "noncharacter-in-input-stream": false,
+        "null-character-reference": false,
+        "surrogate-character-reference": false,
+        "surrogate-in-input-stream": false,
+        "unexpected-character-in-attribute-name": false,
+        "unexpected-character-in-unquoted-attribute-value": false,
+        "unexpected-equals-sign-before-attribute-name": false,
+        "unexpected-null-character": false,
+        "unexpected-question-mark-instead-of-tag-name": false,
+        "unexpected-solidus-in-tag": false,
+        "unknown-named-character-reference": false,
+        "end-tag-with-attributes": false,
+        "duplicate-attribute": false,
+        "end-tag-with-trailing-solidus": false,
+        "non-void-html-element-start-tag-with-trailing-solidus": false,
+        "x-invalid-end-tag": false,
+        "x-invalid-namespace": false
+    }]
+}
+```
+
+You can enable HTML syntax errors by opt-in.
+
+For example, if `"x-invalid-end-tag": true` is given then this rule will catch the end tags of elements which have not opened.
+The error codes are defined in [WHATWG spec](https://html.spec.whatwg.org/multipage/parsing.html#parse-errors), but this rule does not support all of those (E.g., it does not catch errors about DOCTYPE).
+Also, The codes which have `x-` prefix are original in this rule because errors in tree construction phase have not codified yet.
+
+- `x-invalid-end-tag` enables the errors about the end tags of elements which have not opened.
+- `x-invalid-namespace` enables the errors about invalid `xmlns` attributes. See also [step 10. of "create an element for a token"](https://html.spec.whatwg.org/multipage/parsing.html#create-an-element-for-the-token).
+
+> TODO(mysticatea): I will revisit errors in tree construction phase after those are codified.

docs/rules/no-reservered-keys.md

@@ -0,0 +1,54 @@
+# Prevent overwrite reserved keys (no-reservered-keys)
+
+This rule prevents to use reserved names from to avoid conflicts and unexpected behavior.
+
+## Rule Details
+
+:-1: Examples of **incorrect** code for this rule:
+
+```js
+export default {
+  props: {
+    $el: String
+  },
+  computed: {
+    $on: {
+      get () {
+      }
+    }
+  },
+  data: {
+    _foo: null
+  },
+  methods: {
+    $nextTick () {
+    }
+  }
+}
+```
+
+## :wrench: Options
+
+This rule has an object option:
+
+`"reserved"`: [] (default) array of dissalowed names inside `groups`.
+
+`"groups"`: [] (default) array of additional groups to search for duplicates.
+
+### Example:
+
+```
+vue/no-dupe-keys: [2, {
+  reserved: ['foo']
+}]
+```
+
+:-1: Examples of **incorrect** code for this configuration
+
+```js
+export default {
+  computed: {
+    foo () {}
+  }
+}
+```

docs/rules/require-prop-types.md

@@ -0,0 +1,42 @@
+# Prop definitions should be detailed (require-prop-types)
+
+In committed code, prop definitions should always be as detailed as possible, specifying at least type(s).
+
+## :book: Rule Details
+
+This rule enforces that a `props` statement contains type definition.
+
+:-1: Examples of **incorrect** code for this rule:
+
+```js
+export default {
+  props: ['status']
+}
+```
+
+:+1: Examples of **correct** code for this rule:
+
+```js
+export default {
+  props: {
+    status: String
+  }
+}
+```
+
+```js
+export default {
+  props: {
+    status: {
+      type: String,
+      required: true,
+      validate: function (value) {
+        return ['syncing', 'synced', 'version-conflict', 'error'].indexOf(value) !== -1
+      }
+    }
+  }
+}
+```
+## :wrench: Options
+
+Nothing.

lib/rules/html-end-tags.js

@@ -19,12 +19,12 @@ const utils = require('../utils')
  * Creates AST event handlers for html-end-tags.
  *
  * @param {RuleContext} context - The rule context.
- * @returns {object} AST event handlers.
+ * @returns {Object} AST event handlers.
  */
 function create (context) {
   utils.registerTemplateBodyVisitor(context, {
     VElement (node) {
-      const name = node.startTag.id.name
+      const name = node.name
       const isVoid = utils.isVoidElementName(name)
       const hasEndTag = node.endTag != null
 

lib/rules/html-no-self-closing.js

@@ -19,20 +19,27 @@ const utils = require('../utils')
  * Creates AST event handlers for html-no-self-closing.
  *
  * @param {RuleContext} context - The rule context.
- * @returns {object} AST event handlers.
+ * @returns {Object} AST event handlers.
  */
 function create (context) {
   utils.registerTemplateBodyVisitor(context, {
-    'VStartTag[selfClosing=true]' (node) {
-      if (!utils.isSvgElementName(node.id.name) && !utils.isMathMLElementName(node.id.name)) {
-        const pos = node.range[1] - 2
-        context.report({
-          node,
-          loc: node.loc,
-          message: 'Self-closing should not be used.',
-          fix: (fixer) => fixer.removeRange([pos, pos + 1])
-        })
+    'VElement' (node) {
+      if (utils.isSvgElementName(node.name)) {
+        return
       }
+
+      const sourceCode = context.parserServices.getTemplateBodyTokenStore(context)
+      const lastToken = sourceCode.getLastToken(node.startTag)
+      if (lastToken.type !== 'HTMLSelfClosingTagClose') {
+        return
+      }
+
+      context.report({
+        node: lastToken,
+        loc: lastToken.loc,
+        message: 'Self-closing should not be used.',
+        fix: (fixer) => fixer.removeRange([lastToken.range[0], lastToken.range[0] + 1])
+      })
     }
   })
 
@@ -49,8 +56,10 @@ module.exports = {
     docs: {
       description: 'disallow self-closing elements.',
       category: 'Best Practices',
-      recommended: false
+      recommended: false,
+      replacedBy: []
     },
+    deprecated: true,
     fixable: 'code',
     schema: []
   }

lib/rules/html-quotes.js

@@ -19,7 +19,7 @@ const utils = require('../utils')
  * Creates AST event handlers for html-quotes.
  *
  * @param {RuleContext} context - The rule context.
- * @returns {object} AST event handlers.
+ * @returns {Object} AST event handlers.
  */
 function create (context) {
   const sourceCode = context.getSourceCode()

lib/rules/name-property-casing.js

@@ -21,11 +21,11 @@ function create (context) {
 
   return utils.executeOnVue(context, (obj) => {
     const node = obj.properties
-      .filter(item => (
+      .find(item => (
         item.type === 'Property' &&
         item.key.name === 'name' &&
         item.value.type === 'Literal'
-      ))[0]
+      ))
 
     if (!node) return
 

lib/rules/no-confusing-v-for-v-if.js

@@ -34,12 +34,14 @@ function isUsingIterationVar (vIf) {
  * Creates AST event handlers for no-confusing-v-for-v-if.
  *
  * @param {RuleContext} context - The rule context.
- * @returns {object} AST event handlers.
+ * @returns {Object} AST event handlers.
  */
 function create (context) {
   utils.registerTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='if']" (node) {
-      if (utils.hasDirective(node.parent, 'for') && !isUsingIterationVar(node)) {
+      const element = node.parent.parent
+
+      if (utils.hasDirective(element, 'for') && !isUsingIterationVar(node)) {
         context.report({
           node,
           loc: node.loc,