PortlandLabs / slackbot

src/Command/UptimeCommand.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 UptimeCommand extends SimpleCommand
{

    protected $description = 'Find out when I woke up';

    protected $signature = 'uptime {--t|timestamp : Show as seconds since I started}';

    /**
     * Handle a message
     *
     * @param Message $message
     * @param Manager $manager
     *
     * @return void
     */
    public function run(Message $message, Manager $manager)
    {
        if ($manager && $manager->get('timestamp')) {
            $start = $this->bot->getConnectedTime();
            $now = new \DateTime();
            $this->bot->feignTyping($message->getChannel(), $now->getTimestamp() - $start->getTimestamp());

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?

            return;
        }

        $this->bot->feignTyping($message->getChannel(), $this->bot->getUptime());
    }
}