🐛 Fix mixin before declaration issues

Add check for declaration node content type Fixes sasstools#227 Fixes sasstools#230
donabrams · Oct 1, 2015 · 63a451e · 63a451e
1 parent 21f7821
commit 63a451e
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/lib/rules/mixins-before-declarations.js b/lib/rules/mixins-before-declarations.js
@@ -18,13 +18,22 @@ module.exports = {
       var depth = 0,
           declarationCount = [depth];
 
-      parent.traverse( function (item) {
+      parent.forEach( function (item) {
         if (item.type === 'ruleset') {
           depth++;
           declarationCount[depth] = 0;
         }
         else if (item.type === 'declaration') {
-          declarationCount[depth]++;
+          if (item.first().is('property')) {
+
+            var prop = item.first();
+
+            if(prop.first().is('ident')) {
+
+              declarationCount[depth]++;
+
+            }
+          }
         }
         else if (item.type === 'include') {
           item.forEach('simpleSelector', function (name) {