ProfileRequest::buildHeaders() - Code Metrics - Inspection of "Merge pull request #51 from ker0x/hotfix/issue-50" - ker0x/messenger - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f115d8...f636e9 )

by Romain

created 2017-07-14 01:29 UTC

ProfileRequest::buildHeaders() A

↳ Parent: ProfileRequest

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	2

Importance

Changes

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

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

ker0x / messenger

Push — master ( f115d8...f636e9 )

ProfileRequest::buildHeaders() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like