Skip to content

/ meteor

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

Jump to bottom

HTMLTools: Error parsing script content #3797

Closed
murillo128 opened this issue Feb 23, 2015 · 3 comments
Closed

HTMLTools: Error parsing script content #3797

murillo128 opened this issue Feb 23, 2015 · 3 comments

Comments

@murillo128
Copy link
Contributor

@murillo128 murillo128 commented Feb 23, 2015 

SpacebarsCompiler.parse("<script>var x='<div>';</script>")

yields the following error:

Error: Expected "div" end tag
...=$('<div>');</script>
                        ^

(the ^ is under </ script>)
@murillo128 murillo128 changed the title HTMLJS: Error parsing script content HTMLTools: Error parsing script content Feb 23, 2015
@murillo128 murillo128 mentioned this issue Feb 24, 2015
Merged
@murillo128
Copy link
Contributor Author

@murillo128 murillo128 commented Feb 24, 2015

Sent a small PR with a workarround:
#3805
@mquandalle
Copy link
Contributor

@mquandalle mquandalle commented Feb 25, 2015

Similar to #3619.
@dgreensp dgreensp mentioned this issue Mar 3, 2015
Closed
@dgreensp
Copy link
Contributor

@dgreensp dgreensp commented Mar 3, 2015

Reproduction is in #3619.
@dgreensp dgreensp closed this in 0ede8a4 Mar 27, 2015
@glasser glasser mentioned this issue Apr 7, 2015
Closed
martijnwalraven pushed a commit that referenced this issue Feb 22, 2016
@dgreensp
Merge pull request #3805 from eface2face/issue-3797-fix
533bda6 
Parse <script> tag contents correctly (Fix #3797)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@glasser @dgreensp @murillo128 @mquandalle