Print parser errors again

Author: MarvinKlemp

src/Violation/Renderer/Console/BaseRenderer.php

@@ -29,4 +29,40 @@ public function __construct(OutputInterface $output, MessageHelper $messageHelpe
         $this->output = $output;
         $this->messageHelper = $messageHelper;
     }
+
+    /**
+     * @param Violation[] $violations
+     * @param Error[]     $errors
+     */
+    public function renderViolations(array $violations, array $errors)
+    {
+        $this->printViolations($violations);
+        $this->printErrors($errors);
+    }
+
+    /**
+     * @param Violation[] $violations
+     */
+    protected abstract function printViolations(array $violations);
+
+    /**
+     * @param Error[] $errors
+     */
+    protected function printErrors(array $errors)
+    {
+        if (0 === count($errors)) {
+            return;
+        }
+
+        $this->output->writeln("");
+        $this->output->writeln('<error>Your project contains invalid code:</error>');
+        foreach ($errors as $error) {
+            $this->output->writeln(
+                sprintf(
+                    '<error>%s</error>',
+                    $error->getRawMessage()
+                )
+            );
+        }
+    }
 }

src/Violation/Renderer/Console/DefaultRenderer.php

@@ -24,18 +24,8 @@ public function __construct(OutputInterface $output, MessageHelper $messageHelpe
 
     /**
      * @param Violation[] $violations
-     * @param Error[]     $errors
      */
-    public function renderViolations(array $violations, array $errors)
-    {
-        $this->printViolations($violations);
-        $this->printErrors($errors);
-    }
-
-    /**
-     * @param Violation[] $violations
-     */
-    private function printViolations(array $violations)
+    protected function printViolations(array $violations)
     {
         if (0 === count($violations)) {
             return;
@@ -66,26 +56,6 @@ private function printViolations(array $violations)
         $table->render();
     }
 
-    /**
-     * @param Error[] $errors
-     */
-    private function printErrors(array $errors)
-    {
-        if (0 === count($errors)) {
-            return;
-        }
-
-        $this->output->writeln('<error>Your project contains invalid code:</error>');
-        foreach ($errors as $error) {
-            $this->output->writeln(
-                sprintf(
-                    '<error>%s</error>',
-                    $error->getRawMessage()
-                )
-            );
-        }
-    }
-
     /**
      * @param PhpFileInfo $file
      *

src/Violation/Renderer/Console/SimpleRenderer.php

@@ -20,18 +20,8 @@ public function __construct(OutputInterface $output, MessageHelper $messageHelpe
 
     /**
      * @param Violation[] $violations
-     * @param Error[]     $errors
      */
-    public function renderViolations(array $violations, array $errors)
-    {
-        $this->printViolations($violations);
-        $this->printErrors($errors);
-    }
-
-    /**
-     * @param Violation[] $violations
-     */
-    private function printViolations(array $violations)
+    protected function printViolations(array $violations)
     {
         if (0 === count($violations)) {
             return;
@@ -55,15 +45,4 @@ private function printViolations(array $violations)
             ));
         }
     }
-
-    /**
-     * @param Error[] $errors
-     */
-    private function printErrors(array $errors)
-    {
-        if (0 === count($errors)) {
-            return;
-        }
-
-    }
 }