Issue #78: added option to force colors

Author: phpjsdev623

parallel-lint.php

@@ -28,6 +28,7 @@ function showOptions()
     --exclude       Exclude directory. If you want exclude multiple directories, use
                     multiple exclude parameters.
     -j <num>        Run <num> jobs in parallel (default: 10).
+    --colors        Enable colors in console output. (disables auto detection of color support)
     --no-colors     Disable colors in console output.
     --json          Output results as JSON string (require PHP 5.4).
     --blame         Try to show git blame for row with error.

src/ErrorFormatter.php

@@ -38,12 +38,16 @@ class ErrorFormatter
     /** @var bool */
     private $useColors;
 
+    /** @var bool */
+    private $forceColors;
+
     /** @var bool */
     private $translateTokens;
 
-    public function __construct($useColors = false, $translateTokens = false)
+    public function __construct($useColors = false, $translateTokens = false, $forceColors = false)
     {
         $this->useColors = $useColors;
+        $this->forceColors = $forceColors;
         $this->translateTokens = $translateTokens;
     }
 
@@ -143,10 +147,10 @@ protected function getColoredCodeSnippet($filePath, $lineNumber, $linesBefore =
         }
 
         $colors = new ConsoleColor();
-        $colors->setForceStyle(true);
+        $colors->setForceStyle($this->forceColors);
         $highlighter = new Highlighter($colors);
 
         $fileContent = file_get_contents($filePath);
         return $highlighter->getCodeSnippet($fileContent, $lineNumber, $linesBefore, $linesAfter);
     }
-}
+}

src/Manager.php

@@ -107,7 +107,7 @@ protected function getDefaultOutput(Settings $settings)
         if ($settings->json) {
             return new JsonOutput($writer);
         } else {
-            return ($settings->colors ? new TextOutputColored($writer) : new TextOutput($writer));
+            return ($settings->colors ? new TextOutputColored($writer, $settings->forceColors) : new TextOutput($writer));
         }
     }
 

src/Output.php

@@ -320,13 +320,13 @@ class TextOutputColored extends TextOutput
     /** @var \JakubOnderka\PhpConsoleColor\ConsoleColor */
     private $colors;
 
-    public function __construct(IWriter $writer)
+    public function __construct(IWriter $writer, $forceColors = false)
     {
         parent::__construct($writer);
 
         if (class_exists('\JakubOnderka\PhpConsoleColor\ConsoleColor')) {
             $this->colors = new \JakubOnderka\PhpConsoleColor\ConsoleColor();
-            $this->colors->setForceStyle(true);
+            $this->colors->setForceStyle($forceColors);
         }
     }
 

src/Settings.php

@@ -80,6 +80,12 @@ class Settings
      */
     public $colors = true;
 
+    /**
+     * Do not autodetect if colors are available
+     * @var bool
+     */
+    public $forceColors = false;
+
     /**
      * Output results as JSON string
      * @var bool
@@ -158,6 +164,11 @@ public static function parseArguments(array $arguments)
                         $settings->parallelJobs = max((int) $arguments->getNext(), 1);
                         break;
 
+                    case '--colors':
+                        $settings->forceColors = true;
+                        $settings->colors = true;
+                        break;
+
                     case '--no-colors':
                         $settings->colors = false;
                         break;