Validated option values

Author: mishanga

lib/csscomb.js

@@ -33,9 +33,7 @@ Comb.prototype = {
             if (config.hasOwnProperty(option) && config[option] && this._options[option]) {
                 var handler;
                 try {
-                    handler = require('./options/' + option);
-                    handler.value = config[option];
-                    this._config[option] = handler;
+                    this._config[option] = require('./options/' + option).setValue(config[option]);
                 } catch (e) {}
             }
         }

lib/options/colon-space.js

@@ -1,6 +1,22 @@
 module.exports = {
 
-    value: false,
+    _value: false,
+
+    /**
+     * Sets handler value.
+     *
+     * @param {String|Boolean} value Option value
+     * @returns {Object}
+     */
+    setValue: function(value) {
+        if (!value) return this;
+        if (typeof value === 'string') {
+            this._value = value;
+        } else {
+            this._value = 'after'; // default
+        }
+        return this;
+    },
 
     /**
      * Processes tree node.
@@ -10,12 +26,12 @@ module.exports = {
     process: function(nodeType, node) {
         if (nodeType === 'property') {
             if (node[node.length - 1][0] === 's') node.pop();
-            if (this.value === 'both' || this.value === 'before')
+            if (this._value === 'both' || this._value === 'before')
                 node.push(['s', ' ']);
         }
         if (nodeType === 'value') {
             if (node[0][0] === 's') node.shift();
-            if (this.value === 'both' || this.value === 'after')
+            if (this._value === 'both' || this._value === 'after')
                 node.unshift(['s', ' ']);
         }
     }

lib/options/strip-spaces.js

@@ -2,6 +2,17 @@ module.exports = {
 
     value: false,
 
+    /**
+     * Sets handler value.
+     *
+     * @param {String|Boolean} value Option value
+     * @returns {Object}
+     */
+    setValue: function(value) {
+        this._value = !!value;
+        return this;
+    },
+
     /**
      * Processes tree node.
      * @param {String} nodeType