Upgrade coding standard

Author: natanfelles

composer.json

@@ -34,7 +34,7 @@
     },
     "require-dev": {
         "ext-xdebug": "*",
-        "aplus/coding-standard": "^1.14",
+        "aplus/coding-standard": "^2.0",
         "ergebnis/composer-normalize": "^2.25",
         "jetbrains/phpstorm-attributes": "^1.0",
         "phpmd/phpmd": "^2.13",

src/Debug/LogCollector.php

@@ -45,7 +45,7 @@ public function getActivities() : array
 
     public function getContents() : string
     {
-        if ( ! isset($this->logger)) {
+        if (!isset($this->logger)) {
             return '<p>A Logger instance has not been set on this collector.</p>';
         }
         \ob_start(); ?>
@@ -89,7 +89,7 @@ public function getContents() : string
 
     protected function renderLogs() : string
     {
-        if ( ! $this->hasData()) {
+        if (!$this->hasData()) {
             return '<p>No log has been set.</p>';
         }
         $count = \count($this->getData());

src/Loggers/EmailLogger.php

@@ -22,7 +22,7 @@ class EmailLogger extends Logger
 {
     protected function setDestination(string $destination) : static
     {
-        if ( ! \filter_var($destination, \FILTER_VALIDATE_EMAIL)) {
+        if (!\filter_var($destination, \FILTER_VALIDATE_EMAIL)) {
             throw new InvalidArgumentException('Invalid email destination: ' . $destination);
         }
         $this->destination = $destination;
@@ -36,7 +36,7 @@ protected function makeHeaders(Log $log) : string
         foreach ($headers as $name => $value) {
             $names[] = \strtolower($name);
         }
-        if ( ! \in_array('subject', $names, true)) {
+        if (!\in_array('subject', $names, true)) {
             $headers['Subject'] = 'Log ' . $log->level->name . ' ' . $log->id;
         }
         $headerLines = [];

src/Loggers/FileLogger.php

@@ -22,7 +22,7 @@ class FileLogger extends Logger
 {
     protected function setDestination(string $destination) : static
     {
-        if ( ! \is_file($destination) && ! \is_dir(\dirname($destination))) {
+        if (!\is_file($destination) && !\is_dir(\dirname($destination))) {
             throw new InvalidArgumentException('Invalid file destination: ' . $destination);
         }
         $this->destination = $destination;

src/Loggers/MultiFileLogger.php

@@ -23,7 +23,7 @@ class MultiFileLogger extends Logger
     protected function setDestination(string $destination) : static
     {
         $directory = \realpath($destination);
-        if ( ! $directory || ! \is_dir($directory)) {
+        if (!$directory || !\is_dir($directory)) {
             throw new InvalidArgumentException('Invalid directory destination: ' . $destination);
         }
         $this->destination = $directory . \DIRECTORY_SEPARATOR;

tests/Loggers/MultiFileLoggerTest.php

@@ -17,7 +17,7 @@ final class MultiFileLoggerTest extends TestCase
     protected function setUp() : void
     {
         $destination = \sys_get_temp_dir() . '/logs';
-        if ( ! \is_dir($destination)) {
+        if (!\is_dir($destination)) {
             \mkdir($destination);
         }
         \chmod($destination, 0777);