Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

event listener as dynamicProp is added erroneously to instance's props #5520

Closed
iwusong opened this issue Mar 4, 2022 · 0 comments · Fixed by #5517
Closed

event listener as dynamicProp is added erroneously to instance's props #5520

iwusong opened this issue Mar 4, 2022 · 0 comments · Fixed by #5517
Labels
❗ p4-important Priority 4: this fixes bugs that violate documented behavior, or significantly improves perf. 🐞 bug Something isn't working

Comments

@iwusong
Copy link
Contributor

iwusong commented Mar 4, 2022

Version

3.2.31

Reproduction link

sfc.vuejs.org/

Steps to reproduce

open the console，click button

What is expected?

click trigger once

What is actually happening?

after update,click trigger twice

when event listener as dynamicProp on updateProps, it is added to the instance’s props and it can be implicit binded to other components again by "v-bind=$props". it maybe trigger the callback function multiple times.
iwusong added a commit to iwusong/core that referenced this issue Mar 4, 2022
@iwusong
fix(runtime-core): fix event listener is added erroneously to instanc…
04aa312 
…e's props

fix vuejs#5520
@iwusong iwusong changed the title event listener as dynamicProp is added erroneously to props event listener as dynamicProp is added erroneously to instance's props Mar 4, 2022
@posva posva added 🐞 bug Something isn't working ❗ p4-important Priority 4: this fixes bugs that violate documented behavior, or significantly improves perf. labels Mar 4, 2022
@edison1105 edison1105 mentioned this issue Mar 6, 2022
Closed
@iwusong iwusong mentioned this issue Mar 7, 2022
Merged
yyx990803 pushed a commit that referenced this issue Apr 12, 2022
@iwusong
fix(runtime-core): fix event listener as dynamicProp is added erroneo…
8eceabd 
…usly to props (#5517)

fix #5520
@github-actions github-actions bot locked and limited conversation to collaborators Sep 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
❗ p4-important Priority 4: this fixes bugs that violate documented behavior, or significantly improves perf. 🐞 bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@posva @iwusong