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): mutation bug in getDependenciesFromInjectable
#52450
fix(core): mutation bug in getDependenciesFromInjectable
#52450
Conversation
Previously, we would modified `dep.flags` directly to convert injection flags to booleans. This caused a mutation bug where subsequent calls to `getDependenciesFromInjectable` would result in the flags object containing false for every injection flag. Now, we stop modifying `dep.flags` directly and instead assign the converted flags to a new object.
7c8a8fe
to
e01a5ed
Compare
@AleksanderBodurri FYI this change has merge conflicts with 16.2.x branch (so it'd be merged into 17.0.x and main branches only), please create a new PR for 16.2.x if needed. |
This PR was merged into the repository by commit 078ebea. |
Previously, we would modified `dep.flags` directly to convert injection flags to booleans. This caused a mutation bug where subsequent calls to `getDependenciesFromInjectable` would result in the flags object containing false for every injection flag. Now, we stop modifying `dep.flags` directly and instead assign the converted flags to a new object. PR Close #52450
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…2450) Previously, we would modified `dep.flags` directly to convert injection flags to booleans. This caused a mutation bug where subsequent calls to `getDependenciesFromInjectable` would result in the flags object containing false for every injection flag. Now, we stop modifying `dep.flags` directly and instead assign the converted flags to a new object. PR Close angular#52450
…2450) Previously, we would modified `dep.flags` directly to convert injection flags to booleans. This caused a mutation bug where subsequent calls to `getDependenciesFromInjectable` would result in the flags object containing false for every injection flag. Now, we stop modifying `dep.flags` directly and instead assign the converted flags to a new object. PR Close angular#52450
Previously, we would modified
dep.flags
directly to convert injection flags to booleans. This caused a mutation bug where subsequent calls togetDependenciesFromInjectable
would result in the flags object containing false for every injection flag.Now, we stop modifying
dep.flags
directly and instead assign the converted flags to a new object.