Request::parseData() - Code Metrics - Inspection of "Update examples" - CapMousse/React-Restify - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — dev (718e6b)

by Jérémy

created 2017-02-13 23:26 UTC

Request::parseData() C

↳ Parent: Request

Complexity

Conditions	8
Paths	2

Size

Total Lines	43
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	43
rs	5.3846
cc	8
eloc	26
nc	2
nop	0

<?php

namespace CapMousse\ReactRestify\Http;

use React\Http\Request as ReactHttpRequest;
use Evenement\EventEmitter;

class Request extends EventEmitter
{
    /** @var React\Http\Request */
    public $httpRequest;

    /** @var mixed */
    private $content;

    /** @var array */
    private $data = [];

    /**
     * @param ReactHttpRequest $httpRequest
     */
    public function __construct(ReactHttpRequest $httpRequest)
    {
        $this->httpRequest = $httpRequest;

    }

    /**
     * Set the raw data of the request
     * @param mixed $content
     */
    public function setContent($content)
    {
        $this->content = $content;
    }

    /**
     * Set the raw data of the request
     * @param mixed $content
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     */
    public function getContent()
    {
        return $this->content;
    }

    public function getHeaders()
    {
        $headers = array_change_key_case($this->httpRequest->getHeaders(), CASE_LOWER);
        $headers = array_map(function ($value) {
            return strtolower($value);
        }, $headers);

        return $headers;
    }

    public function parseData()
    {
        $headers = $this->getHeaders();

        if (!in_array($this->httpRequest->getMethod(), ['PUT', 'POST'])) {
            return $this->emit('end');
        }

        $this->httpRequest->on('data', function($data) use ($headers, &$dataResult) {
            $dataResult .= $data;

            if (isset($headers["Content-Length"])) {
                if (strlen($dataResult) == $headers["Content-Length"]) {
                    $this->httpRequest->close();
                }
            } else {
                $this->httpRequest->close();
            }
        });

        $this->httpRequest->on('end', function() use ($headers, &$dataResult) {
            $error = null;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            $data = [];

            if ($dataResult === null) return $this->emit('end');

            if (isset($headers['content-type']) && $headers['content-type'] == 'application/json') {
                $jsonData = json_decode($dataResult, true);

                if ($jsonData === null) {
                    $this->emit('error', [json_last_error_msg()]);
                } else {
                    $data = $jsonData;
                }
            } else {
                parse_str($dataResult, $data);
            }

            $this->setContent($dataResult);
            $this->setData($data);
            $this->emit('end');            
        });
    }

    /**
     * Set the data array
     * @param array $data array of data
     */
    public function setData($data)
    {
        $this->data = array_merge($data, $this->data);
    }

    /**
     * Get the data array
     * @return array
     */
    public function getData()
    {
        return $this->data;
    }

    public function __get($name)
    {
        return isset($this->data[$name]) ? $this->data[$name] : false;
    }

    public function __set($name, $value)
    {
        $this->data[$name] = $value;
    }
}


1			<?php
2
3			namespace CapMousse\ReactRestify\Http;
4
5			use React\Http\Request as ReactHttpRequest;
6			use Evenement\EventEmitter;
7
8			class Request extends EventEmitter
9			{
10			/** @var React\Http\Request */
11			public $httpRequest;
12
13			/** @var mixed */
14			private $content;
15
16			/** @var array */
17			private $data = [];
18
19			/**
20			* @param ReactHttpRequest $httpRequest
21			*/
22			public function __construct(ReactHttpRequest $httpRequest)
23			{
24			$this->httpRequest = $httpRequest;
			0 ignored issues – show Documentation Bug introduced 2017-02-13 23:29 UTC by Report Bug Copy Issue Report It seems like `$httpRequest` of type `object<React\Http\Request>` is incompatible with the declared type `object<CapMousse\ReactRe...ttp\React\Http\Request>` of property `$httpRequest`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
25			}
26
27			/**
28			* Set the raw data of the request
29			* @param mixed $content
30			*/
31			public function setContent($content)
32			{
33			$this->content = $content;
34			}
35
36			/**
37			* Set the raw data of the request
38			* @param mixed $content
			0 ignored issues – show Bug introduced 2017-02-13 23:29 UTC by Report Bug Copy Issue Report There is no parameter named `$content`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
39			*/
40			public function getContent()
41			{
42			return $this->content;
43			}
44
45			public function getHeaders()
46			{
47			$headers = array_change_key_case($this->httpRequest->getHeaders(), CASE_LOWER);
48			$headers = array_map(function ($value) {
49			return strtolower($value);
50			}, $headers);
51
52			return $headers;
53			}
54
55			public function parseData()
56			{
57			$headers = $this->getHeaders();
58
59			if (!in_array($this->httpRequest->getMethod(), ['PUT', 'POST'])) {
60			return $this->emit('end');
61			}
62
63			$this->httpRequest->on('data', function($data) use ($headers, &$dataResult) {
64			$dataResult .= $data;
65
66			if (isset($headers["Content-Length"])) {
67			if (strlen($dataResult) == $headers["Content-Length"]) {
68			$this->httpRequest->close();
69			}
70			} else {
71			$this->httpRequest->close();
72			}
73			});
74
75			$this->httpRequest->on('end', function() use ($headers, &$dataResult) {
76			$error = null;
			0 ignored issues – show Unused Code introduced 2017-02-13 23:30 UTC by Report Bug Copy Issue Report `$error` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
77			$data = [];
78
79			if ($dataResult === null) return $this->emit('end');
80
81			if (isset($headers['content-type']) && $headers['content-type'] == 'application/json') {
82			$jsonData = json_decode($dataResult, true);
83
84			if ($jsonData === null) {
85			$this->emit('error', [json_last_error_msg()]);
86			} else {
87			$data = $jsonData;
88			}
89			} else {
90			parse_str($dataResult, $data);
91			}
92
93			$this->setContent($dataResult);
94			$this->setData($data);
95			$this->emit('end');
96			});
97			}
98
99			/**
100			* Set the data array
101			* @param array $data array of data
102			*/
103			public function setData($data)
104			{
105			$this->data = array_merge($data, $this->data);
106			}
107
108			/**
109			* Get the data array
110			* @return array
111			*/
112			public function getData()
113			{
114			return $this->data;
115			}
116
117			public function __get($name)
118			{
119			return isset($this->data[$name]) ? $this->data[$name] : false;
120			}
121
122			public function __set($name, $value)
123			{
124			$this->data[$name] = $value;
125			}
126			}
127

CapMousse / React-Restify

GitHub Access Token became invalid

Branch — dev (718e6b)

Request::parseData() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like