Slackbot::getCommandObjectByMessage() - Code Metrics - Inspection of "Simplify get command from message" - iranianpep/botonomous - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5712f6...cce3fc )

by Ehsan

created 2017-07-13 09:39 UTC

Slackbot::getCommandObjectByMessage() A

↳ Parent: Slackbot

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	17
ccs	8
cts	8
cp	1
rs	9.4285
cc	3
eloc	8
nc	3
nop	1
crap	3

<?php

namespace Botonomous;

use Botonomous\listener\EventListener;
use Botonomous\plugin\AbstractPlugin;

/**
 * Class Botonomous.
 */
class Slackbot extends AbstractBot
{
    private $commands;
    private $lastError;
    private $currentCommand;

    /**
     * Botonomous constructor.
     *
     * @param Config|null $config
     *
     * @throws \Exception
     */
    public function __construct(Config $config = null)
    {
        if ($config !== null) {
            $this->setConfig($config);
        }

        $this->setTimezone();
    }

    /**
     * Set the timezone.
     */
    private function setTimezone()
    {
        // set timezone
        date_default_timezone_set($this->getConfig()->get('timezone'));
    }

    /**
     * @param null|string $key
     *
     * @return mixed
     */
    public function getRequest($key = null)
    {
        return $this->getListener()->getRequest($key);
    }

    /**
     * @return AbstractBaseSlack|null|void
     */
    private function handleMessageActions()
    {
        $post = $this->getRequestUtility()->getPost();

        // ignore if payload is not set
        if (!isset($post['payload'])) {
            /* @noinspection PhpInconsistentReturnPointsInspection */
            return;
        }

        // posted payload is in JSON
        $payload = json_decode($post['payload'], true);

        return (new MessageAction())->load($payload);
    }

    /**
     * @throws \Exception
     */
    private function handleSendResponse()
    {
        // 1. Start listening
        $this->getListener()->listen();

        // 2. verify the request
        try {
            $verificationResult = $this->getListener()->verifyRequest();

            if ($verificationResult['success'] !== true) {
                throw new \Exception($verificationResult['message']);
            }
        } catch (\Exception $e) {
            throw $e;
        }

        // 3. pre process the request
        $this->preProcessRequest();

        // 4. check access control
        if ($this->checkAccessControl() !== true) {
            return;
        }

        // 5. set the current command
        $message = $this->getListener()->getMessage();
        $this->setCurrentCommand($this->getMessageUtility()->extractCommandName($message));

        // 6. log the message
        $this->getLoggerUtility()->logChat(__METHOD__, $message);

        // 7. send confirmation message if is enabled
        $this->getSender()->sendConfirmation();

        // 8. And send the response to the channel, only if the response is not empty
        $response = $this->respond($message);

        if (!empty($response)) {
            $this->getSender()->send($response);
        }
    }

    /**
     * @return bool
     */
    private function checkAccessControl()
    {
        // if accessControlEnabled is not set true ignore the check and return true
        if ($this->getConfig()->get('accessControlEnabled') !== true) {
            return true;
        }

        if ($this->getBlackList()->isBlackListed() !== false) {
            // found in blacklist
            $this->getSender()->send($this->getDictionary()->get('generic-messages')['blacklistedMessage']);

            return false;
        }

        if ($this->getWhiteList()->isWhiteListed() !== true) {
            // not found in whitelist
            $this->getSender()->send($this->getDictionary()->get('generic-messages')['whitelistedMessage']);

            return false;
        }

        return true;
    }

    /**
     * @throws \Exception
     */
    public function run()
    {
        switch ($this->getListener()->determineAction()) {
            case 'oauth':
                return $this->handleOAuth();
            case 'message_actions':
                return $this->handleMessageActions();
            case 'url_verification':
                return $this->handleUrlVerification();
            default:
                return $this->handleSendResponse();
        }
    }

