Add Rector and apply basic rules

Author: devnix

composer.json

@@ -26,7 +26,8 @@
         "squizlabs/php_codesniffer": "^3.0",
         "brick/varexporter": "^0.3.5",
         "friendsofphp/php-cs-fixer": "^3.2",
-        "oscarotero/php-cs-fixer-config": "^2.0"
+        "oscarotero/php-cs-fixer-config": "^2.0",
+        "rector/rector": "^2.0"
     },
     "autoload": {
         "psr-4": {
@@ -41,8 +42,10 @@
     "scripts": {
         "test": [
             "phpunit",
-            "phpcs"
+            "phpcs",
+            "rector process --dry-run"
         ],
-        "cs-fix": "php-cs-fixer fix"
+        "cs-fix": "php-cs-fixer fix",
+        "rector": "rector process"
     }
 }

rector.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Set\ValueObject\SetList;
+
+return RectorConfig::configure()
+    ->withPaths([
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+    ->withPhpSets()
+    ->withTypeCoverageLevel(1)
+    ->withDeadCodeLevel(2)
+    ->withCodeQualityLevel(10)
+    ->withCodingStyleLevel(0)
+;

src/Loader/MoLoader.php

@@ -63,7 +63,7 @@ public function loadString(string $string, ?Translations $translations = null):
                     }
 
                     $headerChunks = preg_split('/:\s*/', $headerLine, 2);
-                    $translations->getHeaders()->set($headerChunks[0], isset($headerChunks[1]) ? $headerChunks[1] : '');
+                    $translations->getHeaders()->set($headerChunks[0], $headerChunks[1] ?? '');
                 }
 
                 continue;
@@ -73,13 +73,13 @@ public function loadString(string $string, ?Translations $translations = null):
             $chunks = explode("\x04", $original, 2);
 
             if (isset($chunks[1])) {
-                list($context, $original) = $chunks;
+                [$context, $original] = $chunks;
             }
 
             $chunks = explode("\x00", $original, 2);
 
             if (isset($chunks[1])) {
-                list($original, $plural) = $chunks;
+                [$original, $plural] = $chunks;
             }
 
             $translation = $this->createTranslation($context, $original, $plural);

src/Loader/PoLoader.php

@@ -174,7 +174,7 @@ private static function parseHeaders(?string $string): array
             // Useful for distinguishing between header definitions and possible continuations of a header entry.
             if (preg_match('/^[\w-]+:/', $line)) {
                 $pieces = array_map('trim', explode(':', $line, 2));
-                list($name, $value) = $pieces;
+                [$name, $value] = $pieces;
 
                 $headers[$name] = $value;
                 continue;

src/Loader/StrictPoLoader.php

@@ -384,7 +384,7 @@ private function readPluralTranslation(bool $throwIfNotFound = false): bool
         if (($translation = $this->translation->getTranslation()) !== null) {
             array_unshift($translations, $translation);
         }
-        if (count($translations) !== (int) $index) {
+        if (count($translations) !== $index) {
             throw new Exception("The msgstr has an invalid index{$this->getErrorPosition()}");
         }
         $data = $this->readQuotedString('msgstr');

src/Scanner/FunctionsHandlersTrait.php

@@ -15,7 +15,7 @@ protected function gettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 1)) {
             return null;
         }
-        list($original) = $function->getArguments();
+        [$original] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -30,7 +30,7 @@ protected function ngettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 2)) {
             return null;
         }
-        list($original, $plural) = $function->getArguments();
+        [$original, $plural] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -45,7 +45,7 @@ protected function pgettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 2)) {
             return null;
         }
-        list($context, $original) = $function->getArguments();
+        [$context, $original] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -60,7 +60,7 @@ protected function dgettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 2)) {
             return null;
         }
-        list($domain, $original) = $function->getArguments();
+        [$domain, $original] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -75,7 +75,7 @@ protected function dpgettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 3)) {
             return null;
         }
-        list($domain, $context, $original) = $function->getArguments();
+        [$domain, $context, $original] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -90,7 +90,7 @@ protected function npgettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 3)) {
             return null;
         }
-        list($context, $original, $plural) = $function->getArguments();
+        [$context, $original, $plural] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -105,7 +105,7 @@ protected function dngettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 3)) {
             return null;
         }
-        list($domain, $original, $plural) = $function->getArguments();
+        [$domain, $original, $plural] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,
@@ -120,7 +120,7 @@ protected function dnpgettext(ParsedFunction $function): ?Translation
         if (!$this->checkFunction($function, 4)) {
             return null;
         }
-        list($domain, $context, $original, $plural) = $function->getArguments();
+        [$domain, $context, $original, $plural] = $function->getArguments();
 
         $translation = $this->addComments(
             $function,

src/Scanner/ParsedFunction.php

@@ -21,7 +21,7 @@ public function __construct(string $name, string $filename, int $line, ?int $las
         $this->name = $name;
         $this->filename = $filename;
         $this->line = $line;
-        $this->lastLine = isset($lastLine) ? $lastLine : $line;
+        $this->lastLine = $lastLine ?? $line;
     }
 
     public function __debugInfo()

tests/MoGeneratorTest.php

@@ -26,7 +26,7 @@ public function testMoGenerator()
         $translation->translate('Orixinal');
         $translations->add($translation);
 
-        $translation = Translation::create('context-1', 'Other comment', 'Other comments');
+        $translation = Translation::create('context-1', 'Other comment');
         $translation->translate('Outro comentario');
         $translation->translatePlural('Outros comentarios');
         $translations->add($translation);

tests/PoGeneratorTest.php

@@ -33,7 +33,7 @@ public function testPoLoader()
         $translation->getReferences()->add('/my/template.php', 45);
         $translations->add($translation);
 
-        $translation = Translation::create('context-1', 'Other comment', 'Other comments');
+        $translation = Translation::create('context-1', 'Other comment');
         $translation->translate('Outro comentario');
         $translation->translatePlural('Outros comentarios');
         $translation->getExtractedComments()->add('Not sure about this');