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

fix: allow plugins to be re-installed in localVue instance (fixes #406) #411

Merged
merged 2 commits into from
Feb 5, 2018

Conversation

kellym
Copy link
Contributor

@kellym kellym commented Feb 5, 2018

This fixes an issue that occurs when the global Vue object has had a plugin installed to it.

In this scenario, installing the same plugin into the localVue instance created via createLocalVue() will mark the plugin as not installed and fails to reinstall it because of an additional check inside the Vue.use method.

The core source seems to be the process of cloning the original Vue object, in that plugins that are classes are of type function instead of type object, so the original objects are passed through.

@eddyerburgh
Copy link
Member

Thanks for the PR 😀

The tests are failing for Vue 2.0.x. Are you able to debug and fix?

@kellym
Copy link
Contributor Author

kellym commented Feb 5, 2018

Hmm, definitely. I'll take a look.

@eddyerburgh eddyerburgh merged commit 9381736 into vuejs:dev Feb 5, 2018
@eddyerburgh
Copy link
Member

Thanks 😀

@kellym
Copy link
Contributor Author

kellym commented Feb 5, 2018

No, thank you. :) This package makes testing a dream compared to what I'm used to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants