Psr7 - Code Metrics - Inspection of "System will now support requests with following Co..." - nazar-pc/CleverStyle-Framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 61475a...8376f2 )

by Nazar

created 2016-03-22 02:39 UTC

Psr7 A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	125
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	4
Bugs	1	Features	2

Metric	Value
wmc	22
c	4
b	1
f	2
lcom	1
cbo	0
dl	0
loc	125
rs	10

5 Methods

Rating	Name	Size	Complexity
C	from_psr7_server()	29	8
A	from_psr7_query()	3	1
B	from_psr7_files_internal()	30	6
A	from_psr7()	6	1
B	from_psr7_data_and_files()	30	6

<?php
/**
 * @package   CleverStyle CMS
 * @author    Nazar Mokrynskyi <[email protected]>
 * @copyright Copyright (c) 2016, Nazar Mokrynskyi
 * @license   MIT License, see license.txt
 */
namespace cs\Request;
use
	Exception,
	cs\ExitException;

trait Psr7 {
	/**
	 * Initialize request from PSR-7 request object
	 *
	 * @param \Psr\Http\Message\ServerRequestInterface $request
	 *
	 * @throws \cs\ExitException
	 */
	function from_psr7 ($request) {
		$this->from_psr7_server($request);
		$this->from_psr7_query($request);
		$this->from_psr7_data_and_files($request);
		$this->init_route();
	}
	/**
	 * @param \Psr\Http\Message\ServerRequestInterface $request
	 */
	protected function from_psr7_server ($request) {
		$uri          = $request->getUri();
		$this->method = $request->getMethod();
		$this->host   = $uri->getHost();
		$this->scheme = $uri->getScheme();
		$this->secure = $this->scheme == 'https';
		if (
			(!$this->secure && $uri->getPort() != 80) ||
			($this->secure && $uri->getPort() != 443)
		) {
			$this->host .= ':'.$uri->getPort();
		}
		$this->protocol     = 'HTTP/'.$request->getProtocolVersion();
		$this->path         = $uri->getPath();
		$this->query_string = $uri->getQuery();
		/** @noinspection NestedTernaryOperatorInspection */
		$this->uri         = $this->path.($this->query_string ? "?$this->query_string" : '') ?: '/';
		$this->remote_addr = @$request->getServerParams()['REMOTE_ADDR'] ?: '127.0.0.1';
		$this->ip          = $this->ip(
			[
				'HTTP_X_FORWARDED_FOR'     => $request->getHeaderLine('x-forwarded-for'),
				'HTTP_CLIENT_IP'           => $request->getHeaderLine('client-ip'),
				'HTTP_X_FORWARDED'         => $request->getHeaderLine('x-forwarded'),
				'HTTP_X_CLUSTER_CLIENT_IP' => $request->getHeaderLine('x-cluster-client-ip'),
				'HTTP_FORWARDED_FOR'       => $request->getHeaderLine('forwarded-for'),
				'HTTP_FORWARDED'           => $request->getHeaderLine('forwarded')
			]
		);
	}
	/**
	 * @param \Psr\Http\Message\ServerRequestInterface $request
	 */
	protected function from_psr7_query ($request) {
		$this->query = $request->getQueryParams();
	}
	/**
	 * @todo Implement custom stream wrapper for files and data in general in order to avoid data duplication
	 *
	 * @param \Psr\Http\Message\ServerRequestInterface $request
	 *
	 * @throws ExitException
	 */
	protected function from_psr7_data_and_files ($request) {
		$data         = [];
		$data_stream  = null;
		$content_type = $this->header('content-type');
		$body_stream  = $request->getBody();
		if (preg_match('#^application/([^+\s]+\+)?json#', $content_type)) {
			$data = _json_decode((string)$body_stream) ?: [];
		} elseif (strpos($content_type, 'application/x-www-form-urlencoded') === 0) {
			@parse_str((string)$body_stream, $data);
		} else {
			try {
				$position = $body_stream->tell();
				$body_stream->rewind();
				$data_stream = fopen('php://temp', 'w+b');
				while (!$body_stream->eof()) {
					fwrite($data_stream, $body_stream->read(1024));
				}
				$body_stream->seek($position);
			} catch (Exception $e) {
				// Do nothing
			}
		}
		$this->init_data_and_files(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
			$data,
			$this->from_psr7_files_internal(
				$request->getUploadedFiles()
			),
			$data_stream
		);
	}
	/**
	 * @param array|\Psr\Http\Message\UploadedFileInterface $files
	 *
	 * @return array|\Psr\Http\Message\UploadedFileInterface
	 */
	protected function from_psr7_files_internal ($files) {
		if (is_array($files)) {
			foreach ($files as $field => &$file) {
				$file = $this->from_psr7_files_internal($file);
				if (!$file) {
					unset($files[$field]);
				}
			}
			return $files;
		}
		try {
			$source_file_stream = $files->getStream();
			$position           = $source_file_stream->tell();
			$source_file_stream->rewind();
			$file_stream = fopen('php://temp', 'w+b');
			while (!$source_file_stream->eof()) {
				fwrite($file_stream, $source_file_stream->read(1024));
			}
			$source_file_stream->seek($position);
		} catch (Exception $e) {
			return [];
		}
		return [
			'name'   => $files->getClientFilename(),
			'type'   => $files->getClientMediaType(),
			'size'   => $files->getSize(),
			'stream' => $file_stream,
			'error'  => $files->getError()
		];
	}
}


