Remove bool arguments

Author: tommy-muehle

src/Factory/ToolFactory.php

@@ -27,14 +27,22 @@ public static function createTool($name, $directory, array $parameters)
 
         $parameters = array_merge($defaults, $parameters);
 
-        return new Tool(
+        $tool = new Tool(
             $name,
             self::getFilename($name, $directory),
             $parameters['url'],
-            $parameters['sign-url'],
-            $parameters['force-replace'],
-            $parameters['only-dev']
+            $parameters['sign-url']
         );
+
+        if (true === $parameters['force-replace']) {
+            $tool->activateForceReplace();
+        }
+
+        if (false === $parameters['only-dev']) {
+            $tool->disableOnlyDev();
+        }
+
+        return $tool;
     }
 
     /**

src/Model/Tool.php

@@ -42,17 +42,31 @@ class Tool
      * @param string $filename
      * @param string $url
      * @param string $signUrl
-     * @param bool   $forceReplace
-     * @param bool   $onlyDev
+     *
+     * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      */
-    public function __construct($name, $filename, $url, $signUrl = null, $forceReplace = false, $onlyDev = true)
+    public function __construct($name, $filename, $url, $signUrl = null)
     {
         $this->name = $name;
         $this->filename = $filename;
         $this->url = $url;
         $this->signUrl = $signUrl;
-        $this->forceReplace = $forceReplace;
-        $this->onlyDev = $onlyDev;
+    }
+
+    /**
+     * @return void
+     */
+    public function activateForceReplace()
+    {
+        $this->forceReplace = true;
+    }
+
+    /**
+     * @return void
+     */
+    public function disableOnlyDev()
+    {
+        $this->onlyDev = false;
     }
 
     /**