WhiteList::isEmailWhiteListed() - Code Metrics - Inspection of "First phase of changing Slackbot name to Botonomou..." - iranianpep/botonomous - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b808b9...919917 )

by Ehsan

created 2017-05-03 10:11 UTC

WhiteList::isEmailWhiteListed() A

↳ Parent: WhiteList

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

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

<?php

namespace Botonomous;

class WhiteList extends AbstractAccessList
{
    /**
     * WhiteList constructor.
     *
     * @param $request
     */
    public function __construct($request)
    {
        $this->setRequest($request);
    }

    /**
     * @return bool
     */
    public function isWhiteListed()
    {
        $usernameCheck = true;
        $userIdCheck = true;
        $userEmailCheck = true;

        if ($this->isUsernameWhiteListed() === false) {
            $usernameCheck = false;
        }

        if ($this->isUserIdWhiteListed() === false) {
            $userIdCheck = false;
        }

        if ($this->isEmailWhiteListed() === false) {
            $userEmailCheck = false;
        }

        return $usernameCheck === true && $userIdCheck === true && $userEmailCheck === true ? true : false;
    }

    /**
     * @return bool
     */
    public function isUsernameWhiteListed()
    {
        return empty($this->findInListByRequestKey('user_name', $this->getShortClassName(), 'username')) ? false : true;
    }

    /**
     * @return bool
     */
    public function isUserIdWhiteListed()
    {
        return empty($this->findInListByRequestKey('user_id', $this->getShortClassName(), 'userId')) ? false : true;
    }

    /**
     * @return bool
     * @return mixed
     */
    public function isEmailWhiteListed()
    {
        // user_name is set, load the blacklist to start checking
        $list = $this->getSubAccessControlList($this->getShortClassName());

        // currently if list is not set we do not check it
        if (!isset($list['userEmail'])) {
            return true;
        }

        return $this->isEmailInList($list);
    }
}


1		<?php
2
3		namespace Botonomous;
4
5		class WhiteList extends AbstractAccessList
6		{
7		/**
8		* WhiteList constructor.
9		*
10		* @param $request
11		*/
12	12	public function __construct($request)
13		{
14	12	$this->setRequest($request);
15	12	}
16
17		/**
18		* @return bool
19		*/
20	3	public function isWhiteListed()
21		{
22	3	$usernameCheck = true;
23	3	$userIdCheck = true;
24	3	$userEmailCheck = true;
25
26	3	if ($this->isUsernameWhiteListed() === false) {
27	2	$usernameCheck = false;
28		}
29
30	3	if ($this->isUserIdWhiteListed() === false) {
31	2	$userIdCheck = false;
32		}
33
34	3	if ($this->isEmailWhiteListed() === false) {
35	2	$userEmailCheck = false;
36		}
37
38	3	return $usernameCheck === true && $userIdCheck === true && $userEmailCheck === true ? true : false;
39		}
40
41		/**
42		* @return bool
43		*/
44	4	public function isUsernameWhiteListed()
45		{
46	4	return empty($this->findInListByRequestKey('user_name', $this->getShortClassName(), 'username')) ? false : true;
47		}
48
49		/**
50		* @return bool
51		*/
52	4	public function isUserIdWhiteListed()
53		{
54	4	return empty($this->findInListByRequestKey('user_id', $this->getShortClassName(), 'userId')) ? false : true;
55		}
56
57		/**
58		* @return bool
59		* @return mixed
60		*/
61	6	public function isEmailWhiteListed()
62		{
63		// user_name is set, load the blacklist to start checking
64	6	$list = $this->getSubAccessControlList($this->getShortClassName());
65
66		// currently if list is not set we do not check it
67	6	if (!isset($list['userEmail'])) {
68	1	return true;
69		}
70
71	5	return $this->isEmailInList($list);
72		}
73		}
74

iranianpep / botonomous

Push — master ( b808b9...919917 )

WhiteList::isEmailWhiteListed() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like