eXpansion\Bundle\AdminChat\ChatCommand\OneParameterCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	117
Duplicated Lines	46.15 %

Coupling/Cohesion

Components	2
Dependencies	10

Test Coverage

Coverage

86.67%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	8	1
A	execute()	24	24	3
A	__construct()	30	30	1

<?php

namespace eXpansion\Bundle\AdminChat\ChatCommand;

use eXpansion\Framework\AdminGroups\Helpers\AdminGroups;
use eXpansion\Framework\Core\Helpers\ChatNotification;
use eXpansion\Framework\Core\Helpers\Time;
use eXpansion\Framework\Core\Helpers\TMString;
use eXpansion\Framework\Core\Services\DedicatedConnection\Factory;
use eXpansion\Framework\Core\Storage\PlayerStorage;
use Maniaplanet\DedicatedServer\Connection;
use Maniaplanet\DedicatedServer\Xmlrpc\Exception as DedicatedException;
use Psr\Log\LoggerInterface;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;

/**
 * Class ReasonUserCommand
 *
 * @author  Reaby
 * @package eXpansion\Bundle\AdminChat\ChatCommand
 */
class OneParameterCommand extends AbstractConnectionCommand
{
    /**
     * Description of the parameter
     *
     * @var string
     */
    protected $parameterDescription;

    /**
     * Description of the command.
     *
     * @var string
     */
    protected $description;

    /**
     * Message to display in chat.
     *
     * @var string
     */
    protected $chatMessage;

    /**
     * Name of the dedicated function to call.
     *
     * @var string
     */
    protected $functionName;

    /**
     * OneParameterCommand constructor.
     *
     * @param                  $command
     * @param string           $permission
     * @param array            $aliases
     * @param string           $functionName
     * @param string           $parameterDescription
     * @param AdminGroups      $adminGroupsHelper
     * @param Connection       $connection

There is no parameter named $connection. Was it maybe removed?

     * @param ChatNotification $chatNotification
     * @param PlayerStorage    $playerStorage
     * @param LoggerInterface  $logger
     * @param Time             $timeHelper
     */
    public function __construct(

This method seems to be duplicated in your project.

        $command,
        $permission,
        array $aliases = [],
        $functionName,
        $parameterDescription,
        AdminGroups $adminGroupsHelper,
        Factory $factory,
        ChatNotification $chatNotification,
        PlayerStorage $playerStorage,
        LoggerInterface $logger,
        Time $timeHelper
    ) {
        parent::__construct(
            $command,
            $permission,
            $aliases,
            $adminGroupsHelper,
            $factory,
            $chatNotification,
            $playerStorage,
            $logger,
            $timeHelper
        );

        $this->description = 'expansion_admin_chat.'.strtolower($functionName).'.description';
        $this->chatMessage = 'expansion_admin_chat.'.strtolower($functionName).'.msg';
        $this->functionName = (string) $functionName;
        $this->parameterDescription = (string) $parameterDescription;
    }


    /**
     * @inheritdoc
     */
    protected function configure()
    {
        parent::configure();

        $this->inputDefinition->addArgument(
            new InputArgument('parameter', InputArgument::REQUIRED, $this->parameterDescription)
        );
    }

    /**
     * @inheritdoc
     */
    public function execute($login, InputInterface $input)

This method seems to be duplicated in your project.

    {
        $nickName = $this->playerStorage->getPlayerInfo($login)->getNickName();
        $parameter = $input->getArgument('parameter');
        $group = $this->getGroupLabel($login);
        try {
            $this->factory->getConnection()->{$this->functionName}($parameter);
            $this->chatNotification->sendMessage(
                $this->chatMessage,
                $this->isPublic ? null : $login,
                ['%adminLevel%' => $group, '%admin%' => $nickName, "%parameter%" => $parameter]
            );

            $logMessage = $this->chatNotification->getMessage($this->chatMessage,
                ['%adminLevel%' => $group, '%admin%' => $nickName, "%parameter%" => $parameter], "en");
            $this->logger->info("[".$login."] ".TMString::trimStyles($logMessage));

        } catch (DedicatedException $e) {
            $this->logger->error("Error on admin command", ["exception" => $e]);
            $this->chatNotification->sendMessage("expansion_admin_chat.dedicatedexception", $login,
                ["%message%" => $e->getMessage()]);
        }

    }
}