10000 fix(compiler-cli): only bind inputs that are part of microsyntax to a structural directive by JoostK · Pull Request #52453 · angular/angular · GitHub
[go: up one dir, main page]

Skip to content

fix(compiler-cli): only bind inputs that are part of microsyntax to a structural directive #52453

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JoostK
Copy link
Member
@JoostK JoostK commented Oct 30, 2023

Prior to this change the template type-check generator would incorrectly apply inputs and attributes to a structural directive, where only the bindings as present in microsyntax are actually bound to the directive. This introduced a problem where usages of template variables could not be resolved, because the template variables are out-of-scope of the template element itself.

Closes #49931

@JoostK JoostK added target: patch This PR is targeted for the next patch release area: compiler Issues related to `ngc`, Angular's template compiler labels Oct 30, 2023
@ngbot ngbot bot modified the milestone: Backlog Oct 30, 2023
@JoostK JoostK marked this pull request as ready for review October 30, 2023 20:00
@JoostK JoostK added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 30, 2023
@JoostK JoostK force-pushed the ngtsc/ttc/tmpl-bindings branch from c2e9826 to 1c63c6c Compare November 9, 2023 17:48
@JoostK JoostK force-pushed the ngtsc/ttc/tmpl-bindings branch 2 times, most recently from 705ef0f to dc3ce91 Compare June 15, 2024 10:22
@JoostK JoostK requested a review from atscott June 15, 2024 10:42
… structural directive

Prior to this change the template type-check generator would incorrectly apply inputs
and attributes to a structural directive, where only the bindings as present in microsyntax
are actually bound to the directive. This introduced a problem where usages of template
variables could not be resolved, because the template variables are out-of-scope of the
template element itself.

Closes angular#49931
@JoostK JoostK force-pushed the ngtsc/ttc/tmpl-bindings branch from dc3ce91 to ba9865b Compare October 6, 2024 19:27
@pullapprove pullapprove bot removed the request for review from atscott October 6, 2024 19:28
@angular-robot angular-robot bot removed the area: compiler Issues related to `ngc`, Angular's template compiler label Oct 6, 2024
@ngbot ngbot bot removed this from the Backlog milestone Oct 6, 2024
@JoostK JoostK added the area: compiler Issues related to `ngc`, Angular's template compiler label Oct 6, 2024
@ngbot ngbot bot added this to the Backlog milestone Oct 6, 2024
@JoostK JoostK requested a review from crisbeto October 6, 2024 19:28
@thePunderWoman thePunderWoman marked this pull request as draft May 21, 2025 15:24
@thePunderWoman thePunderWoman removed the request for review from crisbeto June 3, 2025 10:57
@thePunderWoman
Copy link
Contributor

Hey @JoostK. Should we land this?

@JoostK
Copy link
Member Author
JoostK commented Jun 30, 2025

Hey @JoostK. Should we land this?

Yes, I'd think so as it fixes a reported bug. Let me rebase during the coming days 👍

@JoostK JoostK added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews area: compiler Issues related to `ngc`, Angular's template compiler compiler: template type-checking target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uninformative error message with ngForTrackBy - Error: Could not resolve [object Object] / undefined
2 participants
0