PortlandLabs / slackbot

src/Command/RoleCommand.php

<?php
namespace PortlandLabs\Slackbot\Command;

use PortlandLabs\Slackbot\Command\Argument\Manager;

This use statement conflicts with another class in this namespace, PortlandLabs\Slackbot\Command\Manager. Consider defining an alias.

use PortlandLabs\Slackbot\Slack\Rtm\Event\Message;

class RoleCommand extends SimpleCommand
{

    protected $description = 'Find out what "Role" you have';

    protected $signature = 'role';

    /**
     * Handle a message
     *
     * @param Message $message
     * @param Manager $manager
     *
     * @return void
     */
    public function run(Message $message, Manager $manager)
    {
        $role = class_basename(get_class($this->checker->getRole($message)));
        $an = in_array($role[0], ['A', 'E', 'I', 'O', 'U']) && $role !== 'User' ? 'an' : 'a';

        $this->bot->feignTyping(
            $message->getChannel(),

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?

            sprintf('You are %s `%s`', $an, $role));
    }
}