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(core): Injector correctly honors the @Self flag #24520

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@mhevery
Member

mhevery commented Jun 14, 2018

Injector was incorrectly returning instance from parent injector even
when @Self was specified.

fix(core): Injector correctly honors the @self flag
Injector was incorrectly returning instance from parent injector even
when `@self` was specified.
@mhevery

This comment has been minimized.

@mary-poppins

This comment has been minimized.

mary-poppins commented Jun 14, 2018

@alxhub

alxhub approved these changes Jun 14, 2018

mhevery added a commit that referenced this pull request Jun 14, 2018

fix(core): Injector correctly honors the @self flag (#24520)
Injector was incorrectly returning instance from parent injector even
when `@self` was specified.

PR Close #24520

@mhevery mhevery closed this in ccbda9d Jun 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment