Fix: Add a new group OTHER_DIRECTIVES

Author: ydfzgyj

docs/rules/attributes-order.md

@@ -19,8 +19,10 @@ ex: 'id'
 ex: 'ref', 'key', 'slot'
 - TWO\_WAY\_BINDING
 ex: 'v-model'
+- OTHER_DIRECTIVES
+ex: 'v-custom-directive'
 - OTHER_ATTR
-ex: 'custom-prop="foo"', 'v-bind:prop="foo"', ':prop="foo"', 'v-custom-directive'
+ex: 'custom-prop="foo"', 'v-bind:prop="foo"', ':prop="foo"'
 - EVENTS
 ex: '@click="functionCall"', 'v-on="event"'
 - CONTENT
@@ -87,7 +89,7 @@ Specify custom order of attribute groups
 :+1: Examples of **correct** code with custom order`:
 
 ```html
-<!-- 'vue/attributes-order': [2, { order: ['LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'OTHER_ATTR', 'EVENTS', 'CONTENT', 'DEFINITION'] }] -->
+<!-- 'vue/attributes-order': [2, { order: ['LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'OTHER_DIRECTIVES', 'OTHER_ATTR', 'EVENTS', 'CONTENT', 'DEFINITION'] }] -->
 <div
   prop-one="prop"
   prop-two="prop"
@@ -96,7 +98,7 @@ Specify custom order of attribute groups
 ```
 
 ```html
-<!-- 'vue/attributes-order': [2, { order: ['LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'DEFINITION', 'OTHER_ATTR', 'EVENTS', 'CONTENT'] }] -->
+<!-- 'vue/attributes-order': [2, { order: ['LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'DEFINITION', 'OTHER_DIRECTIVES', 'OTHER_ATTR', 'EVENTS', 'CONTENT'] }] -->
 <div
   ref="header"
   is="header"
@@ -108,7 +110,7 @@ Specify custom order of attribute groups
 :-1: Examples of **incorrect** code with custom order`:
 
 ```html
-<!-- 'vue/attributes-order': [2, { order: ['LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'DEFINITION', 'OTHER_ATTR', 'EVENTS', 'CONTENT'] }] -->
+<!-- 'vue/attributes-order': [2, { order: ['LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'DEFINITION', 'OTHER_DIRECTIVES', 'OTHER_ATTR', 'EVENTS', 'CONTENT'] }] -->
 <div
   ref="header"
   prop-one="prop"

lib/rules/attributes-order.js

@@ -24,7 +24,7 @@ function getAttributeType (name, isDirective) {
     } else if (name === 'html' || name === 'text') {
       return 'CONTENT'
     } else {
-      return 'OTHER_ATTR'
+      return 'OTHER_DIRECTIVES'
     }
   } else {
     if (name === 'is') {
@@ -50,7 +50,7 @@ function getPosition (attribute, attributeOrder) {
 
 function create (context) {
   const sourceCode = context.getSourceCode()
-  let attributeOrder = ['DEFINITION', 'LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'OTHER_ATTR', 'EVENTS', 'CONTENT']
+  let attributeOrder = ['DEFINITION', 'LIST_RENDERING', 'CONDITIONALS', 'RENDER_MODIFIERS', 'GLOBAL', 'UNIQUE', 'TWO_WAY_BINDING', 'OTHER_DIRECTIVES', 'OTHER_ATTR', 'EVENTS', 'CONTENT']
   if (context.options[0] && context.options[0].order) {
     attributeOrder = context.options[0].order
   }

tests/lib/rules/attributes-order.js

@@ -190,6 +190,7 @@ tester.run('attributes-order', rule, {
             'GLOBAL',
             'UNIQUE',
             'TWO_WAY_BINDING',
+            'OTHER_DIRECTIVES',
             'OTHER_ATTR',
             'EVENTS',
             'CONTENT',
@@ -208,6 +209,7 @@ tester.run('attributes-order', rule, {
             'UNIQUE',
             'TWO_WAY_BINDING',
             'DEFINITION',
+            'OTHER_DIRECTIVES',
             'OTHER_ATTR',
             'EVENTS',
             'CONTENT']
@@ -293,6 +295,7 @@ tester.run('attributes-order', rule, {
             'UNIQUE',
             'TWO_WAY_BINDING',
             'DEFINITION',
+            'OTHER_DIRECTIVES',
             'OTHER_ATTR',
             'EVENTS',
             'CONTENT']