NilPortugues\MessageBus\CommandBus\CommandBus

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	97.73 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	8	8	2
A	__invoke()	20	20	4

<?php

namespace NilPortugues\MessageBus\CommandBus;

use NilPortugues\Assert\Assert;
use NilPortugues\MessageBus\CommandBus\Contracts\Command;
use NilPortugues\MessageBus\CommandBus\Contracts\CommandBusMiddleware as CommandBusMiddlewareInterface;

class CommandBus implements CommandBusMiddlewareInterface

This class seems to be duplicated in your project.

{
    /** @var callable[] */
    protected $middleware = [];

    /**
     * StackedCommandBus constructor.
     *
     * @param callable[] $middleware
     */
    public function __construct(array $middleware = [])
    {
        foreach ($middleware as $commandBusMiddleware) {
            Assert::isInstanceOf($commandBusMiddleware, CommandBusMiddlewareInterface::class);
        }

        $this->middleware = $middleware;
    }

    /**
     * @param Command       $command
     * @param callable|null $next
     */
    public function __invoke(Command $command, callable $next = null)
    {
        $middleware = $this->middleware;
        $current = array_shift($middleware);

        if (empty($middleware) && !empty($current)) {
            $current->__invoke($command);

The method __invoke cannot be called on $current (of type callable).

            return;
        }

        foreach ($middleware as $commandBusMiddleware) {
            $callable = function ($command) use ($commandBusMiddleware) {
                return $commandBusMiddleware($command);
            };

            $current->__invoke($command, $callable);
            $current = $commandBusMiddleware;
        }
    }
}