PortlandLabs\Slackbot\Command\HelpCommand::run()

Complexity

Conditions	7
Paths	6

Size

Total Lines	41
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function run(Message $message, ArgumentManager $manager)
    {
        $output = [];

        $filter = new Filter();
        $summary = new Summary();

        $userRole = $this->checker->getRole($message);

        $commandManager = $this->bot->commands();
        $commandName = $manager->get('command');

        $output[] = 'You can use the following commands:';

        foreach ($commandManager->all() as $command) {
            if (!$command instanceof ConsoleStyleCommand) {
                continue;
            }

            $arguments = new ArgumentManager();
            $arguments = $command->configure($arguments);
            $commandRole = $arguments->getRole();

            if (!$userRole instanceof $commandRole) {
                continue;
            }

            if ($commandName && $arguments->getCommand() === $commandName) {
                $this->outputUsage($message, $arguments, $command);
                return;
            }

            $filter->setArguments($arguments->all());

            // Output the simple usage statement
            $orderedArguments = array_merge($filter->withPrefix(), $filter->withoutPrefix());
            $short = $summary->short($orderedArguments);
            $output[] = "• `{$arguments->getCommand()}" . ($short ? " {$short}`" : '`') . ' ' . $command->getDescription();
        }

        $this->bot->feignTyping($message->getChannel(), implode("\n", $output));

It seems like $message->getChannel() can also be of type null; however, parameter $channel of PortlandLabs\Slackbot\Bot::feignTyping() does only seem to accept string, maybe add an additional type check?

    }
}