Bugzilla – Full Text Bug Listing
| Summary: | [tern] ESLint validation no longer runs on HTML files with <script> blocks | ||
|---|---|---|---|
| Product: | [ECD] Orion | Reporter: | Curtis Windatt <curtis.windatt.public> |
| Component: | JS Tools | Assignee: | Olivier Thomann <Olivier_Thomann> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | Michael_Rennie |
| Version: | 10.0 | ||
| Target Milestone: | 11.0 | ||
| Hardware: | PC | ||
| OS: | Windows 7 | ||
| Whiteboard: | |||
This will be fixed as part of the fix for bug 483607. I need to fix this in order to pass all tests after my changes for that bug report. This is fixed with commit c48f2eac25f45e245dc864bd7e58bead36a0d97b. |
<html><head><script>function a(){}; var xx = 0;;;; a(); this.x;</script></head></html> This should give me an ESLint warning about multiple semi-colons. There are similar tests in validatorTests that are passing. A quick debug shows the file.ast we are using to validate has no body (empty Program node). Looks like our other tools are still working inside script blocks. Another case to test: <html><body onload="foo();;;"></body></html>