    /**
     * handle OAuth.
     */
    private function handleOAuth()
    {
        return $this->getOauth()->doOauth();
    }

    /**
     * @throws \Exception
     */
    private function handleUrlVerification()
    {
        $request = $this->getRequestUtility()->getPostedBody();

        if (empty($request['challenge'])) {
            throw new \Exception('Challenge is missing for URL verification');
        }

        echo $request['challenge'];
    }

    /**
     * Pre-process the request.
     */
    private function preProcessRequest()
    {
        $request = $this->getListener()->getRequest();

        // remove the trigger_word from beginning of the message
        if (!empty($request['trigger_word'])) {
            $request['text'] = $this->getMessageUtility()->removeTriggerWord(
                $request['text'],
                $request['trigger_word']
            );

            $this->getListener()->setRequest($request);
        }
    }

    /**
     * @param null $message
     *
     * @throws \Exception
     *
     * @return mixed
     */
    public function respond($message = null)
    {
        try {
            $command = $this->getCommandByMessage($message);

            if (!$command instanceof Command) {
                // something went wrong, error will tell us!
                return $this->getLastError();
            }

            $pluginClass = $this->getPluginClassByCommand($command);

            // check action exists
            $action = $command->getAction();
            if (!method_exists($pluginClass, $action)) {
                $className = get_class($pluginClass);
                throw new \Exception("Action / function: '{$action}' does not exist in '{$className}'");
            }

            return $pluginClass->$action();
        } catch (\Exception $e) {
            throw $e;
        }
    }

    /**
     * Get plugin class by command.
     *
     * @param Command $command
     *
     * @throws \Exception
     *
     * @return AbstractPlugin
     */
    private function getPluginClassByCommand(Command $command)
    {
        // create the class
        $pluginClassFile = $command->getClass();
        $pluginClass = new $pluginClassFile($this);

        // check class is valid
        if (!$pluginClass instanceof AbstractPlugin) {
            $className = get_class($pluginClass);
            throw new \Exception("Couldn't create class: '{$className}'");
        }

        return $pluginClass;
    }

    /**
     * @param null $message
     *
     * @return Command|void
     */
    public function getCommandByMessage($message = null)
    {
        // If message is not set, get it from the current request
        if ($message === null) {
            $message = $this->getListener()->getMessage();
        }

        if (empty($message)) {
            $this->setLastError('Message is empty');
            return;
        }

        /*
         * Process the message.
         */
        return $this->getCommandObjectByMessage($message);
    }

    /**
     * @param $message
     *
     * @return Command|void
     */
    private function getCommandObjectByMessage($message)
    {
        $command = $this->getMessageUtility()->extractCommandName($message);

        // check command name
        if (empty($command)) {
            // get the default command if no command is find in the message
            $command = $this->getConfig()->get('defaultCommand');

            if (empty($command)) {
                $this->setLastError($this->getDictionary()->get('generic-messages')['noCommandMessage']);
                return;
            }
        }

        return $this->getCommandObjectByCommand($command);
    }

    /**
     * @param $command
     *
     * @return Command|void
     */
    private function getCommandObjectByCommand($command)
    {
        $commandObject = $this->getCommandContainer()->getAsObject($command);

        if ($this->validateCommandObject($commandObject) !== true) {
            return;
        }

        return $commandObject;
    }

    /**
     * Validate the command object.
     *
     * @param Command|null $commandObject
     *
     * @return bool
     */
    private function validateCommandObject($commandObject)
    {
        // check command details
        if (empty($commandObject)) {
            $this->setLastError(
                $this->getDictionary()->getValueByKey('generic-messages', 'unknownCommandMessage')
            );

            return false;
        }

        return true;
    }

    /**
     * @return array
     */
    public function getCommands()
    {
        if (!isset($this->commands)) {
            $this->setCommands($this->getCommandContainer()->getAllAsObject());
        }

        return $this->commands;
    }

