lib/config: add setPluginActiveState function

geanttechnology · Jul 13, 2019 · 357de5f · 357de5f
1 parent b533063
commit 357de5f
Showing 1 changed file with 29 additions and 30 deletions.
diff --git a/lib/svgo/config.js b/lib/svgo/config.js
@@ -76,23 +76,8 @@ function preparePluginsArray(plugins) {
 
             } else {
 
-              plugin = Object.assign({}, require('../../plugins/' + key));
-
-              // name: {}
-              if (typeof item[key] === 'object') {
-                  plugin.params = Object.assign({}, plugin.params || {}, item[key]);
-                  plugin.active = true;
-
-              // name: false
-              } else if (item[key] === false) {
-                 plugin.active = false;
-
-              // name: true
-              } else if (item[key] === true) {
-                 plugin.active = true;
-              }
-
-              plugin.name = key;
+                plugin = setPluginActiveState(Object.assign({}, require('../../plugins/' + key)), item, key);
+                plugin.name = key;
             }
 
         // name
@@ -138,19 +123,7 @@ function extendConfig(defaults, config) {
                     defaults.plugins.forEach(function(plugin) {
 
                         if (plugin.name === key) {
-                            // name: {}
-                            if (typeof item[key] === 'object') {
-                                plugin.params = Object.assign({}, plugin.params || {}, item[key]);
-                                plugin.active = true;
-
-                            // name: false
-                            } else if (item[key] === false) {
-                               plugin.active = false;
-
-                            // name: true
-                            } else if (item[key] === true) {
-                               plugin.active = true;
-                            }
+                            plugin = setPluginActiveState(plugin, item, key);
                         }
                     });
                 }
@@ -212,3 +185,29 @@ function optimizePluginsArray(plugins) {
     }, []);
 
 }
+
+/**
+ * Sets plugin to active or inactive state.
+ *
+ * @param {Object} plugin
+ * @param {Object} item
+ * @param {Object} key
+ * @return {Object} plugin
+ */
+function setPluginActiveState(plugin, item, key) {
+    // name: {}
+    if (typeof item[key] === 'object') {
+        plugin.params = Object.assign({}, plugin.params || {}, item[key]);
+        plugin.active = true;
+
+    // name: false
+    } else if (item[key] === false) {
+        plugin.active = false;
+
+    // name: true
+    } else if (item[key] === true) {
+        plugin.active = true;
+    }
+
+    return plugin;
+}