Closed
Bug 627304
Opened 13 years ago
Closed 13 years ago
add-on validation claims a javascript syntax error when there isn't one
Categories
(addons.mozilla.org Graveyard :: Developer Pages, defect, P3)
addons.mozilla.org Graveyard
Developer Pages
Tracking
(Not tracked)
VERIFIED
FIXED
5.12.7
People
(Reporter: als, Assigned: basta)
References
()
Details
(Whiteboard: [validator])
Attachments
(1 file)
9.12 KB,
application/octet-stream
|
Details |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-GB; rv:1.9.2) Gecko/20100115 Firefox/3.6 Build Identifier: This warning has been present for every version of the add-on. The add-on runs fine. Reproducible: Always Steps to Reproduce: 1.validate the add-on 2. 3. Actual Results: JavaScript Syntax Error Warning: A syntax error in the JavaScript halted validation of that file. Warning: Message: missing ; before statement chrome/blacken.jar/chrome/content/blacken.js 1 2 3 r gBlacken = { Expected Results: Line 3 actually has "var gBlacken = {" Perhaps the garbled report is part of the problem.
Updated•13 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Priority: -- → P3
Hardware: x86 → All
Target Milestone: --- → 5.12.7
Updated•13 years ago
|
Assignee: nobody → mbasta
Comment 1•13 years ago
|
||
Assignee | ||
Comment 2•13 years ago
|
||
Fixed: https://github.com/mattbasta/amo-validator/commit/89832f6c9d04fd1e9513b67e0c4405844574848a The problem was related: while attempting to remove BOMs (which can break cssutils), too many characters were being stripped. This is fixed now using the new textfilter module that I built for the JS tests.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 3•13 years ago
|
||
No JS errors on uploading the attached test file. Marking this as verified @ https://addons.allizom.org/en-US/developers/upload/1dea078604284dcebe5294f18137563d
Status: RESOLVED → VERIFIED
Whiteboard: [validator]
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•