RestartCommand   A
last analyzed

Complexity

Total Complexity 4

Size/Duplication

Total Lines 42
Duplicated Lines 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
wmc 4
eloc 14
c 1
b 0
f 0
dl 0
loc 42
rs 10

2 Methods

Rating   Name   Duplication   Size   Complexity  
A run() 0 18 3
A stop() 0 4 1
1
<?php
2
namespace PortlandLabs\Slackbot\Command;
3
4
use PortlandLabs\Slackbot\Bot;
5
use PortlandLabs\Slackbot\Command\Argument\Manager;
0 ignored issues
show
Bug introduced by
This use statement conflicts with another class in this namespace, PortlandLabs\Slackbot\Command\Manager. Consider defining an alias.

Let?s assume that you have a directory layout like this:

.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php

and let?s assume the following content of Bar.php:

// Bar.php
namespace OtherDir;

use SomeDir\Foo; // This now conflicts the class OtherDir\Foo

If both files OtherDir/Foo.php and SomeDir/Foo.php are loaded in the same runtime, you will see a PHP error such as the following:

PHP Fatal error:  Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php

However, as OtherDir/Foo.php does not necessarily have to be loaded and the error is only triggered if it is loaded before OtherDir/Bar.php, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias:

// Bar.php
namespace OtherDir;

use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore.
Loading history...
6
use PortlandLabs\Slackbot\Permission\Admin;
7
use PortlandLabs\Slackbot\Slack\Rtm\Event\Message;
8
9
class RestartCommand extends SimpleCommand
10
{
11
12
    protected $description = 'Cause the bot to shut down';
13
14
    protected $signature = 'restart {botID? : The ID of the bot to restart}';
15
16
    protected $role = Admin::class;
17
18
    /**
19
     * @param Message $message
20
     * @param Manager $arguments
21
     * @param Bot $bot
22
     *
23
     * @throws \CL\Slack\Exception\SlackException
24
     * @throws \GuzzleHttp\Exception\GuzzleException
25
     */
26
    public function run(Message $message, Manager $arguments)
27
    {
28
        // If an id is specified
29
        $passedId = $arguments->get('botID');
30
        if ($passedId && $passedId !== $this->bot->getId()) {
31
            return;
32
        }
33
34
        $text = sprintf('Okay *' . $this->bot->getId() . '* is restarting, hold on... _I was up for %s_', $this->bot->getUptime());
35
36
        // Stop after a few seconds for sure
37
        $this->bot->getLoop()->addTimer(2, function() {
38
            $this->stop();
39
        });
40
41
        // Stop once we get confirmation that our message made it
42
        $this->bot->feignTyping($message->getChannel(), $text)->done(function() {
0 ignored issues
show
Bug introduced by
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? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

42
        $this->bot->feignTyping(/** @scrutinizer ignore-type */ $message->getChannel(), $text)->done(function() {
Loading history...
43
            $this->stop();
44
        });
45
    }
46
47
    protected function stop()
48
    {
49
        $this->bot->logger()->debug('[CMD.RST] Shutting down Bot...');
50
        $this->bot->stop();
51
    }
52
}
53