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

Throw on {{}} #3412

Closed
vsavkin opened this issue Jul 30, 2015 · 4 comments
Closed

Throw on {{}} #3412

vsavkin opened this issue Jul 30, 2015 · 4 comments

Comments

@vsavkin
Copy link
Contributor

vsavkin commented Jul 30, 2015

The compiler should throw on {{}}.

@pkozlowski-opensource
Copy link
Member

@vsavkin were you planning to check it on the parser level (parseInterpolation) or somewhere else in the compiler (property and text parser)?

@vsavkin
Copy link
Contributor Author

vsavkin commented Jul 31, 2015

I was thinking to check it in Parser. I think that an empty input to the parser is invalid regardless of where it comes from (hostProperties, properties, text).

@pkozlowski-opensource
Copy link
Member

Right. I was actually bumping into this corner case when passing null (by accident). Agree that having an informative err message would help here.

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants