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(compiler-cli): report error when a reference target is missing instead of crashing #39805
Conversation
…stead of crashing If a template declares a reference to a missing target then referring to that reference from elsewhere in the template would crash the template type checker, due to a regression introduced in angular#38618. This commit fixes the crash by ensuring that the invalid reference will resolve to a variable of type any. Fixes angular#39744
@maximodee You could install the build artifacts from this PR: Here's some instructions. You need at least |
Hi
Thanks for getting back to me. I was also hesitant to apply the fixes as declared just because, as you mention they’re not intended for production usage. I took a chance based on the other comment in the bug report related to it being caused by capitalisation of Form vs form. As I’d recently updated to 11 from 8 of Angular part of the process recommendations I’d change ngForm -> ng-form and reverting this change made the error disappear.
Thanks for your help.
|
…stead of crashing (#39805) If a template declares a reference to a missing target then referring to that reference from elsewhere in the template would crash the template type checker, due to a regression introduced in #38618. This commit fixes the crash by ensuring that the invalid reference will resolve to a variable of type any. Fixes #39744 PR Close #39805
Any idea when this fix will be included as part of an update as it's causing major problems trying to debug issues caused by reorganising our modules? |
@andyb70 it would normally have been today, but there won't be a release this week due to Thanksgiving. |
@JoostK thanks for the feedback, looks like I've got a painful few days ahead of me then!! |
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. |
If a template declares a reference to a missing target then referring to
that reference from elsewhere in the template would crash the template
type checker, due to a regression introduced in #38618. This commit
fixes the crash by ensuring that the invalid reference will resolve to
a variable of type any.
Fixes #39744