Slackbot::preProcessRequest() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/irania..." - iranianpep/botonomous - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5f4a5c...88834d )

by Ehsan

created 2017-07-13 04:02 UTC

Slackbot::preProcessRequest() A

↳ Parent: Slackbot

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2

Importance

Changes

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

<?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
     *
     * @throws \Exception
     *
     * @return bool|Command

     */
    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 false;
        }

        /**
         * Process the 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 false;
            }
        }

        $commandObject = $this->getCommandContainer()->getAsObject($command);

        try {
            if ($this->validateCommandObject($commandObject, $command) !== true) {
                return false;
            }
        } catch (\Exception $e) {
            throw $e;
        }

        return $commandObject;
    }

    /**
     * Validate the command object.
     *
     * @param $commandObject
     * @param $command
     * @return bool
     * @throws \Exception
     */
    private function validateCommandObject($commandObject, $command)
    {
        // check command details
        if (empty($commandObject)) {
            $this->setLastError(
                $this->getDictionary()->getValueByKey('generic-messages', 'unknownCommandMessage', ['command' => $command])

            );

            return false;
        }

        if (!$commandObject instanceof Command) {
            throw new \Exception('Command is not an object');
        }

        // check the plugin for the command
        if (empty($commandObject->getPlugin())) {
            throw new \Exception('Plugin is not set for this command');
        }

        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		* @throws \Exception
260		*
261		* @return bool\|Command
		0 ignored issues – show Documentation introduced 2017-07-13 04:04 UTC by Report Bug Copy Issue Report Should the return type not be `false\|Command\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
262		*/
263	7	public function getCommandByMessage($message = null)
264		{
265		// If message is not set, get it from the current request
266	7	if ($message === null) {
267	4	$message = $this->getListener()->getMessage();
268		}
269
270	7	if (empty($message)) {
271	1	$this->setLastError('Message is empty');
272
273	1	return false;
274		}
275
276		/**
277		* Process the message.
278		*/
279	6	$command = $this->getMessageUtility()->extractCommandName($message);
280
281		// check command name
282	6	if (empty($command)) {
283		// get the default command if no command is find in the message
284	2	$command = $this->getConfig()->get('defaultCommand');
285
286	2	if (empty($command)) {
287	2	$this->setLastError($this->getDictionary()->get('generic-messages')['noCommandMessage']);
288
289	2	return false;
290		}
291		}
292
293	4	$commandObject = $this->getCommandContainer()->getAsObject($command);
294
295		try {
296	4	if ($this->validateCommandObject($commandObject, $command) !== true) {
297	4	return false;
298		}
299		} catch (\Exception $e) {
300		throw $e;
301		}
302
303	4	return $commandObject;
304		}
305
306		/**
307		* Validate the command object.
308		*
309		* @param $commandObject
310		* @param $command
311		* @return bool
312		* @throws \Exception
313		*/
314	4	private function validateCommandObject($commandObject, $command)
315		{
316		// check command details
317	4	if (empty($commandObject)) {
318	1	$this->setLastError(
319	1	$this->getDictionary()->getValueByKey('generic-messages', 'unknownCommandMessage', ['command' => $command])
		0 ignored issues – show Coding Style introduced 2017-07-13 04:04 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 123 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
320		);
321
322	1	return false;
323		}
324
325	4	if (!$commandObject instanceof Command) {
326		throw new \Exception('Command is not an object');
327		}
328
329		// check the plugin for the command
330	4	if (empty($commandObject->getPlugin())) {
331		throw new \Exception('Plugin is not set for this command');
332		}
333
334	4	return true;
335		}
336
337		/**
338		* @return array
339		*/
340	2	public function getCommands()
341		{
342	2	if (!isset($this->commands)) {
343	1	$this->setCommands($this->getCommandContainer()->getAllAsObject());
344		}
345
346	2	return $this->commands;
347		}
348
349		/**
350		* @param array $commands
351		*/
352	2	public function setCommands(array $commands)
353		{
354	2	$this->commands = $commands;
355	2	}
356
357		/**
358		* @return string
359		*/
360	4	public function getLastError()
361		{
362	4	return $this->lastError;
363		}
364
365		/**
366		* @param string $lastError
367		*/
368	4	public function setLastError($lastError)
369		{
370	4	$this->lastError = $lastError;
371	4	}
372
373		/**
374		* Return the current command.
375		*
376		* @return string
377		*/
378	1	public function getCurrentCommand()
379		{
380	1	return $this->currentCommand;
381		}
382
383		/**
384		* @param string $currentCommand
385		*/
386	2	public function setCurrentCommand($currentCommand)
387		{
388	2	$this->currentCommand = $currentCommand;
389	2	}
390
391		/**
392		* Determine if bot user id is mentioned in the message.
393		*
394		* @return bool
395		*/
396	1	public function youTalkingToMe()
397		{
398	1	$message = $this->getListener()->getMessage();
399
400	1	if (empty($message)) {
401	1	return false;
402		}
403
404	1	if ($this->getMessageUtility()->isBotMentioned($message) === true) {
405	1	return true;
406		}
407
408	1	$listener = $this->getListener();
409		// check direct messages
410	1	return $listener instanceof EventListener && $listener->getEvent()->isDirectMessage() === true;
411		}
412		}
413

iranianpep / botonomous

Push — master ( 5f4a5c...88834d )

Slackbot::preProcessRequest() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like