yswery\DNS\Event\ServerStartEvent

Complexity

Total Complexity

2

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getSocket()	0	3	1
A	__construct()	0	3	1

<?php
/*
 * This file is part of PHP DNS Server.
 *
 * (c) Yif Swery <yiftachswr@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace yswery\DNS\Event;

use React\Datagram\Socket;
use React\Datagram\SocketInterface;
use Symfony\Component\EventDispatcher\Event;

class ServerStartEvent extends Event
{
    /**
     * @var Socket
     */
    private $socket;

    /**
     * ServerStartEvent constructor.
     *
     * @param SocketInterface $socket
     */
    public function __construct(SocketInterface $socket)
    {
        $this->socket = $socket;

$socket is of type React\Datagram\SocketInterface, but the property $socket was declared to be of type React\Datagram\Socket. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * @return Socket
     */
    public function getSocket(): Socket
    {
        return $this->socket;
    }
}