SlashCommandListener::verifyOrigin() - Code Metrics - Inspection of "Revert changes" - iranianpep/botonomous - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ff83e3...5f4a5c )

by Ehsan

created 2017-06-29 13:39 UTC

SlashCommandListener::verifyOrigin() B

↳ Parent: SlashCommandListener

Complexity

Conditions	4
Paths	4

Size

Total Lines	29
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	29
ccs	10
cts	10
cp	1
rs	8.5806
c	0
b	0
f	0
cc	4
eloc	16
nc	4
nop	0
crap	4

<?php

namespace Botonomous\listener;

class SlashCommandListener extends AbstractBaseListener
{
    const VERIFICATION_TOKEN = 'verificationToken';

    /**
     * @return mixed
     */
    public function extractRequest()
    {
        $postRequest = $this->getRequestUtility()->getPost();

        if (empty($postRequest)) {
            /* @noinspection PhpInconsistentReturnPointsInspection */
            return;
        }

        return $postRequest;
    }

    /**
     * @throws \Exception
     *
     * @return array
     */
    public function verifyOrigin()
    {
        $token = $this->getRequest('token');

        if (empty($token)) {
            return [
                'success' => false,
                'message' => 'Token is missing',
            ];
        }

        $expectedToken = $this->getConfig()->get(self::VERIFICATION_TOKEN);

        if (empty($expectedToken)) {
            throw new \Exception('Token must be set in the config');
        }

        if ($token === $expectedToken) {
            return [
                'success' => true,
                'message' => 'Awesome!',
            ];
        }

        return [
            'success' => false,
            'message' => 'Token is not valid',
        ];
    }

    /**
     * @throws \Exception
     *
     * @return bool
     */
    public function isThisBot()
    {
        $userId = $this->getRequest('user_id');
        $username = $this->getRequest('user_name');

        return ($userId == 'USLACKBOT' || $username == 'slackbot') ? true : false;
    }

    /**
     * @return string
     */
    public function getChannelId()
    {
        return $this->getRequest('channel_id');
    }
}


1		<?php
2
3		namespace Botonomous\listener;
4
5		class SlashCommandListener extends AbstractBaseListener
6		{
7		const VERIFICATION_TOKEN = 'verificationToken';
8
9		/**
10		* @return mixed
11		*/
12	9	public function extractRequest()
13		{
14	9	$postRequest = $this->getRequestUtility()->getPost();
15
16	9	if (empty($postRequest)) {
17		/* @noinspection PhpInconsistentReturnPointsInspection */
18	5	return;
19		}
20
21	4	return $postRequest;
22		}
23
24		/**
25		* @throws \Exception
26		*
27		* @return array
28		*/
29	6	public function verifyOrigin()
30		{
31	6	$token = $this->getRequest('token');
32
33	6	if (empty($token)) {
34		return [
35	2	'success' => false,
36		'message' => 'Token is missing',
37		];
38		}
39
40	5	$expectedToken = $this->getConfig()->get(self::VERIFICATION_TOKEN);
41
42	5	if (empty($expectedToken)) {
43	1	throw new \Exception('Token must be set in the config');
44		}
45
46	5	if ($token === $expectedToken) {
47		return [
48	5	'success' => true,
49		'message' => 'Awesome!',
50		];
51		}
52
53		return [
54	1	'success' => false,
55		'message' => 'Token is not valid',
56		];
57		}
58
59		/**
60		* @throws \Exception
61		*
62		* @return bool
63		*/
64	11	public function isThisBot()
65		{
66	11	$userId = $this->getRequest('user_id');
67	11	$username = $this->getRequest('user_name');
68
69	11	return ($userId == 'USLACKBOT' \|\| $username == 'slackbot') ? true : false;
70		}
71
72		/**
73		* @return string
74		*/
75	3	public function getChannelId()
76		{
77	3	return $this->getRequest('channel_id');
78		}
79		}
80

iranianpep / botonomous

Push — master ( ff83e3...5f4a5c )

SlashCommandListener::verifyOrigin() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like