eXpansion\Framework\Core\ChatCommand\RestartExpansion

Complexity

Total Complexity

3

Size/Duplication

Total Lines	62
Duplicated Lines	27.42 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	17	17	1
A	execute()	0	20	2

<?php

namespace eXpansion\Framework\Core\ChatCommand;

use eXpansion\Framework\AdminGroups\Helpers\AdminGroups;
use eXpansion\Framework\AdminGroups\Model\AbstractAdminChatCommand;
use eXpansion\Framework\Core\Helpers\ChatNotification;
use eXpansion\Framework\Core\Services\Application;
use eXpansion\Framework\Core\Storage\GameDataStorage;
use eXpansion\Framework\Core\Storage\PlayerStorage;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Process\Process;


/**
 * Class RestartExpansion
 *
 * @package eXpansion\Framework\Core\ChatCommand;
 * @author  oliver de Cramer <oliverde8@gmail.com>
 */
class RestartExpansion extends AbstractAdminChatCommand
{
    /** @var ChatNotification */
    protected $chatNotification;

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

    /** @var GameDataStorage */
    protected $gameData;

    /** @var Application */
    protected $application;

    public function __construct(

This method seems to be duplicated in your project.

        ChatNotification $chatNotification,
        PlayerStorage $playerStorage,
        GameDataStorage $gameData,
        Application $application,
        $command,
        string $permission,
        $aliases = [],
        AdminGroups $adminGroups
    ) {
        parent::__construct($command, $permission, $aliases, $adminGroups);

        $this->chatNotification = $chatNotification;
        $this->playerStorage = $playerStorage;
        $this->gameData = $gameData;
        $this->application = $application;
    }


    /**
     * Method called to execute the chat command.
     *
     * @param string         $login
     * @param InputInterface $input
     *
     * @return mixed
     */
    public function execute($login, InputInterface $input)
    {
        $player = $this->playerStorage->getPlayerInfo($login);

$player is not used, you could remove the assignment.

        $scriptToExecute = 'run.sh';
        if ($this->gameData->getServerOs() == GameDataStorage::OS_WINDOWS) {
            $scriptToExecute = 'run.bat';
        }

        $player = $this->playerStorage->getPlayerInfo($login);
        $this->chatNotification->sendMessage(
            'expansion_core.chat_commands.restart.message',
            null,
            ['%nickname%' => $player->getNickName()]
        );
        $this->application->stopApplication();

        $process = new Process("bin/" . $scriptToExecute . " &");
        $process->start();
    }
}