Overblog\GraphQLBundle\Event\EventDispatcherVersionHelper

Complexity

Total Complexity

3

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

83.33%

Importance

Changes

0

2 Methods

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

<?php

namespace Overblog\GraphQLBundle\Event;

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

/**
 * @internal
 * TODO(mcg-web): delete hack after migrating Symfony >= 4.3
 */
final class EventDispatcherVersionHelper
{
    /**
     * @return bool
     */
    public static function isForLegacy()
    {
        return Kernel::VERSION_ID < 40300;
    }

    /**
     * @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);

The call to Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch() has too many arguments starting with $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?

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