Merge pull request #23 from HTMLParseErrorWG/review-fixes2

Add error for duplicate attribute
inikulin · May 30, 2017 · 8eb950a · 8eb950a
2 parents 13a204c + f6054be
commit 8eb950a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/common/error_codes.js b/lib/common/error_codes.js
@@ -47,5 +47,6 @@ module.exports = {
     undefinedCharacterReference: 'undefined-character-reference',
     missingWhitespaceBeforeDoctypeName: 'missing-whitespace-before-doctype-name',
     missingDoctypeName: 'missing-doctype-name',
-    invalidCharacterSequenceAfterDoctypeName: 'invalid-character-sequence-after-doctype-name'
+    invalidCharacterSequenceAfterDoctypeName: 'invalid-character-sequence-after-doctype-name',
+    duplicateAttribute: 'duplicate-attribute'
 };
diff --git a/lib/tokenizer/index.js b/lib/tokenizer/index.js
@@ -416,6 +416,9 @@ Tokenizer.prototype._leaveAttrName = function (toState) {
     if (Tokenizer.getTokenAttr(this.currentToken, this.currentAttr.name) === null)
         this.currentToken.attrs.push(this.currentAttr);
 
+    else
+        this._err(ERR.duplicateAttribute);
+
     this.state = toState;
 };