eXpansion\Bundle\AdminChat\ChatCommand\AbstractConnectionCommand

Complexity

Total Complexity

1

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	18	1

<?php

namespace eXpansion\Bundle\AdminChat\ChatCommand;

use eXpansion\Framework\AdminGroups\Helpers\AdminGroups;
use eXpansion\Framework\AdminGroups\Model\AbstractAdminChatCommand;
use eXpansion\Framework\Core\Helpers\ChatNotification;
use eXpansion\Framework\Core\Storage\PlayerStorage;
use Maniaplanet\DedicatedServer\Connection;
use Monolog\Logger;
use Psr\Log\LoggerInterface;

/**
 * Class Restart
 *
 * @package eXpansion\Bundle\AdminChat\ChatCommand;
 * @author oliver de Cramer <oliverde8@gmail.com>
 */
abstract class AbstractConnectionCommand extends AbstractAdminChatCommand
{
    /** @var Connection  */
    protected $connection;

    /** @var ChatNotification  */
    protected $chatNotification;

    /** @var PlayerStorage  */
    protected $playerStorage;

    /** @var Logger */
    protected $logger;

    public function __construct(
        $command,
        $permission,
        array $aliases = [],
        AdminGroups $adminGroupsHelper,
        Connection $connection,
        ChatNotification $chatNotification,
        PlayerStorage $playerStorage,
        LoggerInterface $logger
    )
    {
        parent::__construct($command, $permission, $aliases, $adminGroupsHelper);

        $this->connection = $connection;
        $this->chatNotification = $chatNotification;
        $this->playerStorage = $playerStorage;
        $this->logger = $logger;

$logger is of type object<Psr\Log\LoggerInterface>, but the property $logger was declared to be of type object<Monolog\Logger>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }
}