eXpansion\Bundle\AdminChat\ChatCommand\ReasonUserCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	133
Duplicated Lines	24.06 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

87.8%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	32	32	1
A	configure()	0	11	1
B	execute()	0	24	3

<?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\Storage\PlayerStorage;
use Maniaplanet\DedicatedServer\Connection;
use Psr\Log\LoggerInterface;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;

/**
 * Class ReasonUserCommand
 *
 * @author    de Cramer Oliver<oliverde8@gmail.com>
 * @copyright 2017 eXpansion
 * @package   eXpansion\Bundle\AdminChat\ChatCommand
 */
class ReasonUserCommand extends AbstractConnectionCommand
{
    /**
     * Description of the login parameter
     *
     * @var string
     */
    protected $parameterLoginDescription;

    /**
     * Description of the reason parameter.
     *
     * @var string
     */
    protected $parameterReasonDescription;

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

    /**
     * ReasonUserCommand constructor.
     *
     * @param                  $command
     * @param string           $permission
     * @param array            $aliases
     * @param AdminGroups      $description

There is no parameter named $description. Did you maybe mean $parameterLoginDescription?

     * @param Connection       $chatMessage

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

     * @param ChatNotification $functionName
     * @param PlayerStorage    $parameterLoginDescription
     * @param LoggerInterface  $parameterReasonDescription
     * @param AdminGroups      $adminGroupsHelper
     * @param Connection       $connection
     * @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,
        $parameterLoginDescription,
        $parameterReasonDescription,
        AdminGroups $adminGroupsHelper,
        Connection $connection,
        ChatNotification $chatNotification,
        PlayerStorage $playerStorage,
        LoggerInterface $logger,
        Time $timeHelper
    ) {
        parent::__construct(
            $command,
            $permission,
            $aliases,
            $adminGroupsHelper,
            $connection,
            $chatNotification,
            $playerStorage,
            $logger,
            $timeHelper
        );

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

It seems like $functionName of type object<eXpansion\Framework\Core\Helpers\ChatNotification> is incompatible with the declared type string of property $functionName.

        $this->parameterLoginDescription = $parameterLoginDescription;

It seems like $parameterLoginDescription of type object<eXpansion\Framework\Core\Storage\PlayerStorage> is incompatible with the declared type string of property $parameterLoginDescription.

        $this->parameterReasonDescription = $parameterReasonDescription;

It seems like $parameterReasonDescription of type object<Psr\Log\LoggerInterface> is incompatible with the declared type string of property $parameterReasonDescription.

    }


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

        $this->inputDefinition->addArgument(
            new InputArgument('login', InputArgument::REQUIRED, $this->parameterLoginDescription)
        );
        $this->inputDefinition->addArgument(
            new InputArgument('reason', InputArgument::REQUIRED, $this->parameterReasonDescription)
        );
    }


    /**
     * @inheritdoc
     */
    public function execute($login, InputInterface $input)
    {
        $nickName = $this->playerStorage->getPlayerInfo($login)->getNickName();
        $playerLogin = $input->getArgument('login');
        $reason = $input->getArgument('reason');
        $group = $this->getGroupLabel($login);

        $playerNickName = $this->playerStorage->getPlayerInfo($playerLogin)->getNickName();
        try {
            $this->connection->{$this->functionName}($playerLogin, $reason);
            $this->chatNotification->sendMessage(
                $this->chatMessage,
                $this->isPublic ? null : $login,
                ['%adminLevel%' => $group, '%admin%' => $nickName, '%player%' => $playerNickName, "%reason%" => $reason]
            );
        } catch (\Exception $e) {
            $this->chatNotification->sendMessage(
                'expansion_admin_chat.dedicatedexception',
                $login,
                ['%message%' => $e->getMessage()]
            );
        }

    }
}