1			<?php
2			/**
3			* @package CleverStyle CMS
4			* @author Nazar Mokrynskyi <[email protected]>
5			* @copyright Copyright (c) 2016, Nazar Mokrynskyi
6			* @license MIT License, see license.txt
7			*/
8			namespace cs\Request;
9			use
10			Exception,
11			cs\ExitException;
12
13			trait Psr7 {
14			/**
15			* Initialize request from PSR-7 request object
16			*
17			* @param \Psr\Http\Message\ServerRequestInterface $request
18			*
19			* @throws \cs\ExitException
20			*/
21			function from_psr7 ($request) {
22			$this->from_psr7_server($request);
23			$this->from_psr7_query($request);
24			$this->from_psr7_data_and_files($request);
25			$this->init_route();
26			}
27			/**
28			* @param \Psr\Http\Message\ServerRequestInterface $request
29			*/
30			protected function from_psr7_server ($request) {
31			$uri = $request->getUri();
32			$this->method = $request->getMethod();
33			$this->host = $uri->getHost();
34			$this->scheme = $uri->getScheme();
35			$this->secure = $this->scheme == 'https';
36			if (
37			(!$this->secure && $uri->getPort() != 80) \|\|
38			($this->secure && $uri->getPort() != 443)
39			) {
40			$this->host .= ':'.$uri->getPort();
41			}
42			$this->protocol = 'HTTP/'.$request->getProtocolVersion();
43			$this->path = $uri->getPath();
44			$this->query_string = $uri->getQuery();
45			/** @noinspection NestedTernaryOperatorInspection */
46			$this->uri = $this->path.($this->query_string ? "?$this->query_string" : '') ?: '/';
47			$this->remote_addr = @$request->getServerParams()['REMOTE_ADDR'] ?: '127.0.0.1';
48			$this->ip = $this->ip(
49			[
50			'HTTP_X_FORWARDED_FOR' => $request->getHeaderLine('x-forwarded-for'),
51			'HTTP_CLIENT_IP' => $request->getHeaderLine('client-ip'),
52			'HTTP_X_FORWARDED' => $request->getHeaderLine('x-forwarded'),
53			'HTTP_X_CLUSTER_CLIENT_IP' => $request->getHeaderLine('x-cluster-client-ip'),
54			'HTTP_FORWARDED_FOR' => $request->getHeaderLine('forwarded-for'),
55			'HTTP_FORWARDED' => $request->getHeaderLine('forwarded')
56			]
57			);
58			}
59			/**
60			* @param \Psr\Http\Message\ServerRequestInterface $request
61			*/
62			protected function from_psr7_query ($request) {
63			$this->query = $request->getQueryParams();
64			}
65			/**
66			* @todo Implement custom stream wrapper for files and data in general in order to avoid data duplication
67			*
68			* @param \Psr\Http\Message\ServerRequestInterface $request
69			*
70			* @throws ExitException
71			*/
72			protected function from_psr7_data_and_files ($request) {
73			$data = [];
74			$data_stream = null;
75			$content_type = $this->header('content-type');
76			$body_stream = $request->getBody();
77			if (preg_match('#^application/([^+\s]+\+)?json#', $content_type)) {
78			$data = _json_decode((string)$body_stream) ?: [];
79			} elseif (strpos($content_type, 'application/x-www-form-urlencoded') === 0) {
80			@parse_str((string)$body_stream, $data);
81			} else {
82			try {
83			$position = $body_stream->tell();
84			$body_stream->rewind();
85			$data_stream = fopen('php://temp', 'w+b');
86			while (!$body_stream->eof()) {
87			fwrite($data_stream, $body_stream->read(1024));
88			}
89			$body_stream->seek($position);
90			} catch (Exception $e) {
91			// Do nothing
92			}
93			}
94			$this->init_data_and_files(
			1 ignored issue – show Bug introduced 2016-03-22 02:43 UTC by Report Bug Copy Issue Report It seems like `init_data_and_files()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
95			$data,
96			$this->from_psr7_files_internal(
97			$request->getUploadedFiles()
98			),
99			$data_stream
100			);
101			}
102			/**
103			* @param array\|\Psr\Http\Message\UploadedFileInterface $files
104			*
105			* @return array\|\Psr\Http\Message\UploadedFileInterface
106			*/
107			protected function from_psr7_files_internal ($files) {
108			if (is_array($files)) {
109			foreach ($files as $field => &$file) {
110			$file = $this->from_psr7_files_internal($file);
111			if (!$file) {
112			unset($files[$field]);
113			}
114			}
115			return $files;
116			}
117			try {
118			$source_file_stream = $files->getStream();
119			$position = $source_file_stream->tell();
120			$source_file_stream->rewind();
121			$file_stream = fopen('php://temp', 'w+b');
122			while (!$source_file_stream->eof()) {
123			fwrite($file_stream, $source_file_stream->read(1024));
124			}
125			$source_file_stream->seek($position);
126			} catch (Exception $e) {
127			return [];
128			}
129			return [
130			'name' => $files->getClientFilename(),
131			'type' => $files->getClientMediaType(),
132			'size' => $files->getSize(),
133			'stream' => $file_stream,
134			'error' => $files->getError()
135			];
136			}
137			}
138

nazar-pc / CleverStyle-Framework

Push — master ( 61475a...8376f2 )

Psr7 A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

Duplication Side-by-Side

Filter issues like