change error level

Author: yaniswang

coverage.html

@@ -14,7 +14,7 @@ HTMLHint.addRule({
             for(var i=0, l=attrs.length;i<l;i++){
                 attr = attrs[i];
                 if(attr.quote === '' && attr.value === ''){
-                    reporter.error('The attribute [ '+attr.name+' ] must set value.', event.line, col + attr.index, self, attr.raw);
+                    reporter.warn('The attribute [ '+attr.name+' ] must set value.', event.line, col + attr.index, self, attr.raw);
                 }
             }
         });

lib/htmlhint.js

@@ -9,7 +9,7 @@ HTMLHint.addRule({
         var self = this;
         function onComment(event){
             if(event.long === false && event.content.toLowerCase() !== 'doctype html'){
-                reporter.error('Doctype must be html5.', event.line, event.col, self, event.raw);
+                reporter.warn('Doctype must be html5.', event.line, event.col, self, event.raw);
             }
         }
         function onTagStart(){

src/rules/attr-value-not-empty.js

@@ -9,7 +9,7 @@ HTMLHint.addRule({
         var self = this;
         function onTagStart(event){
             if(event.tagName.toLowerCase() === 'script'){
-                reporter.error('The script tag can not be used in head.', event.line, event.col, self, event.raw);
+                reporter.warn('The script tag can not be used in head.', event.line, event.col, self, event.raw);
             }
         }
         function onTagEnd(event){

src/rules/doctype-html5.js

@@ -38,15 +38,15 @@ HTMLHint.addRule({
                     attr = attrs[i];
                     if(attr.name.toLowerCase() === 'id'){
                         if(regId.test(attr.value) === false){
-                            reporter.error(message, event.line, col + attr.index, self, attr.raw);
+                            reporter.warn(message, event.line, col + attr.index, self, attr.raw);
                         }
                     }
                     if(attr.name.toLowerCase() === 'class'){
                         var arrClass = attr.value.split(/\s+/g), classValue;
                         for(var j=0, l2=arrClass.length;j<l2;j++){
                             classValue = arrClass[j];
                             if(classValue && regId.test(classValue) === false){
-                                reporter.error(message, event.line, col + attr.index, self, classValue);
+                                reporter.warn(message, event.line, col + attr.index, self, classValue);
                             }
                         }
                     }

src/rules/head-script-disabled.js

@@ -18,7 +18,7 @@ HTMLHint.addRule({
                     }
                 }
                 if(haveAlt === false){
-                    reporter.error('Alt of img tag must be set value.', event.line, event.col, self, event.raw);
+                    reporter.warn('Alt of img tag must be set value.', event.line, event.col, self, event.raw);
                 }
             }
         });

src/rules/id-class-value.js

@@ -9,7 +9,7 @@ HTMLHint.addRule({
         var self = this;
         parser.addListener('tagstart', function(event){
             if(event.tagName.toLowerCase() === 'style'){
-                reporter.error('Style tag can not be use.', event.line, event.col, self, event.raw);
+                reporter.warn('Style tag can not be use.', event.line, event.col, self, event.raw);
             }
         });
     }

src/rules/img-alt-require.js

@@ -12,7 +12,7 @@ HTMLHint.addRule({
             var tagName = event.tagName.toLowerCase();
             if(mapEmptyTags[tagName] !== undefined){
                 if(!event.close){
-                    reporter.error('The empty tag must closed by self.', event.line, event.col, self, event.raw);
+                    reporter.warn('The empty tag must closed by self.', event.line, event.col, self, event.raw);
                 }
             }
         });

src/rules/style-disabled.js

@@ -16,6 +16,7 @@ describe('Rules: attr-value-not-empty', function(){
         expect(messages[0].rule.id).to.be('attr-value-not-empty');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(7);
+        expect(messages[0].type).to.be('warning');
     });
 
     it('Attribute value closed by quote but no value should not result in an error', function(){

src/rules/tag-self-close.js

@@ -16,6 +16,7 @@ describe('Rules: doctype-html5', function(){
         expect(messages[0].rule.id).to.be('doctype-html5');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(1);
+        expect(messages[0].type).to.be('warning');
     });
 
     it('Doctype html5 should not result in an error', function(){

test/rules/attr-value-not-empty.spec.js

@@ -16,6 +16,7 @@ describe('Rules: head-script-disabled', function(){
         expect(messages[0].rule.id).to.be('head-script-disabled');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(7);
+        expect(messages[0].type).to.be('warning');
     });
 
     it('Internal Script in head should result in an error', function(){

test/rules/doctype-html5.spec.js

@@ -16,9 +16,11 @@ describe('Rules: id-class-value', function(){
         expect(messages[0].rule.id).to.be('id-class-value');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(5);
+        expect(messages[0].type).to.be('warning');
         expect(messages[1].rule.id).to.be('id-class-value');
         expect(messages[1].line).to.be(1);
         expect(messages[1].col).to.be(17);
+        expect(messages[1].type).to.be('warning');
     });
 
     it('Id and class value be lower case and split by underline should not result in an error', function(){

test/rules/head-script-disabled.spec.js

@@ -16,6 +16,7 @@ describe('Rules: img-alt-require', function(){
         expect(messages[0].rule.id).to.be('img-alt-require');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(1);
+        expect(messages[0].type).to.be('warning');
     });
 
     it('Img tag have alt attr should not result in an error', function(){

test/rules/id-class-value.spec.js

@@ -16,6 +16,7 @@ describe('Rules: style-disabled', function(){
         expect(messages[0].rule.id).to.be('style-disabled');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(7);
+        expect(messages[0].type).to.be('warning');
     });
 
     it('Stylesheet link should not result in an error', function(){

test/rules/img-alt-require.spec.js

@@ -16,9 +16,11 @@ describe('Rules: tag-self-close', function(){
         expect(messages[0].rule.id).to.be('tag-self-close');
         expect(messages[0].line).to.be(1);
         expect(messages[0].col).to.be(1);
+        expect(messages[0].type).to.be('warning');
         expect(messages[1].rule.id).to.be('tag-self-close');
         expect(messages[1].line).to.be(1);
         expect(messages[1].col).to.be(5);
+        expect(messages[1].type).to.be('warning');
     });
 
     it('Closed empty tag should not result in an error', function(){