warn if $set is used on a property that already exist

(15 commentaires) (0 réaction) (0 personne assignée)TypeScript (33 833 forks)batch import

feature requestgood first issuehas PR

Métriques du dépôt

2.5.16

See the codepen snippet. Follow the comment there to change the vm's data and see what happens.

Even if the prop already exists, using set still makes it reactive, thus trigger view update.

Using set later doesn't update the view.

Related source code: https://github.com/vuejs/vue/blob/3eb37acf98e2d9737de897ebe7bdb7e9576bcc21/src/core/observer/index.js#L192 I think most users would expect set to make the prop reactive whenever it's used. I initially opened an issue for the api doc because it wasn't clear (for me) about this. But the comment in the source code is.

Direction de recherche: Étudiez le code source dans src/core/observer/index.js, en particulier la fonction `set`. Le problème mentionne que si une propriété existe déjà, Vue.set ne déclenche pas la réactivité. Examinez comment l'observateur gère la définition des propriétés. Le comportement attendu est que Vue.set doit toujours rendre une propriété réactive, même si elle existe déjà. Proposez d'ajouter une vérification pour garantir que si la propriété existe déjà, elle appelle toujours `defineReactive` pour la rendre réactive. En outre, considérez les implications pour les objets imbriqués et les tableaux.
Stack technique: javascript
Domaine: frontend
Type d'issue: Bug
Difficulté: 2
Temps estimé: 1-3 heures
Statut d'activité: Active
Clarté: Plutôt claire
Prérequis: Vue
Accessibilité débutant: 60