Add error severity levels

src/Error.php

@@ -11,14 +11,18 @@ class Error implements \JsonSerializable
     /** @var string */
     protected $message;
 
+    /** @var string */
+    protected $severity;
+
     /**
      * @param string $filePath
      * @param string $message
      */
-    public function __construct($filePath, $message)
+    public function __construct($filePath, $message, $severity = 'error')
     {
         $this->filePath = $filePath;
         $this->message = rtrim($message);
+        $this->severity = $severity;
     }
 
     /**
@@ -29,6 +33,14 @@ public function getMessage()
         return $this->message;
     }
 
+    /**
+     * @return string
+     */
+    public function getSeverity()
+    {
+        return $this->severity;
+    }
+
     /**
      * @return string
      */
@@ -63,7 +75,7 @@ public function getShortFilePath()
     public function jsonSerialize()
     {
         return array(
-            'type' => 'error',
+            'type' => $this->getSeverity(),
             'file' => $this->getFilePath(),
             'message' => $this->getMessage(),
         );
@@ -221,6 +233,7 @@ public function jsonSerialize()
             'message' => $this->getMessage(),
             'normalizeMessage' => $this->getNormalizedMessage(),
             'blame' => $this->blame,
+            'severity' => $this->getSeverity(),
         );
     }
 }

src/Output.php

@@ -422,15 +422,18 @@ public function writeResult(Result $result, ErrorFormatter $errorFormatter, $ign
             if ($error instanceof SyntaxError) {
                 $line = $error->getLine();
                 $source = "Syntax Error";
+                $severity = $error->getSeverity();
             } else {
                 $line = 1;
                 $source = "Linter Error";
+                $severity = 'error';
             }
 
             $errors[$error->getShortFilePath()][] = array(
                 'message' => $message,
                 'line' => $line,
-                'source' => $source
+                'source' => $source,
+                'severity' => $severity,
             );
         }
 
@@ -439,8 +442,9 @@ public function writeResult(Result $result, ErrorFormatter $errorFormatter, $ign
             foreach ($fileErrors as $fileError) {
                 $this->writer->write(
                     sprintf(
-                        '        <error line="%d" severity="ERROR" message="%s" source="%s" />',
+                        '        <error line="%d" severity="%s" message="%s" source="%s" />',
                         $fileError['line'],
+                        $fileError['severity'],
                         $fileError['message'],
                         $fileError['source']
                     ) .

src/ParallelLint.php

@@ -96,7 +96,7 @@ public function lint(array $files)
 
                     } else if ($process->containsError()) {
                         $checkedFiles[] = $file;
-                        $errors[] = $this->triggerSyntaxErrorCallback(new SyntaxError($file, $process->getSyntaxError()));
+                        $errors[] = $this->triggerSyntaxErrorCallback(new SyntaxError($file, $process->getSyntaxError(), $process->getSeverity()));
                         $processCallback(self::STATUS_ERROR, $file);
 
                     } else if ($process->isSuccess()) {
@@ -135,7 +135,7 @@ public function lint(array $files)
 
                 } else if ($process->containsError()) {
                     $checkedFiles[] = $file;
-                    $errors[] = $this->triggerSyntaxErrorCallback(new SyntaxError($file, $process->getSyntaxError()));
+                    $errors[] = $this->triggerSyntaxErrorCallback(new SyntaxError($file, $process->getSyntaxError(), $process->getSeverity()));
                     $processCallback(self::STATUS_ERROR, $file);
 
                 } else {

src/Process/LintProcess.php

@@ -14,6 +14,11 @@ class LintProcess extends PhpProcess
      */
     private $showDeprecatedErrors;
 
+    /**
+     * @var string
+     */
+    protected $severity;
+
     /**
      * @param PhpExecutable $phpExecutable
      * @param string $fileToCheck Path to file to check
@@ -62,20 +67,23 @@ public function getSyntaxError()
             // Look for fatal errors first
             foreach (explode("\n", $this->getOutput()) as $line) {
                 if ($this->containsFatalError($line)) {
+                    $this->severity = 'error';
                     return $line;
                 }
             }
 
             // Look for parser errors second
             foreach (explode("\n", $this->getOutput()) as $line) {
                 if ($this->containsParserError($line)) {
+                    $this->severity = 'error';
                     return $line;
                 }
             }
 
             // Look for deprecated errors third
             foreach (explode("\n", $this->getOutput()) as $line) {
                 if ($this->containsDeprecatedError($line)) {
+                    $this->severity = 'warning';
                     return $line;
                 }
             }
@@ -86,6 +94,19 @@ public function getSyntaxError()
         return false;
     }
 
+    /**
+     * @return string
+     */
+    public function getSeverity()
+    {
+        if (!empty($this->severity)) {
+            return $this->severity;
+        } else {
+            $error = $this->getSyntaxError();
+            return $this->severity;
+        }
+    }
+
     /**
      * @return bool
      * @throws RunTimeException