Issues (37)

src/Middleware/Consume/MiddlewareConsumeStack.php (2 issues)

Labels
Severity
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Yiisoft\Queue\Middleware\Consume;
6
7
use Closure;
8
9
final class MiddlewareConsumeStack implements MessageHandlerConsumeInterface
10
{
11
    /**
12
     * Contains a stack of middleware wrapped in handlers.
13
     * Each handler points to the handler of middleware that will be processed next.
14
     *
15
     * @var MessageHandlerConsumeInterface|null stack of middleware
16
     */
17
    private ?MessageHandlerConsumeInterface $stack = null;
18
19
    /**
20
     * @param Closure[] $middlewares Middlewares.
21
     * @param MessageHandlerConsumeInterface $finishHandler Fallback handler
22
     * events.
23
     */
24 19
    public function __construct(
25
        private array $middlewares,
26
        private MessageHandlerConsumeInterface $finishHandler,
27
    ) {
28 19
    }
29
30 19
    public function handleConsume(ConsumeRequest $request): ConsumeRequest
31
    {
32 19
        if ($this->stack === null) {
33 19
            $this->build();
34
        }
35
36
        /** @psalm-suppress PossiblyNullReference */
37 19
        return $this->stack->handleConsume($request);
0 ignored issues
show
The method handleConsume() does not exist on null. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

37
        return $this->stack->/** @scrutinizer ignore-call */ handleConsume($request);

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
38
    }
39
40 19
    private function build(): void
41
    {
42 19
        $handler = $this->finishHandler;
43
44 19
        foreach ($this->middlewares as $middleware) {
45 7
            $handler = $this->wrap($middleware, $handler);
46
        }
47
48 19
        $this->stack = $handler;
49
    }
50
51
    /**
52
     * Wrap handler by middlewares.
53
     */
54 7
    private function wrap(Closure $middlewareFactory, MessageHandlerConsumeInterface $handler): MessageHandlerConsumeInterface
55
    {
56 7
        return new class ($middlewareFactory, $handler) implements MessageHandlerConsumeInterface {
57
            private ?MiddlewareConsumeInterface $middleware = null;
58
59
            public function __construct(
60
                private Closure $middlewareFactory,
61
                private MessageHandlerConsumeInterface $handler,
62
            ) {
63 7
            }
64
65
            public function handleConsume(ConsumeRequest $request): ConsumeRequest
66
            {
67 7
                if ($this->middleware === null) {
68 7
                    $this->middleware = ($this->middlewareFactory)();
69
                }
70
71 7
                return $this->middleware->processConsume($request, $this->handler);
0 ignored issues
show
The method processConsume() does not exist on null. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

71
                return $this->middleware->/** @scrutinizer ignore-call */ processConsume($request, $this->handler);

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
72
            }
73 7
        };
74
    }
75
}
76