Apply Rector changes (CI)

src/Adapter/AdapterInterface.php

@@ -30,8 +30,6 @@ public function status(string $id): JobStatus;
 
     /**
      * Pushing a message to the queue. Adapter sets message ID if available.
-     *
-     * @param MessageInterface $message
      */
     public function push(MessageInterface $message): void;
 

src/Exception/AdapterConfiguration/ChannelIncorrectlyConfigured.php

@@ -17,9 +17,7 @@ class ChannelIncorrectlyConfigured extends InvalidArgumentException implements F
     /**
      * ChannelIncorrectlyConfigured constructor.
      *
-     * @param string $channel
      * @param mixed|object $definition
-     * @param int $code
      * @param Throwable|null $previous
      */
     public function __construct(string $channel, $definition, int $code = 0, ?Throwable $previous = null)

src/Message/Message.php

@@ -7,7 +7,6 @@
 final class Message implements MessageInterface
 {
     /**
-     * @param string $handlerName
      * @param mixed $data Message data, encodable by a queue adapter
      * @param array $metadata Message metadata, encodable by a queue adapter
      * @param string|null $id Message id

src/Middleware/Consume/MiddlewareFactoryConsume.php

@@ -87,13 +87,11 @@ private function getFromContainer(string $middlewareDefinition): MiddlewareConsu
     private function wrapCallable(callable $callback): MiddlewareConsumeInterface
     {
         return new class ($callback, $this->container) implements MiddlewareConsumeInterface {
-            private ContainerInterface $container;
             private $callback;
 
-            public function __construct(callable $callback, ContainerInterface $container)
+            public function __construct(callable $callback, private ContainerInterface $container)
             {
                 $this->callback = $callback;
-                $this->container = $container;
             }
 
             public function processConsume(ConsumeRequest $request, MessageHandlerConsumeInterface $handler): ConsumeRequest

src/Middleware/FailureHandling/FailureMiddlewareDispatcher.php

@@ -18,7 +18,6 @@ final class FailureMiddlewareDispatcher
     private array $stack = [];
 
     /**
-     * @param MiddlewareFactoryFailureInterface $middlewareFactory
      * @param array[][]|callable[][]|MiddlewareFailureInterface[][]|string[][] $middlewareDefinitions
      */
     public function __construct(

src/Middleware/FailureHandling/Implementation/ExponentialDelayMiddleware.php

@@ -28,7 +28,6 @@ final class ExponentialDelayMiddleware implements MiddlewareFailureInterface
      * @param float $delayInitial The first delay period
      * @param float $delayMaximum The maximum delay period
      * @param float $exponent Message handling delay will be increased by this multiplication each time it fails
-     * @param DelayMiddlewareInterface $delayMiddleware
      * @param QueueInterface|null $queue
      */
     public function __construct(

src/Middleware/FailureHandling/MiddlewareFactoryFailure.php

@@ -90,13 +90,11 @@ private function getFromContainer(string $middlewareDefinition): MiddlewareFailu
     private function wrapCallable(callable $callback): MiddlewareFailureInterface
     {
         return new class ($callback, $this->container) implements MiddlewareFailureInterface {
-            private ContainerInterface $container;
             private $callback;
 
-            public function __construct(callable $callback, ContainerInterface $container)
+            public function __construct(callable $callback, private ContainerInterface $container)
             {
                 $this->callback = $callback;
-                $this->container = $container;
             }
 
             public function processFailure(FailureHandlingRequest $request, MessageFailureHandlerInterface $handler): FailureHandlingRequest

src/Middleware/InvalidMiddlewareDefinitionException.php

@@ -31,14 +31,12 @@ public function __construct($middlewareDefinition, int $code = 0, ?Throwable $pr
     }
 
     /**
-     * @param mixed $middlewareDefinition
-     *
      * @return string|null
      */
     private function convertDefinitionToString(mixed $middlewareDefinition): ?string
     {
         if (is_object($middlewareDefinition)) {
-            return 'an instance of "' . get_class($middlewareDefinition) . '"';
+            return 'an instance of "' . $middlewareDefinition::class . '"';
         }
 
         if (is_string($middlewareDefinition)) {

src/Middleware/Push/MiddlewareFactoryPush.php

@@ -87,13 +87,11 @@ private function getFromContainer(string $middlewareDefinition): MiddlewarePushI
     private function wrapCallable(callable $callback): MiddlewarePushInterface
     {
         return new class ($callback, $this->container) implements MiddlewarePushInterface {
-            private ContainerInterface $container;
             private $callback;
 
-            public function __construct(callable $callback, ContainerInterface $container)
+            public function __construct(callable $callback, private ContainerInterface $container)
             {
                 $this->callback = $callback;
-                $this->container = $container;
             }
 
             public function processPush(PushRequest $request, MessageHandlerPushInterface $handler): PushRequest

src/QueueFactory.php

@@ -29,9 +29,6 @@ final class QueueFactory implements QueueFactoryInterface
      * "Definition" here is a {@see Factory} definition
      * @param QueueInterface $queue A default queue implementation. `$queue->withAdapter()` will be returned
      * with the `get` method
-     * @param ContainerInterface $container
-     * @param CallableFactory $callableFactory
-     * @param Injector $injector
      * @param bool $enableRuntimeChannelDefinition A flag whether to enable a such behavior when there is no
      * explicit channel adapter definition: `return $this->queue->withAdapter($this->adapter->withChannel($channel)`
      * When this flag is set to false, only explicit definitions from the $definition parameter are used.
@@ -71,10 +68,8 @@ public function get(string $channel = self::DEFAULT_CHANNEL_NAME): QueueInterfac
     }
 
     /**
-     * @param string $channel
      *
      * @throws ChannelIncorrectlyConfigured
-     *
      * @return QueueInterface
      */
     private function create(string $channel): QueueInterface

tests/Unit/Middleware/Consume/MiddlewareDispatcherTest.php

@@ -162,7 +162,7 @@ public function handleConsume(ConsumeRequest $request): ConsumeRequest
     private function createDispatcher(
         ContainerInterface $container = null,
     ): ConsumeMiddlewareDispatcher {
-        $container = $container ?? $this->createContainer([AdapterInterface::class => new FakeAdapter()]);
+        $container ??= $this->createContainer([AdapterInterface::class => new FakeAdapter()]);
         $callableFactory = new CallableFactory($container);
 
         return new ConsumeMiddlewareDispatcher(

tests/Unit/Middleware/FailureHandling/InvalidMiddlewareDefinitionExceptionTest.php

@@ -35,9 +35,6 @@ public function dataBase(): array
 
     /**
      * @dataProvider dataBase
-     *
-     * @param mixed $definition
-     * @param string $expected
      */
     public function testBase(mixed $definition, string $expected): void
     {
@@ -55,8 +52,6 @@ public function dataUnknownDefinition(): array
 
     /**
      * @dataProvider dataUnknownDefinition
-     *
-     * @param mixed $definition
      */
     public function testUnknownDefinition(mixed $definition): void
     {

tests/Unit/Middleware/FailureHandling/MiddlewareDispatcherTest.php

@@ -159,7 +159,7 @@ public function handleFailure(FailureHandlingRequest $request): FailureHandlingR
     private function createDispatcher(
         ContainerInterface $container = null,
     ): FailureMiddlewareDispatcher {
-        $container = $container ?? $this->createContainer([AdapterInterface::class => new FakeAdapter()]);
+        $container ??= $this->createContainer([AdapterInterface::class => new FakeAdapter()]);
         $callableFactory = new CallableFactory($container);
 
         return new FailureMiddlewareDispatcher(new MiddlewareFactoryFailure($container, $callableFactory), []);

tests/Unit/Middleware/FailureHandling/MiddlewareFactoryTest.php

@@ -154,7 +154,7 @@ public function testInvalidMiddlewareWithWrongController(): void
 
     private function getMiddlewareFactory(ContainerInterface $container = null): MiddlewareFactoryFailureInterface
     {
-        $container = $container ?? $this->getContainer([AdapterInterface::class => new FakeAdapter()]);
+        $container ??= $this->getContainer([AdapterInterface::class => new FakeAdapter()]);
 
         return new MiddlewareFactoryFailure($container, new CallableFactory($container));
     }

tests/Unit/Middleware/Push/MiddlewareFactoryTest.php

@@ -145,7 +145,7 @@ public function testInvalidMiddlewareWithWrongController(): void
 
     private function getMiddlewareFactory(ContainerInterface $container = null): MiddlewareFactoryPushInterface
     {
-        $container = $container ?? $this->getContainer([AdapterInterface::class => new FakeAdapter()]);
+        $container ??= $this->getContainer([AdapterInterface::class => new FakeAdapter()]);
 
         return new MiddlewareFactoryPush($container, new CallableFactory($container));
     }