Remove cruft log + fix boolean check for selectors

gardr · Mar 31, 2014 · e458e56 · leftieFriele · Mar 31, 2014 · e458e56
1 parent a20aa06
commit e458e56
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -77,8 +77,6 @@ internals.resolveIncludes = function(options){
     // todo: dependencies relative to same path? ./
     options.validate.forEach(readDependencies);
     options.preprocess.forEach(readDependencies);
-
-    console.log('options.preprocess', options.preprocess);
 };
 internals.run = function (options, callback) {
     if (!callback){

diff --git a/lib/rule/validate/css.js b/lib/rule/validate/css.js
@@ -26,9 +26,11 @@ internals.declartionVialoation = function (declarations) {
 var RE_VALID_FIRST_SELECTOR = /^[#|\.]{1}/;
 
 internals.hasSelectorViolation = function (rule) {
-    return rule && rule.selectors
+    return !!(
+        rule && rule.selectors && rule.selectors.length > 0
         && !rule.selectors[0].match(RE_VALID_FIRST_SELECTOR) &&
-        internals.declartionVialoation(rule.declarations);
+        internals.declartionVialoation(rule.declarations)
+    );
 };
 
 internals.reportErrorOnStyleContent = function (config, report) {
@@ -50,6 +52,8 @@ internals.reportErrorOnStyleContent = function (config, report) {
                 return;
             }
             var method = config.strictRules ? 'error' : 'warn';
+            console.log('internals.hasSelectorViolation(rule)', internals.hasSelectorViolation(rule));
+            console.log(rule);
             report[method]('Styling from style-tag without class or ID prefix found: \"' +
                 rule.selectors.join(', ') + '\"', {
                     'code': internals.getCode(rule)