Gears\Event\Symfony\EventBus

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	dispatch()	0	4	1

<?php

/*
 * event-symfony-event-dispatcher (https://github.com/phpgears/event-symfony-event-dispatcher).
 * Event bus with Symfony Event Dispatcher.
 *
 * @license MIT
 * @link https://github.com/phpgears/event-symfony-event-dispatcher
 * @author Julián Gutiérrez <juliangut@gmail.com>
 */

declare(strict_types=1);

namespace Gears\Event\Symfony\Dispatcher;

use Gears\Event\Event;
use Gears\Event\EventBus as EventBusInterface;

final class EventBus implements EventBusInterface
{
    /**
     * Wrapped event dispatcher.
     *
     * @var ContainerAwareDispatcher
     */
    private $wrappedDispatcher;

    /**
     * EventBus constructor.
     *
     * @param ContainerAwareDispatcher $wrappedDispatcher

Should the type for parameter $wrappedDispatcher not be EventDispatcher?

     */
    public function __construct(EventDispatcher $wrappedDispatcher)
    {
        $this->wrappedDispatcher = $wrappedDispatcher;

$wrappedDispatcher is of type object<Gears\Event\Symfony\Dispatcher\EventDispatcher>, but the property $wrappedDispatcher was declared to be of type object<Gears\Event\Symfony\Dispatcher\ContainerAwareDispatcher>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * {@inheritdoc}
     */
    public function dispatch(Event $event): void
    {
        $this->wrappedDispatcher->dispatch(\get_class($event), new EventEnvelope($event));
    }
}