User::fromArray() - Code Metrics - Inspection of "Add ProjectCommandTest" - danielpieper/mrcli - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6c4a7c...60c24a )

by Daniel

created 2019-02-07 07: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	42	public function __construct(
38		int $id,
39		string $username,
40		string $name,
41		string $state,
42		string $avatarUrl,
43		string $webUrl
44		) {
45	42	$this->id = $id;
46	42	$this->username = $username;
47	42	$this->name = $name;
48	42	$this->state = $state;
49	42	$this->avatarUrl = $avatarUrl;
50	42	$this->webUrl = $webUrl;
51	42	}
52
53		/**
54		* @param array $user
55		* @return User
56		*/
57	42	public static function fromArray(array $user): self
58		{
59	42	return new self(
60	42	(int)$user['id'],
61	42	(string)$user['username'],
62	42	(string)$user['name'],
63	42	(string)$user['state'],
64	42	(string)$user['avatar_url'],
65	42	(string)$user['web_url']
66		);
67		}
68
69		/**
70		* @return int
71		*/
72	8	public function getId(): int
73		{
74	8	return $this->id;
75		}
76
77		/**
78		* @return string
79		*/
80	13	public function getUsername(): string
81		{
82	13	return $this->username;
83		}
84
85		/**
86		* @return string
87		*/
88	1	public function getName(): string
89		{
90	1	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	2	public function getWebUrl(): string
129		{
130	2	return $this->webUrl;
131		}
132		}
133

danielpieper / mrcli

Push — master ( 6c4a7c...60c24a )

User::fromArray() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like