AlexCool\Rcon\Command\Wrapper\Minecraft\CoreCommandWrapper

Complexity

Total Complexity

4

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	help()	0	13	3
A	__construct()	0	4	1

<?php

namespace AlexCool\Rcon\Command\Wrapper\Minecraft;

use AlexCool\Rcon\Client\MinecraftClient;
use AlexCool\Rcon\Command\Wrapper\AbstractCommandWrapper;
use AlexCool\Rcon\Utils\CommandFormatter;

/**
 * @package AlexCool\Rcon\Command\Wrapper\Minecraft
 */
class CoreCommandWrapper extends AbstractCommandWrapper
{
    /**
     * Command list
     */
    const HELP_COMMAND = 'help';

    /**
     * @var MinecraftClient
     */
    protected $client;

    /**
     * @var CommandFormatter
     */
    protected $commandFormatter;

    /**
     * @param MinecraftClient $client
     * @param CommandFormatter $commandFormatter
     */
    public function __construct(MinecraftClient $client, CommandFormatter $commandFormatter)
    {
        $this->client = $client;
        $this->commandFormatter = $commandFormatter;
    }

    /**
     * Shows a list of server or plugin commands in the console or in-game.
     *
     * @param string|null $topic
     * @param int|null $pageNumber
     *
     * @return bool|mixed
     */
    public function help(string $topic = null, int $pageNumber = null)
    {
        $this->commandFormatter->addElement('%s', self::HELP_COMMAND);

        if ($topic) {
            $this->commandFormatter->addElement('%s', $topic);
        }

        if ($pageNumber) {

The expression $pageNumber of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            $this->commandFormatter->addElement('%u', $pageNumber);
        }

        return $this->sendCommand($this->commandFormatter->compile());
    }
}