ByTIC\EventDispatcher\Dispatcher\Traits\ListenForInterfacesTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getListeners()	0	7	2
A	addInterfaceListeners()	0	10	3

<?php

namespace ByTIC\EventDispatcher\Dispatcher\Traits;

/**
 * Trait ListenForInterfacesTrait
 * @package ByTIC\EventDispatcher\Dispatcher\Traits
 */
trait ListenForInterfacesTrait
{
    public function getListeners(string $eventName = null): array
    {
        $listeners = parent::getListeners($eventName);

        return class_exists($eventName, false)

It seems like $eventName can also be of type null; however, parameter $class of class_exists() does only seem to accept string, maybe add an additional type check?

            ? $this->addInterfaceListeners($eventName, $listeners)
            : $listeners;
    }

    /**
     * Add the listeners for the event's interfaces to the given array.
     *
     * @param string $eventName
     * @param array $listeners
     * @return array
     */
    protected function addInterfaceListeners($eventName, array $listeners = []): array
    {
        foreach (class_implements($eventName) as $interface) {
            $interfaceListeners = parent::getListeners($interface);
            if (count($interfaceListeners)) {
                $listeners = array_merge($listeners, $interfaceListeners);
            }
        }

        return $listeners;
    }
}