ProfileRequest::buildBody() - Code Metrics - Inspection of "[feature #27] Add API for Messenger Profile" - ker0x/messenger - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#38)

by Romain

created 2017-03-19 03:00 UTC

ProfileRequest::buildBody() A

↳ Parent: ProfileRequest

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	6
cts	6
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	8
nc	3
nop	0
crap	3

<?php
namespace Kerox\Messenger\Request;

use Kerox\Messenger\Model\ProfileSettings;

class ProfileRequest extends AbstractRequest
{

    /**
     * @var mixed
     */
    protected $profileSettings;

    /**
     * ProfileRequest constructor.
     *
     * @param string $pageToken
     * @param mixed $profileSettings
     */
    public function __construct(string $pageToken, $profileSettings)
    {
        parent::__construct($pageToken);

        $this->profileSettings = $profileSettings;
    }

    /**
     * @return null|array
     */
    protected function buildHeaders()
    {
        $headers = [
            'Content-Type' => 'application/json',
        ];

        return (is_string($this->profileSettings)) ? null : $headers;
    }

    /**
     * @return null|array|\Kerox\Messenger\Model\ProfileSettings
     */
    protected function buildBody()
    {
        $body = null;
        if ($this->profileSettings instanceof ProfileSettings) {
            $body = $this->profileSettings;
        } elseif (is_array($this->profileSettings)) {
            $body = [
                'fields' => $this->profileSettings
            ];
        }

        return $body;
    }

    /**
     * @return array
     */
    protected function buildQuery(): array
    {
        $query = parent::buildQuery();

        if (is_string($this->profileSettings)) {
            $query += [
                'fields' => $this->profileSettings
            ];
        }

        return $query;
    }
}


1		<?php
2		namespace Kerox\Messenger\Request;
3
4		use Kerox\Messenger\Model\ProfileSettings;
5
6		class ProfileRequest extends AbstractRequest
7		{
8
9		/**
10		* @var mixed
11		*/
12		protected $profileSettings;
13
14		/**
15		* ProfileRequest constructor.
16		*
17		* @param string $pageToken
18		* @param mixed $profileSettings
19		*/
20	2	public function __construct(string $pageToken, $profileSettings)
21		{
22	2	parent::__construct($pageToken);
23
24	2	$this->profileSettings = $profileSettings;
25	2	}
26
27		/**
28		* @return null\|array
29		*/
30	2	protected function buildHeaders()
31		{
32		$headers = [
33	2	'Content-Type' => 'application/json',
34		];
35
36	2	return (is_string($this->profileSettings)) ? null : $headers;
37		}
38
39		/**
40		* @return null\|array\|\Kerox\Messenger\Model\ProfileSettings
41		*/
42	2	protected function buildBody()
43		{
44	2	$body = null;
45	2	if ($this->profileSettings instanceof ProfileSettings) {
46	1	$body = $this->profileSettings;
47	1	} elseif (is_array($this->profileSettings)) {
48		$body = [
49		'fields' => $this->profileSettings
50		];
51		}
52
53	2	return $body;
54		}
55
56		/**
57		* @return array
58		*/
59	2	protected function buildQuery(): array
60		{
61	2	$query = parent::buildQuery();
62
63	2	if (is_string($this->profileSettings)) {
64		$query += [
65	1	'fields' => $this->profileSettings
66		];
67		}
68
69	2	return $query;
70		}
71		}
72

ker0x / messenger

Pull Request — master (#38)

ProfileRequest::buildBody() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like