-
-
Notifications
You must be signed in to change notification settings - Fork 681
/
Copy pathno-use-v-if-with-v-for.js
112 lines (106 loc) · 3.37 KB
/
no-use-v-if-with-v-for.js
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
/**
* @author Yosuke Ota
*
* Style guide: https://vuejs.org/style-guide/rules-essential.html#avoid-v-if-with-v-for
*/
'use strict'
const utils = require('../utils')
/**
* Check whether the given `v-if` node is using the variable which is defined by the `v-for` directive.
* @param {VDirective} vIf The `v-if` attribute node to check.
* @returns {boolean} `true` if the `v-if` is using the variable which is defined by the `v-for` directive.
*/
function isUsingIterationVar(vIf) {
return !!getVForUsingIterationVar(vIf)
}
/** @param {VDirective} vIf */
function getVForUsingIterationVar(vIf) {
if (!vIf.value) {
return null
}
const element = vIf.parent.parent
for (const reference of vIf.value.references) {
const targetVFor = element.variables.find(
(variable) =>
variable.id.name === reference.id.name && variable.kind === 'v-for'
)
if (targetVFor) {
return targetVFor
}
}
return null
}
module.exports = {
meta: {
type: 'suggestion',
docs: {
description: 'disallow using `v-if` on the same element as `v-for`',
categories: ['vue3-essential', 'vue2-essential'],
url: 'https://eslint.vuejs.org/rules/no-use-v-if-with-v-for.html'
},
fixable: null,
schema: [
{
type: 'object',
properties: {
allowUsingIterationVar: {
type: 'boolean'
}
},
additionalProperties: false
}
],
messages: {
movedToWrapper: "This 'v-if' should be moved to the wrapper element.",
shouldUseComputed:
"The '{{iteratorName}}' {{kind}} inside 'v-for' directive should be replaced with a computed property that returns filtered array instead. You should not mix 'v-for' with 'v-if'."
}
},
/** @param {RuleContext} context */
create(context) {
const options = context.options[0] || {}
const allowUsingIterationVar = options.allowUsingIterationVar === true // default false
return utils.defineTemplateBodyVisitor(context, {
/** @param {VDirective} node */
"VAttribute[directive=true][key.name.name='if']"(node) {
const element = node.parent.parent
if (utils.hasDirective(element, 'for')) {
if (isUsingIterationVar(node)) {
if (!allowUsingIterationVar) {
const vForVar = getVForUsingIterationVar(node)
if (!vForVar) {
return
}
let targetVForExpr = vForVar.id.parent
while (targetVForExpr.type !== 'VForExpression') {
targetVForExpr = /** @type {ASTNode} */ (targetVForExpr.parent)
}
const iteratorNode = targetVForExpr.right
context.report({
node,
loc: node.loc,
messageId: 'shouldUseComputed',
data: {
iteratorName:
iteratorNode.type === 'Identifier'
? iteratorNode.name
: context.getSourceCode().getText(iteratorNode),
kind:
iteratorNode.type === 'Identifier'
? 'variable'
: 'expression'
}
})
}
} else {
context.report({
node,
loc: node.loc,
messageId: 'movedToWrapper'
})
}
}
}
})
}
}