added rector

Author: limenet

composer.json

@@ -22,14 +22,16 @@
         "spatie/laravel-package-tools": "^1.13.0"
     },
     "require-dev": {
+        "driftingly/rector-laravel": "^0.17.0",
         "laravel/pint": "^1.7",
         "nunomaduro/collision": "^7.3.3",
         "nunomaduro/larastan": "^2.5.1",
         "orchestra/testbench": "^8.0.11",
         "phpstan/extension-installer": "^1.2",
         "phpstan/phpstan-deprecation-rules": "^1.1.3",
         "phpstan/phpstan-phpunit": "^1.3.10",
-        "phpunit/phpunit": "^10.0.18"
+        "phpunit/phpunit": "^10.0.18",
+        "rector/rector": "^0.15.23"
     },
     "autoload": {
         "psr-4": {
@@ -52,7 +54,8 @@
         "analyse": "vendor/bin/phpstan analyse",
         "test": "vendor/bin/phpunit",
         "test-coverage": "vendor/bin/phpunit --coverage",
-        "format": "vendor/bin/pint"
+        "format": "vendor/bin/pint",
+        "rector": "vendor/bin/rector process"
     },
     "config": {
         "sort-packages": true,

rector.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Set\ValueObject\LevelSetList;
+use RectorLaravel\Set\LaravelSetList;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->paths([
+        __DIR__.'/config',
+        __DIR__.'/src',
+        __DIR__.'/tests',
+    ]);
+
+    $rectorConfig->sets([
+        LevelSetList::UP_TO_PHP_81,
+        LaravelSetList::LARAVEL_100,
+    ]);
+};

src/Client/ElasticaClient.php

@@ -43,8 +43,6 @@ public function disableEventListener(): void
 
     public function listensToEvents(): bool
     {
-        return self::$listenToEvents !== null
-            ? self::$listenToEvents
-            : config('elastica-bridge.events.listen', true);
+        return self::$listenToEvents ?? config('elastica-bridge.events.listen', true);
     }
 }

src/Commands/StatusCommand.php

@@ -57,7 +57,7 @@ public function handle(): int
             if ($hasBlueGreen) {
                 try {
                     $activeBlueGreen = $indexConfig->getBlueGreenActiveElasticaIndex()->getName();
-                } catch (BlueGreenIndicesIncorrectlySetupException $exception) {
+                } catch (BlueGreenIndicesIncorrectlySetupException) {
                     $hasBlueGreen = false;
                 }
             }

src/Events/EventHandler.php

@@ -10,8 +10,8 @@
 class EventHandler
 {
     public function __construct(
-        private ElasticaClient $elasticaClient,
-        private ModelEventListener $modelEventListener
+        private readonly ElasticaClient $elasticaClient,
+        private readonly ModelEventListener $modelEventListener
     ) {
     }
 

src/Index/AbstractIndex.php

@@ -172,7 +172,7 @@ final public function getBlueGreenInactiveElasticaIndex(): Index
 
     final public function indexingLock(): Lock
     {
-        return Cache::lock(__CLASS__.$this->getName());
+        return Cache::lock(self::class.$this->getName());
     }
 
     public function ingoreIndexingErrors(): bool

src/Jobs/SetupIndex.php

@@ -83,7 +83,7 @@ private function setup(ElasticaClient $elastica): void
     {
         try {
             $this->indexConfig->getBlueGreenActiveSuffix();
-        } catch (BlueGreenIndicesIncorrectlySetupException $exception) {
+        } catch (BlueGreenIndicesIncorrectlySetupException) {
             $elastica->getIndex($this->indexConfig->getName().IndexInterface::INDEX_SUFFIX_BLUE)->addAlias($this->indexConfig->getName());
         }
     }

src/Services/ModelEventListener.php

@@ -22,7 +22,7 @@ class ModelEventListener
 
     private const EVENT_DELETED = 'deleted';
 
-    public const EVENTS = [
+    final public const EVENTS = [
         self::EVENT_CREATED,
         self::EVENT_UPDATED,
         self::EVENT_SAVED,