Overblog\GraphQLBundle\Event\EventDispatcherVersionHelper

Complexity

Total Complexity

4

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	isForLegacy()	0	7	2
A	dispatch()	0	6	2

<?php

namespace Overblog\GraphQLBundle\Event;

use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Symfony\Component\HttpKernel\Kernel;

/**
 * @internal
 */
final class EventDispatcherVersionHelper
{
    private static $isForLegacy;

    /**
     * @return bool
     */
    public static function isForLegacy()
    {
        if (null === self::$isForLegacy) {
            self::$isForLegacy = \version_compare(Kernel::VERSION, '4.3.0', '<');
        }

        return self::$isForLegacy;
    }

    /**
     * @param EventDispatcherInterface $dispatcher
     * @param object                   $event
     * @param string|null              $eventName
     *
     * @return object the event
     */
    public static function dispatch(EventDispatcherInterface $dispatcher, $event, $eventName)
    {
        if (self::isForLegacy()) {
            return $dispatcher->dispatch($eventName, $event);

It seems like $eventName can also be of type string; however, parameter $event of Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch() does only seem to accept object, maybe add an additional type check?

The call to Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch() has too many arguments starting with $event.

        } else {
            return $dispatcher->dispatch($event, $eventName);
        }
    }
}