Allow options to accepts different boolean values

If you want an option to accept `true` value and reject `false`: accepts: { boolean: [true] } If you want an option to accept any boolean value: accepts: { boolean: [true, false] }
csscomb · Jun 9, 2014 · fa20869 · fa20869
1 parent ff11557
commit fa20869
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 42 deletions.
diff --git a/lib/csscomb.js b/lib/csscomb.js
@@ -81,6 +81,8 @@ var Comb = function(config) {
 
         switch (valueType) {
             case 'boolean':
+                if (pattern.indexOf(value) < 0) throw new Error(' Value must be ' +
+                    'one of the following: ' + pattern.join(', ') + '.');
                 return value;
             case 'number':
                 if (value !== parseInt(value)) throw new Error('Value must be an integer.');

diff --git a/lib/options/always-semicolon.js b/lib/options/always-semicolon.js
@@ -1,15 +1,7 @@
 module.exports = {
     name: 'always-semicolon',
 
-    /**
-     * Sets handler value.
-     * @param {Boolean} value Option value
-     * @returns {Boolean}
-     */
-    setValue: function(value) {
-        if (value !== true) throw new Error('The option accepts only `true` value.');
-        return value;
-    },
+    accepts: { boolean: [true] },
 
     /**
      * Processes tree node.

diff --git a/lib/options/color-shorthand.js b/lib/options/color-shorthand.js
@@ -1,7 +1,7 @@
 module.exports = {
     name: 'color-shorthand',
 
-    accepts: { boolean: true },
+    accepts: { boolean: [true, false] },
 
     /**
      * Processes tree node.

diff --git a/lib/options/eof-newline.js b/lib/options/eof-newline.js
@@ -1,7 +1,7 @@
 module.exports = {
     name: 'eof-newline',
 
-    accepts: { boolean: true },
+    accepts: { boolean: [true, false] },
 
     /**
      * Processes tree node.

diff --git a/lib/options/leading-zero.js b/lib/options/leading-zero.js
@@ -1,7 +1,7 @@
 module.exports = {
     name: 'leading-zero',
 
-    accepts: { boolean: true },
+    accepts: { boolean: [true, false] },
 
     /**
      * Processes tree node.

diff --git a/lib/options/remove-empty-rulesets.js b/lib/options/remove-empty-rulesets.js
@@ -77,16 +77,7 @@ module.exports = (function() {
     return {
         name: 'remove-empty-rulesets',
 
-        /**
-         * Sets handler value.
-         *
-         * @param {Boolean} value Option value
-         * @returns {Boolean}
-         */
-        setValue: function(value) {
-            if (value !== true) throw new Error('The option accepts only `true` value.');
-            return value;
-        },
+        accepts: { boolean: [true] },
 
         /**
          * Remove rulesets with no declarations.

diff --git a/lib/options/strip-spaces.js b/lib/options/strip-spaces.js
@@ -12,16 +12,7 @@ module.exports = (function() {
     return {
         name: 'strip-spaces',
 
-        /**
-         * Sets handler value.
-         *
-         * @param {Boolean} value Option value
-         * @returns {Boolean}
-         */
-        setValue: function(value) {
-            if (value !== true) throw new Error('The option accepts only `true` value.');
-            return value;
-        },
+        accepts: { boolean: [true] },
 
         /**
          * Processes tree node.

diff --git a/lib/options/unitless-zero.js b/lib/options/unitless-zero.js
@@ -1,16 +1,7 @@
 module.exports = {
     name: 'unitless-zero',
 
-    /**
-     * Sets handler value.
-     *
-     * @param {Boolean} value Option value
-     * @returns {Boolean}
-     */
-    setValue: function(value) {
-        if (value !== true) throw new Error('The option accepts only `true` value.');
-        return value;
-    },
+    accepts: { boolean: [true] },
 
     /**
      * Processes tree node.