123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869 |
- /**
- * @author Toru Nagashima
- * @copyright 2017 Toru Nagashima. All rights reserved.
- * See LICENSE file in root directory for full license.
- */
- 'use strict'
- // ------------------------------------------------------------------------------
- // Requirements
- // ------------------------------------------------------------------------------
- const utils = require('../utils')
- // ------------------------------------------------------------------------------
- // Helpers
- // ------------------------------------------------------------------------------
- /**
- * 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) {
- const element = vIf.parent.parent
- return Boolean(
- vIf.value &&
- vIf.value.references.some((reference) =>
- element.variables.some(
- (variable) =>
- variable.id.name === reference.id.name && variable.kind === 'v-for'
- )
- )
- )
- }
- // ------------------------------------------------------------------------------
- // Rule Definition
- // ------------------------------------------------------------------------------
- module.exports = {
- meta: {
- type: 'suggestion',
- docs: {
- description: 'disallow confusing `v-for` and `v-if` on the same element',
- categories: ['vue3-recommended', 'recommended'],
- url: 'https://eslint.vuejs.org/rules/no-confusing-v-for-v-if.html'
- },
- deprecated: true,
- replacedBy: ['no-use-v-if-with-v-for'],
- fixable: null,
- schema: []
- },
- /** @param {RuleContext} context */
- create(context) {
- return utils.defineTemplateBodyVisitor(context, {
- "VAttribute[directive=true][key.name.name='if']"(node) {
- const element = node.parent.parent
- if (utils.hasDirective(element, 'for') && !isUsingIterationVar(node)) {
- context.report({
- node,
- loc: node.loc,
- message: "This 'v-if' should be moved to the wrapper element."
- })
- }
- }
- })
- }
- }
|