User::fromArray() - Code Metrics - Inspection of "Add MergeRequestApprovalServiceTests, Update makef..." - danielpieper/mrcli - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4c1740...b98411 )

by Daniel

created 2019-01-22 16:53 UTC

User::fromArray() A

↳ Parent: User

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	1

Importance

Changes

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

<?php declare(strict_types=1);

namespace DanielPieper\MergeReminder\ValueObject;

class User
{
    public const STATE_ACTIVE = 'active';
    public const STATE_BLOCKED = 'blocked';

    /** @var int */
    private $id;

    /** @var string */
    private $username;

    /** @var string */
    private $name;

    /** @var string */
    private $state;

    /** @var string */
    private $avatarUrl;

    /** @var string */
    private $webUrl;

    /**
     * Project constructor.
     * @param int $id
     * @param string $username
     * @param string $name
     * @param string $state
     * @param string $avatarUrl
     * @param string $webUrl
     */
    public function __construct(
        int $id,
        string $username,
        string $name,
        string $state,
        string $avatarUrl,
        string $webUrl
    ) {
        $this->id = $id;
        $this->username = $username;
        $this->name = $name;
        $this->state = $state;
        $this->avatarUrl = $avatarUrl;
        $this->webUrl = $webUrl;
    }

    /**
     * @param array $user
     * @return User
     */
    public static function fromArray(array $user): self
    {
        return new self(
            (int)$user['id'],
            (string)$user['username'],
            (string)$user['name'],
            (string)$user['state'],
            (string)$user['avatar_url'],
            (string)$user['web_url']
        );
    }

    /**
     * @return int
     */
    public function getId(): int
    {
        return $this->id;
    }

    /**
     * @return string
     */
    public function getUsername(): string
    {
        return $this->username;
    }

    /**
     * @return string
     */
    public function getName(): string
    {
        return $this->name;
    }

    /**
     * @return string
     */
    public function getState(): string
    {
        return $this->state;
    }

    /**
     * @return bool
     */
    public function isActive(): bool
    {
        return $this->state === self::STATE_ACTIVE;
    }

    /**
     * @return bool
     */
    public function isBlocked(): bool
    {
        return $this->state === self::STATE_BLOCKED;
    }

    /**
     * @return string
     */
    public function getAvatarUrl(): string
    {
        return $this->avatarUrl;
    }

    /**
     * @return string
     */
    public function getWebUrl(): string
    {
        return $this->webUrl;
    }
}


1		<?php declare(strict_types=1);
2
3		namespace DanielPieper\MergeReminder\ValueObject;
4
5		class User
6		{
7		public const STATE_ACTIVE = 'active';
8		public const STATE_BLOCKED = 'blocked';
9
10		/** @var int */
11		private $id;
12
13		/** @var string */
14		private $username;
15
16		/** @var string */
17		private $name;
18
19		/** @var string */
20		private $state;
21
22		/** @var string */
23		private $avatarUrl;
24
25		/** @var string */
26		private $webUrl;
27
28		/**
29		* Project constructor.
30		* @param int $id
31		* @param string $username
32		* @param string $name
33		* @param string $state
34		* @param string $avatarUrl
35		* @param string $webUrl
36		*/
37	3	public function __construct(
38		int $id,
39		string $username,
40		string $name,
41		string $state,
42		string $avatarUrl,
43		string $webUrl
44		) {
45	3	$this->id = $id;
46	3	$this->username = $username;
47	3	$this->name = $name;
48	3	$this->state = $state;
49	3	$this->avatarUrl = $avatarUrl;
50	3	$this->webUrl = $webUrl;
51	3	}
52
53		/**
54		* @param array $user
55		* @return User
56		*/
57	3	public static function fromArray(array $user): self
58		{
59	3	return new self(
60	3	(int)$user['id'],
61	3	(string)$user['username'],
62	3	(string)$user['name'],
63	3	(string)$user['state'],
64	3	(string)$user['avatar_url'],
65	3	(string)$user['web_url']
66		);
67		}
68
69		/**
70		* @return int
71		*/
72	1	public function getId(): int
73		{
74	1	return $this->id;
75		}
76
77		/**
78		* @return string
79		*/
80		public function getUsername(): string
81		{
82		return $this->username;
83		}
84
85		/**
86		* @return string
87		*/
88		public function getName(): string
89		{
90		return $this->name;
91		}
92
93		/**
94		* @return string
95		*/
96		public function getState(): string
97		{
98		return $this->state;
99		}
100
101		/**
102		* @return bool
103		*/
104		public function isActive(): bool
105		{
106		return $this->state === self::STATE_ACTIVE;
107		}
108
109		/**
110		* @return bool
111		*/
112		public function isBlocked(): bool
113		{
114		return $this->state === self::STATE_BLOCKED;
115		}
116
117		/**
118		* @return string
119		*/
120		public function getAvatarUrl(): string
121		{
122		return $this->avatarUrl;
123		}
124
125		/**
126		* @return string
127		*/
128		public function getWebUrl(): string
129		{
130		return $this->webUrl;
131		}
132		}
133

danielpieper / mrcli

Push — master ( 4c1740...b98411 )

User::fromArray() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like