CredentialResponseItem - Code Metrics - Inspection of "Renamed namespace from Resource\Api to Model (#293..." - mailgun/mailgun-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( da873d...74929b )

by Tobias

created 2017-02-23 19:56 UTC

CredentialResponseItem A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

Importance

Changes

Metric	Value
wmc	10
lcom	0
cbo	0
dl	0
loc	83
ccs	0
cts	20
cp	0
rs	10
c	0
b	0
f	0

6 Methods

Rating	Name	Size	Complexity
B	create()	9	5
A	__construct()	7	1
A	getSizeBytes()	4	1
A	getCreatedAt()	4	1
A	getMailbox()	4	1
A	getLogin()	4	1

<?php

/*
 * Copyright (C) 2013-2016 Mailgun
 *
 * This software may be modified and distributed under the terms
 * of the MIT license. See the LICENSE file for details.
 */

namespace Mailgun\Model\Domain;

/**
 * @author Sean Johnson <[email protected]>
 */
final class CredentialResponseItem
{
    /**
     * @var int|null
     */
    private $sizeBytes;

    /**
     * @var \DateTime
     */
    private $createdAt;

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

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

    /**
     * @param array $data
     *
     * @return self
     */
    public static function create(array $data)
    {
        $sizeBytes = isset($data['size_bytes']) ? $data['size_bytes'] : null;
        $mailbox = isset($data['mailbox']) ? $data['mailbox'] : null;
        $login = isset($data['login']) ? $data['login'] : null;
        $createdAt = isset($data['created_at']) ? new \DateTime($data['created_at']) : null;

        return new self($sizeBytes, $createdAt, $mailbox, $login);
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
    }

    /**
     * @param int       $sizeBytes
     * @param \DateTime $createdAt
     * @param string    $mailbox
     * @param string    $login
     */
    private function __construct($sizeBytes, \DateTime $createdAt, $mailbox, $login)
    {
        $this->sizeBytes = $sizeBytes;
        $this->createdAt = $createdAt;
        $this->mailbox = $mailbox;
        $this->login = $login;
    }

    /**
     * @return int|null
     */
    public function getSizeBytes()
    {
        return $this->sizeBytes;
    }

    /**
     * @return \DateTime
     */
    public function getCreatedAt()
    {
        return $this->createdAt;
    }

    /**
     * @return string
     */
    public function getMailbox()
    {
        return $this->mailbox;
    }

    /**
     * @return string
     */
    public function getLogin()
    {
        return $this->login;
    }
}


1			<?php
2
3			/*
4			* Copyright (C) 2013-2016 Mailgun
5			*
6			* This software may be modified and distributed under the terms
7			* of the MIT license. See the LICENSE file for details.
8			*/
9
10			namespace Mailgun\Model\Domain;
11
12			/**
13			* @author Sean Johnson <[email protected]>
14			*/
15			final class CredentialResponseItem
16			{
17			/**
18			* @var int\|null
19			*/
20			private $sizeBytes;
21
22			/**
23			* @var \DateTime
24			*/
25			private $createdAt;
26
27			/**
28			* @var string
29			*/
30			private $mailbox;
31
32			/**
33			* @var string
34			*/
35			private $login;
36
37			/**
38			* @param array $data
39			*
40			* @return self
41			*/
42			public static function create(array $data)
43			{
44			$sizeBytes = isset($data['size_bytes']) ? $data['size_bytes'] : null;
45			$mailbox = isset($data['mailbox']) ? $data['mailbox'] : null;
46			$login = isset($data['login']) ? $data['login'] : null;
47			$createdAt = isset($data['created_at']) ? new \DateTime($data['created_at']) : null;
48
49			return new self($sizeBytes, $createdAt, $mailbox, $login);
			0 ignored issues – show Bug introduced 2017-02-23 20:00 UTC by Report Bug Copy Issue Report It seems like `$createdAt` defined by `isset($data['created_at'...a['created_at']) : null` on line `47` can be `null`; however, `Mailgun\Model\Domain\Cre...onseItem::__construct()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
50			}
51
52			/**
53			* @param int $sizeBytes
54			* @param \DateTime $createdAt
55			* @param string $mailbox
56			* @param string $login
57			*/
58			private function __construct($sizeBytes, \DateTime $createdAt, $mailbox, $login)
59			{
60			$this->sizeBytes = $sizeBytes;
61			$this->createdAt = $createdAt;
62			$this->mailbox = $mailbox;
63			$this->login = $login;
64			}
65
66			/**
67			* @return int\|null
68			*/
69			public function getSizeBytes()
70			{
71			return $this->sizeBytes;
72			}
73
74			/**
75			* @return \DateTime
76			*/
77			public function getCreatedAt()
78			{
79			return $this->createdAt;
80			}
81
82			/**
83			* @return string
84			*/
85			public function getMailbox()
86			{
87			return $this->mailbox;
88			}
89
90			/**
91			* @return string
92			*/
93			public function getLogin()
94			{
95			return $this->login;
96			}
97			}
98

mailgun / mailgun-php

Push — master ( da873d...74929b )

CredentialResponseItem A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

6 Methods

Duplication Side-by-Side

Filter issues like