vuejs/vue

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

Aperta

#8129 aperta il 4 mag 2018

Vedi su GitHubGuida contributor
 (15 commenti) (0 reazioni) (0 assegnatari)TypeScript (33.833 fork)batch import
AndroidBugGood first issuefeature requestgood first issuehas PR

Metriche repository

Star
 (209.764 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Version

2.5.16

Reproduction link

https://codepen.io/JJPandari/pen/gzLVBq?editors=1010

Steps to reproduce

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

What is expected?

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

What is actually happening?

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.

Guida contributor

Direzione di ricerca
Analizza il codice sorgente in src/core/observer/index.js, in particolare la funzione `set`. Il problema menziona che se una proprietà esiste già, Vue.set non attiva la reattività. Osserva come l'osservatore gestisce l'impostazione delle proprietà. Il comportamento atteso è che Vue.set dovrebbe sempre rendere reattiva una proprietà, anche se esiste già. Suggerisci di aggiungere un controllo per garantire che se la proprietà esiste già, chiami comunque `defineReactive` per renderla reattiva. Inoltre, considera le implicazioni per oggetti annidati e array.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Vue
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60