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

Vue.config.errorHandler called twice when I return a rejected Promise from a handler of a component event #9511

Closed
jfgodoy opened this issue Feb 17, 2019 · 2 comments · Fixed by #9526, mariazevedo88/hash-generator-js#6, tghelere/valorize-vidas#3, tghelere/DeliVuery#15 or tghelere/vue-tetris#1
Closed

Vue.config.errorHandler called twice when I return a rejected Promise from a handler of a component event #9511

jfgodoy opened this issue Feb 17, 2019 · 2 comments · Fixed by #9526, mariazevedo88/hash-generator-js#6, tghelere/valorize-vidas#3, tghelere/DeliVuery#15 or tghelere/vue-tetris#1
Labels
bug
Projects
Maintenance

Comments

@jfgodoy
Copy link

@jfgodoy jfgodoy commented Feb 17, 2019

Version

2.6.6

Reproduction link

https://codepen.io/anon/pen/qgQYra?editors=1011

Steps to reproduce

The problem seems to happen when I handle an event emitted from a component and return a rejected promise. To see the problem in action, open the codepen link, click the button and you will see in the console two logs of the same error, which means the Vue.config.errorHandler was called twice.

What is expected?

I expect the errrorHandler called once.

What is actually happening?

the errorHandler function is called twice.
@posva posva added the bug label Feb 18, 2019
@yyx990803 yyx990803 added this to To do in Maintenance via automation Feb 18, 2019
shasharoman added a commit to shasharoman/vue that referenced this issue Feb 19, 2019
@shasharoman @shasharoman
fix(vuejs#9511): avoid promise catch multiple times
c531319
@shasharoman shasharoman mentioned this issue Feb 19, 2019
Merged
13 tasks
shasharoman added a commit to shasharoman/vue that referenced this issue Feb 19, 2019
@shasharoman @shasharoman
fix(vuejs#9511): add a test case for util/error/invokeWithErrorHandling
749a503
shasharoman added a commit to shasharoman/vue that referenced this issue Feb 19, 2019
@shasharoman @shasharoman
fix(vuejs#9511): update test case for util/error/invokeWithErrorHandling
a7bd451
@marcus-hiles
Copy link

@marcus-hiles marcus-hiles commented Feb 19, 2019

@jfgodoy try using the async keyword with the promise, and lets see if it works for you.
I have another alternative if it doesn't work though

@jfgodoy
Copy link
Author

@jfgodoy jfgodoy commented Feb 19, 2019

hi @marcus-hiles, with async I have the same problem. Only a synchronous throw works

@yyx990803 yyx990803 moved this from To do to In Progress in Maintenance Feb 21, 2019
Maintenance automation moved this from In Progress to Done Feb 21, 2019
yyx990803 added a commit that referenced this issue Feb 21, 2019
@shasharoman @shasharoman @yyx990803
fix(#9511): avoid promise catch multiple times (#9526)
2f3020e 
* fix(#9511): avoid promise catch multiple times

* fix(#9511): add a test case for util/error/invokeWithErrorHandling

* fix(#9511): update test case for util/error/invokeWithErrorHandling
@snyk-bot snyk-bot mentioned this issue Aug 13, 2019
Open
@snyk-bot snyk-bot mentioned this issue Aug 29, 2019
Open
@snyk-bot snyk-bot mentioned this issue Aug 30, 2019
Open
@snyk-bot snyk-bot mentioned this issue Aug 30, 2019
Open
@snyk-bot snyk-bot mentioned this issue Aug 30, 2019
Open
@snyk-bot snyk-bot mentioned this issue Aug 30, 2019
Closed
@snyk-bot snyk-bot mentioned this issue Sep 12, 2019
Open
@tt9133github tt9133github mentioned this issue Sep 18, 2019
Open
@snyk-bot snyk-bot mentioned this issue Sep 22, 2019
Merged
@snyk-bot snyk-bot mentioned this issue Sep 22, 2019
Open
@snyk-bot snyk-bot mentioned this issue Sep 22, 2019
Open
@snyk-bot snyk-bot mentioned this issue Sep 23, 2019
Open
@snyk-bot snyk-bot mentioned this issue Sep 23, 2019
Open
@snyk-bot snyk-bot mentioned this issue Oct 26, 2019
Open
@snyk-bot snyk-bot mentioned this issue Dec 5, 2019
Open
Lostlover added a commit to Lostlover/vue that referenced this issue Dec 10, 2019
@shasharoman @shasharoman @Lostlover
fix(vuejs#9511): avoid promise catch multiple times (vuejs#9526)
dbc9208 
* fix(vuejs#9511): avoid promise catch multiple times

* fix(vuejs#9511): add a test case for util/error/invokeWithErrorHandling

* fix(vuejs#9511): update test case for util/error/invokeWithErrorHandling
@snyk-bot snyk-bot mentioned this issue Jan 20, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Jan 20, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Jan 21, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Jan 21, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Jan 21, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Jan 22, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Feb 6, 2020
Open
@snyk-bot snyk-bot mentioned this issue Feb 6, 2020
Open
@snyk-bot snyk-bot mentioned this issue Feb 8, 2020
Merged
@tt9133github tt9133github mentioned this issue Feb 14, 2020
Open
@snyk-bot snyk-bot mentioned this issue Feb 24, 2020
Closed
@snyk-bot snyk-bot mentioned this issue Feb 25, 2020
Closed
@snyk-bot snyk-bot mentioned this issue Mar 4, 2020
Open
@snyk-bot snyk-bot mentioned this issue Mar 9, 2020
Merged
@snyk-bot snyk-bot mentioned this issue Mar 10, 2020
Open
@snyk-bot snyk-bot mentioned this issue Apr 15, 2020
Open
@snyk-bot snyk-bot mentioned this issue Apr 17, 2020
Merged
erdong-fe pushed a commit to erdong-fe/vue that referenced this issue May 2, 2020
@shasharoman @shasharoman chenzhiguang
fix(vuejs#9511): avoid promise catch multiple times (vuejs#9526)
28aec41 
* fix(vuejs#9511): avoid promise catch multiple times

* fix(vuejs#9511): add a test case for util/error/invokeWithErrorHandling

* fix(vuejs#9511): update test case for util/error/invokeWithErrorHandling
@snyk-bot snyk-bot mentioned this issue May 4, 2020
Open
@snyk-bot snyk-bot mentioned this issue May 5, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
Maintenance
  
Done
Milestone
No milestone
3 participants
@posva @jfgodoy @marcus-hiles