Straight order for options

Author: mishanga

lib/csscomb.js

@@ -9,13 +9,14 @@ var vfs = require('vow-fs');
  * @name Comb
  */
 var Comb = function() {
-    this._options = {
-        'colon-space': {},
-        'rule-indent': {},
-        'sort-order': {},
-        'stick-brace': {},
-        'strip-spaces': {}
-    },
+    this._handlers;
+    this._options = [
+        'strip-spaces',
+        'stick-brace',
+        'colon-space',
+        'rule-indent',
+        'sort-order'
+    ];
     this._config = {};
     this._exclude = null;
 };
@@ -29,14 +30,18 @@ Comb.prototype = {
      * @param {Object} config
      */
     configure: function(config) {
-        for (var option in config) {
-            if (config.hasOwnProperty(option) && config[option] && this._options[option]) {
+        var _this = this;
+        this._handlers = [];
+        this._options.forEach(function(option) {
+            if (config[option]) {
                 try {
                     var handler = require('./options/' + option).setValue(config[option]);
-                    if (handler) this._config[option] = handler;
-                } catch (e) {}
+                    handler && _this._handlers.push(handler);
+                } catch (e) {
+                    throw new Error('Handler ' + option + ' was not found.');
+                }
             }
-        }
+        });
         this._exclude = (config.exclude || []).map(function(pattern) {
             return new minimatch.Minimatch(pattern);
         });
@@ -63,11 +68,9 @@ Comb.prototype = {
         node.forEach(function(node) {
             if (!Array.isArray(node)) return;
             var nodeType = node.shift();
-            for (var option in config) {
-                if (config.hasOwnProperty(option)) {
-                    config[option].process(nodeType, node);
-                }
-            }
+            _this._handlers.forEach(function(handler) {
+                handler.process(nodeType, node);
+            });
             node.unshift(nodeType);
             _this.processNode(node, level);
         });