    /**
     * @param array $commands
     */
    public function setCommands(array $commands)
    {
        $this->commands = $commands;
    }

    /**
     * @return string
     */
    public function getLastError()
    {
        return $this->lastError;
    }

    /**
     * @param string $lastError
     */
    public function setLastError($lastError)
    {
        $this->lastError = $lastError;
    }

    /**
     * Return the current command.
     *
     * @return string
     */
    public function getCurrentCommand()
    {
        return $this->currentCommand;
    }

    /**
     * @param string $currentCommand
     */
    public function setCurrentCommand($currentCommand)
    {
        $this->currentCommand = $currentCommand;
    }

    /**
     * Determine if bot user id is mentioned in the message.
     *
     * @return bool
     */
    public function youTalkingToMe()
    {
        $message = $this->getListener()->getMessage();

        if (empty($message)) {
            return false;
        }

        if ($this->getMessageUtility()->isBotMentioned($message) === true) {
            return true;
        }

        $listener = $this->getListener();
        // check direct messages
        return $listener instanceof EventListener && $listener->getEvent()->isDirectMessage() === true;
    }
}


1		<?php
2
3		namespace Botonomous;
4
5		use Botonomous\listener\EventListener;
6		use Botonomous\plugin\AbstractPlugin;
7
8		/**
9		* Class Botonomous.
10		*/
11		class Slackbot extends AbstractBot
12		{
13		private $commands;
14		private $lastError;
15		private $currentCommand;
16
17		/**
18		* Botonomous constructor.
19		*
20		* @param Config\|null $config
21		*
22		* @throws \Exception
23		*/
24	42	public function __construct(Config $config = null)
25		{
26	42	if ($config !== null) {
27	6	$this->setConfig($config);
28		}
29
30	42	$this->setTimezone();
31	42	}
32
33		/**
34		* Set the timezone.
35		*/
36	42	private function setTimezone()
37		{
38		// set timezone
39	42	date_default_timezone_set($this->getConfig()->get('timezone'));
40	42	}
41
42		/**
43		* @param null\|string $key
44		*
45		* @return mixed
46		*/
47	10	public function getRequest($key = null)
48		{
49	10	return $this->getListener()->getRequest($key);
50		}
51
52		/**
53		* @return AbstractBaseSlack\|null\|void
54		*/
55	1	private function handleMessageActions()
56		{
57	1	$post = $this->getRequestUtility()->getPost();
58
59		// ignore if payload is not set
60	1	if (!isset($post['payload'])) {
61		/* @noinspection PhpInconsistentReturnPointsInspection */
62	1	return;
63		}
64
65		// posted payload is in JSON
66	1	$payload = json_decode($post['payload'], true);
67
68	1	return (new MessageAction())->load($payload);
69		}
70
71		/**
72		* @throws \Exception
73		*/
74	5	private function handleSendResponse()
75		{
76		// 1. Start listening
77	5	$this->getListener()->listen();
78
79		// 2. verify the request
80		try {
81	5	$verificationResult = $this->getListener()->verifyRequest();
82
83	5	if ($verificationResult['success'] !== true) {
84	5	throw new \Exception($verificationResult['message']);
85		}
86	2	} catch (\Exception $e) {
87	2	throw $e;
88		}
89
90		// 3. pre process the request
91	3	$this->preProcessRequest();
92
93		// 4. check access control
94	3	if ($this->checkAccessControl() !== true) {
95	2	return;
96		}
97
98		// 5. set the current command
99	1	$message = $this->getListener()->getMessage();
100	1	$this->setCurrentCommand($this->getMessageUtility()->extractCommandName($message));
101
102		// 6. log the message
103	1	$this->getLoggerUtility()->logChat(__METHOD__, $message);
104
105		// 7. send confirmation message if is enabled
106	1	$this->getSender()->sendConfirmation();
107
108		// 8. And send the response to the channel, only if the response is not empty
109	1	$response = $this->respond($message);
110
111	1	if (!empty($response)) {
112	1	$this->getSender()->send($response);
113		}
114	1	}
115
116		/**
117		* @return bool
118		*/
119	3	private function checkAccessControl()
120		{
121		// if accessControlEnabled is not set true ignore the check and return true
122	3	if ($this->getConfig()->get('accessControlEnabled') !== true) {
123	1	return true;
124		}
125
126	2	if ($this->getBlackList()->isBlackListed() !== false) {
127		// found in blacklist
128	1	$this->getSender()->send($this->getDictionary()->get('generic-messages')['blacklistedMessage']);
129
130	1	return false;
131		}
132
133	1	if ($this->getWhiteList()->isWhiteListed() !== true) {
134		// not found in whitelist
135	1	$this->getSender()->send($this->getDictionary()->get('generic-messages')['whitelistedMessage']);
136
137	1	return false;
138		}
139
140		return true;
141		}
142
143		/**
144		* @throws \Exception
145		*/
146	8	public function run()
147		{
148	8	switch ($this->getListener()->determineAction()) {
149	8	case 'oauth':
150	1	return $this->handleOAuth();
151	7	case 'message_actions':
152	1	return $this->handleMessageActions();
153	6	case 'url_verification':
154	1	return $this->handleUrlVerification();
155		default:
156	5	return $this->handleSendResponse();
157		}
158		}
159
160		/**
161		* handle OAuth.
162		*/
163	1	private function handleOAuth()
164		{
165	1	return $this->getOauth()->doOauth();
166		}
167
168		/**
169		* @throws \Exception
170		*/
171	1	private function handleUrlVerification()
172		{
173	1	$request = $this->getRequestUtility()->getPostedBody();
174
175	1	if (empty($request['challenge'])) {
176		throw new \Exception('Challenge is missing for URL verification');
177		}
178
179	1	echo $request['challenge'];
180	1	}
181
182		/**
183		* Pre-process the request.
184		*/
185	3	private function preProcessRequest()
186		{
187	3	$request = $this->getListener()->getRequest();
188
189		// remove the trigger_word from beginning of the message
190	3	if (!empty($request['trigger_word'])) {
191	3	$request['text'] = $this->getMessageUtility()->removeTriggerWord(
192	3	$request['text'],
193	3	$request['trigger_word']
194		);
195
196	3	$this->getListener()->setRequest($request);
197		}
198	3	}
199
200		/**
201		* @param null $message
202		*
203		* @throws \Exception
204		*
205		* @return mixed
206		*/
207	4	public function respond($message = null)
208		{
209		try {
210	4	$command = $this->getCommandByMessage($message);
211
212	4	if (!$command instanceof Command) {
213		// something went wrong, error will tell us!
214	2	return $this->getLastError();
215		}
216
217	3	$pluginClass = $this->getPluginClassByCommand($command);
218
219		// check action exists
220	3	$action = $command->getAction();
221	3	if (!method_exists($pluginClass, $action)) {
222	1	$className = get_class($pluginClass);
223	1	throw new \Exception("Action / function: '{$action}' does not exist in '{$className}'");
224		}
225
226	2	return $pluginClass->$action();
227	1	} catch (\Exception $e) {
228	1	throw $e;
229		}
230		}
231
232		/**
233		* Get plugin class by command.
234		*
235		* @param Command $command
236		*
237		* @throws \Exception
238		*
239		* @return AbstractPlugin
240		*/
241	3	private function getPluginClassByCommand(Command $command)
242		{
243		// create the class
244	3	$pluginClassFile = $command->getClass();
245	3	$pluginClass = new $pluginClassFile($this);
246
247		// check class is valid
248	3	if (!$pluginClass instanceof AbstractPlugin) {
249		$className = get_class($pluginClass);
250		throw new \Exception("Couldn't create class: '{$className}'");
251		}
252
253	3	return $pluginClass;
254		}
255
256		/**
257		* @param null $message
258		*
259		* @return Command\|void
260		*/
261	7	public function getCommandByMessage($message = null)
262		{
263		// If message is not set, get it from the current request
264	7	if ($message === null) {
265	4	$message = $this->getListener()->getMessage();
266		}
267
268	7	if (empty($message)) {
269	1	$this->setLastError('Message is empty');
270	1	return;
271		}
272
273		/*
274		* Process the message.
275		*/
276	6	return $this->getCommandObjectByMessage($message);
277		}
278
279		/**
280		* @param $message
281		*
282		* @return Command\|void
283		*/
284	6	private function getCommandObjectByMessage($message)
285		{
286	6	$command = $this->getMessageUtility()->extractCommandName($message);
287
288		// check command name
289	6	if (empty($command)) {
290		// get the default command if no command is find in the message
291	2	$command = $this->getConfig()->get('defaultCommand');
292
293	2	if (empty($command)) {
294	2	$this->setLastError($this->getDictionary()->get('generic-messages')['noCommandMessage']);
295	2	return;
296		}
297		}
298
299	4	return $this->getCommandObjectByCommand($command);
300		}
301
302		/**
303		* @param $command
304		*
305		* @return Command\|void
306		*/
307	4	private function getCommandObjectByCommand($command)
308		{
309	4	$commandObject = $this->getCommandContainer()->getAsObject($command);
310
311	4	if ($this->validateCommandObject($commandObject) !== true) {
312	1	return;
313		}
314
315	4	return $commandObject;
316		}
317
318		/**
319		* Validate the command object.
320		*
321		* @param Command\|null $commandObject
322		*
323		* @return bool
324		*/
325	4	private function validateCommandObject($commandObject)
326		{
327		// check command details
328	4	if (empty($commandObject)) {
329	1	$this->setLastError(
330	1	$this->getDictionary()->getValueByKey('generic-messages', 'unknownCommandMessage')
331		);
332
333	1	return false;
334		}
335
336	4	return true;
337		}
338
339		/**
340		* @return array
341		*/
342	2	public function getCommands()
343		{
344	2	if (!isset($this->commands)) {
345	1	$this->setCommands($this->getCommandContainer()->getAllAsObject());
346		}
347
348	2	return $this->commands;
349		}
350
351		/**
352		* @param array $commands
353		*/
354	2	public function setCommands(array $commands)
355		{
356	2	$this->commands = $commands;
357	2	}
358
359		/**
360		* @return string
361		*/
362	4	public function getLastError()
363		{
364	4	return $this->lastError;
365		}
366
367		/**
368		* @param string $lastError
369		*/
370	4	public function setLastError($lastError)
371		{
372	4	$this->lastError = $lastError;
373	4	}
374
375		/**
376		* Return the current command.
377		*
378		* @return string
379		*/
380	1	public function getCurrentCommand()
381		{
382	1	return $this->currentCommand;
383		}
384
385		/**
386		* @param string $currentCommand
387		*/
388	2	public function setCurrentCommand($currentCommand)
389		{
390	2	$this->currentCommand = $currentCommand;
391	2	}
392
393		/**
394		* Determine if bot user id is mentioned in the message.
395		*
396		* @return bool
397		*/
398	1	public function youTalkingToMe()
399		{
400	1	$message = $this->getListener()->getMessage();
401
402	1	if (empty($message)) {
403	1	return false;
404		}
405
406	1	if ($this->getMessageUtility()->isBotMentioned($message) === true) {
407	1	return true;
408		}
409
410	1	$listener = $this->getListener();
411		// check direct messages
412	1	return $listener instanceof EventListener && $listener->getEvent()->isDirectMessage() === true;
413		}
414		}
415

iranianpep / botonomous

Push — master ( 5712f6...cce3fc )

Slackbot::getCommandObjectByMessage() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like