Route - Code Metrics - Inspection of "Updating framework to reflect config class changes" - venta/framework - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 73c405...d57cd2 )

by Konstantins

created 2016-12-19 09:54 UTC

Route C

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	378
Duplicated Lines	0 %

Coupling/Cohesion

Components	9
Dependencies	0

Test Coverage

Coverage

75.51%

Importance

Changes

Metric	Value
wmc	35
lcom	9
cbo	0
dl	0
loc	378
ccs	74
cts	98
cp	0.7551
rs	6.4285
c	0
b	0
f	0

28 Methods

Rating	Name	Size	Complexity
A	__construct()	6	1
A	any()	4	1
A	delete()	4	1
A	get()	4	1
A	head()	4	1
A	options()	4	1
A	patch()	4	1
A	post()	4	1
A	put()	4	1
C	compilePath()	45	8
A	secure()	7	1
A	withDomain()	7	1
A	withHost()	7	1
A	withInput()	7	1
A	withMiddleware()	7	1
A	withName()	7	1
A	withPath()	7	1
A	withVariables()	7	1
A	domain()	4	1
A	host()	4	1
A	input()	4	1
A	methods()	4	1
A	middlewares()	4	1
A	name()	4	1
A	path()	4	1
A	responder()	4	1
A	scheme()	4	1
A	variables()	4	1

<?php declare(strict_types = 1);

namespace Venta\Routing;

use InvalidArgumentException;
use Venta\Contracts\Routing\Route as RouteContract;

/**
 * Class Route
 *
 * @package Venta\Routing
 */
class Route implements RouteContract
{
    /**
     * @var string
     */
    private $domain = '';

    /**
     * Host to apply route to.
     *
     * @var string
     */
    private $host = '';

    /**
     * @var string
     */
    private $input = '';

    /**
     * Route allowed methods.
     *
     * @var string[]
     */
    private $methods = [];

    /**
     * List of middleware class names.
     *
     * @var string[]
     */
    private $middlewares = [];

    /**
     * Route name.
     *
     * @var string
     */
    private $name = '';

    /**
     * Route path
     *
     * @var string
     */
    private $path = '';

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

    /**
     * Scheme to apply route to.
     *
     * @var string
     */
    private $scheme = '';

    /**
     * Route variables.
     *
     * @var string[]
     */
    private $variables = [];

    /**
     * Route constructor.
     *
     * @param string[] $methods
     * @param string $path
     * @param string $responder
     */
    public function __construct(array $methods, string $path, string $responder)
    {
        $this->methods = $methods;
        $this->path = '/' . ltrim($path, '/');
        $this->responder = $responder;
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function any(string $path, $responder): Route
    {
        return new static(['HEAD', 'GET', 'POST', 'PUT', 'PATCH', 'OPTIONS', 'DELETE'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function delete(string $path, $responder): Route
    {
        return new static(['DELETE'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function get(string $path, $responder): Route
    {
        return new static(['GET'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function head(string $path, $responder): Route
    {
        return new static(['HEAD'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function options(string $path, $responder): Route
    {
        return new static(['OPTIONS'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function patch(string $path, $responder): Route
    {
        return new static(['PATCH'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function post(string $path, $responder): Route
    {
        return new static(['POST'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param string $path
     * @param callable|string $responder
     * @return Route
     */
    public static function put(string $path, $responder): Route
    {
        return new static(['PUT'], $path, $responder);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @inheritDoc
     */
    public function compilePath(array $variables = []): string
    {
        $path = $this->path();
        foreach ($variables as $key => $value) {
            $pattern = sprintf('~%s~x', sprintf('\{\s*%s\s*(?::\s*([^{}]*(?:\{(?-1)\}[^{}]*)*))?\}', preg_quote($key)));
            preg_match($pattern, $path, $matches);
            if (isset($matches[1]) && !preg_match('/' . $matches[1] . '/', (string)$value)) {
                throw new InvalidArgumentException(
                    sprintf('Substitution value "%s" does not match "%s" parameter "%s" pattern.',
                        $value,
                        $key,
                        $matches[1]
                    )
                );
            }
            $path = preg_replace($pattern, $value, $path);
        }
        // 1. remove patterns for named prameters
        // 2. remove optional segments' ending delimiters
        // 3. split path into an array of optional segments and remove those
        //    containing unsubstituted parameters starting from the last segment
        $path = preg_replace('/{(\w+):(.+?)}/', '{$1}', $path);
        $path = str_replace(']', '', $path);
        $segments = array_reverse(explode('[', $path));
        foreach ($segments as $n => $segment) {
            if (strpos($segment, '{') !== false) {
                if (isset($segments[$n - 1])) {
                    throw new InvalidArgumentException(
                        'Optional segments with unsubstituted parameters cannot '
                        . 'contain segments with substituted parameters when using FastRoute'
                    );
                }
                unset($segments[$n]);
                if (count($segments) == 0) {
                    preg_match('/{.+}/', $segment, $params);
                    throw new InvalidArgumentException(
                        sprintf('Parameter "%s" is mandatory', $params[0] ?? $segment)
                    );
                }
            }
        }
        $path = implode('', array_reverse($segments));

        return $path;
    }

    /**
     * @inheritDoc
     */
    public function domain(): string
    {
        return $this->domain;
    }

    /**
     * @inheritDoc
     */
    public function host(): string
    {
        return $this->host;
    }

    /**
     * @inheritDoc
     */
    public function input(): string
    {
        return $this->input;
    }

    /**
     * @inheritDoc
     */
    public function methods(): array
    {
        return $this->methods;
    }

    /**
     * @inheritDoc
     */
    public function middlewares(): array
    {
        return $this->middlewares;
    }

    /**
     * @inheritDoc
     */
    public function name(): string
    {
        return $this->name;
    }

    /**
     * @inheritDoc
     */
    public function path(): string
    {
        return $this->path;
    }

    /**
     * @inheritDoc
     */
    public function responder(): string
    {
        return $this->responder;
    }

    /**
     * @inheritDoc
     */
    public function scheme(): string
    {
        return $this->scheme;
    }

    /**
     * @inheritDoc
     */
    public function variables(): array
    {
        return $this->variables;
    }

    /**
     * @inheritDoc
     */
    public function secure(): RouteContract
    {
        $route = clone $this;
        $route->scheme = 'https';

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withDomain(string $domainClass): RouteContract
    {
        $route = clone $this;
        $route->domain = $domainClass;

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withHost(string $host): RouteContract
    {
        $route = clone $this;
        $route->host = $host;

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withInput(string $inputClass): RouteContract
    {
        $route = clone $this;
        $route->input = $inputClass;

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withMiddleware(string $middleware): RouteContract
    {
        $route = clone $this;
        $route->middlewares[] = $middleware;

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withName(string $name): RouteContract
    {
        $route = clone $this;
        $route->name = $name;

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withPath(string $path): RouteContract
    {
        $route = clone $this;
        $route->path = $path;

        return $route;
    }

    /**
     * @inheritDoc
     */
    public function withVariables(array $variables): RouteContract
    {
        $route = clone $this;
        $route->variables = $variables;

        return $route;
    }

}

1		<?php declare(strict_types = 1);
2
3		namespace Venta\Routing;
4
5		use InvalidArgumentException;
6		use Venta\Contracts\Routing\Route as RouteContract;
7
8		/**
9		* Class Route
10		*
11		* @package Venta\Routing
12		*/
13		class Route implements RouteContract
14		{
15		/**
16		* @var string
17		*/
18		private $domain = '';
19
20		/**
21		* Host to apply route to.
22		*
23		* @var string
24		*/
25		private $host = '';
26
27		/**
28		* @var string
29		*/
30		private $input = '';
31
32		/**
33		* Route allowed methods.
34		*
35		* @var string[]
36		*/
37		private $methods = [];
38
39		/**
40		* List of middleware class names.
41		*
42		* @var string[]
43		*/
44		private $middlewares = [];
45
46		/**
47		* Route name.
48		*
49		* @var string
50		*/
51		private $name = '';
52
53		/**
54		* Route path
55		*
56		* @var string
57		*/
58		private $path = '';
59
60		/**
61		* @var string
62		*/
63		private $responder;
64
65		/**
66		* Scheme to apply route to.
67		*
68		* @var string
69		*/
70		private $scheme = '';
71
72		/**
73		* Route variables.
74		*
75		* @var string[]
76		*/
77		private $variables = [];
78
79		/**
80		* Route constructor.
81		*
82		* @param string[] $methods
83		* @param string $path
84		* @param string $responder
85		*/
86	17	public function __construct(array $methods, string $path, string $responder)
87		{
88	17	$this->methods = $methods;
89	17	$this->path = '/' . ltrim($path, '/');
90	17	$this->responder = $responder;
91	17	}
92
93		/**
94		* @param string $path
95		* @param callable\|string $responder
96		* @return Route
97		*/
98		public static function any(string $path, $responder): Route
99		{
100		return new static(['HEAD', 'GET', 'POST', 'PUT', 'PATCH', 'OPTIONS', 'DELETE'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
101		}
102
103		/**
104		* @param string $path
105		* @param callable\|string $responder
106		* @return Route
107		*/
108		public static function delete(string $path, $responder): Route
109		{
110		return new static(['DELETE'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
111		}
112
113		/**
114		* @param string $path
115		* @param callable\|string $responder
116		* @return Route
117		*/
118		public static function get(string $path, $responder): Route
119		{
120		return new static(['GET'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
121		}
122
123		/**
124		* @param string $path
125		* @param callable\|string $responder
126		* @return Route
127		*/
128		public static function head(string $path, $responder): Route
129		{
130		return new static(['HEAD'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
131		}
132
133		/**
134		* @param string $path
135		* @param callable\|string $responder
136		* @return Route
137		*/
138		public static function options(string $path, $responder): Route
139		{
140		return new static(['OPTIONS'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
141		}
142
143		/**
144		* @param string $path
145		* @param callable\|string $responder
146		* @return Route
147		*/
148		public static function patch(string $path, $responder): Route
149		{
150		return new static(['PATCH'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
151		}
152
153		/**
154		* @param string $path
155		* @param callable\|string $responder
156		* @return Route
157		*/
158		public static function post(string $path, $responder): Route
159		{
160		return new static(['POST'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
161		}
162
163		/**
164		* @param string $path
165		* @param callable\|string $responder
166		* @return Route
167		*/
168		public static function put(string $path, $responder): Route
169		{
170		return new static(['PUT'], $path, $responder);
		0 ignored issues – show Documentation introduced 2016-12-01 17:23 UTC by Report Bug Copy Issue Report `$responder` is of type `callable`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
171		}
172
173		/**
174		* @inheritDoc
175		*/
176	3	public function compilePath(array $variables = []): string
177		{
178	3	$path = $this->path();
179	3	foreach ($variables as $key => $value) {
180	3	$pattern = sprintf('~%s~x', sprintf('\{\s%s\s(?::\s([^{}](?:\{(?-1)\}[^{}])))?\}', preg_quote($key)));
181	3	preg_match($pattern, $path, $matches);
182	3	if (isset($matches[1]) && !preg_match('/' . $matches[1] . '/', (string)$value)) {
183	1	throw new InvalidArgumentException(
184	1	sprintf('Substitution value "%s" does not match "%s" parameter "%s" pattern.',
185		$value,
186		$key,
187	1	$matches[1]
188		)
189		);
190		}
191	3	$path = preg_replace($pattern, $value, $path);
192		}
193		// 1. remove patterns for named prameters
194		// 2. remove optional segments' ending delimiters
195		// 3. split path into an array of optional segments and remove those
196		// containing unsubstituted parameters starting from the last segment
197	3	$path = preg_replace('/{(\w+):(.+?)}/', '{$1}', $path);
198	3	$path = str_replace(']', '', $path);
199	3	$segments = array_reverse(explode('[', $path));
200	3	foreach ($segments as $n => $segment) {
201	3	if (strpos($segment, '{') !== false) {
202	2	if (isset($segments[$n - 1])) {
203		throw new InvalidArgumentException(
204		'Optional segments with unsubstituted parameters cannot '
205		. 'contain segments with substituted parameters when using FastRoute'
206		);
207		}
208	2	unset($segments[$n]);
209	2	if (count($segments) == 0) {
210	2	preg_match('/{.+}/', $segment, $params);
211	2	throw new InvalidArgumentException(
212	3	sprintf('Parameter "%s" is mandatory', $params[0] ?? $segment)
213		);
214		}
215		}
216		}
217	3	$path = implode('', array_reverse($segments));
218
219	3	return $path;
220		}
221
222		/**
223		* @inheritDoc
224		*/
225	2	public function domain(): string
226		{
227	2	return $this->domain;
228		}
229
230		/**
231		* @inheritDoc
232		*/
233	7	public function host(): string
234		{
235	7	return $this->host;
236		}
237
238		/**
239		* @inheritDoc
240		*/
241	2	public function input(): string
242		{
243	2	return $this->input;
244		}
245
246		/**
247		* @inheritDoc
248		*/
249	1	public function methods(): array
250		{
251	1	return $this->methods;
252		}
253
254		/**
255		* @inheritDoc
256		*/
257	1	public function middlewares(): array
258		{
259	1	return $this->middlewares;
260		}
261
262		/**
263		* @inheritDoc
264		*/
265	2	public function name(): string
266		{
267	2	return $this->name;
268		}
269
270		/**
271		* @inheritDoc
272		*/
273	9	public function path(): string
274		{
275	9	return $this->path;
276		}
277
278		/**
279		* @inheritDoc
280		*/
281	1	public function responder(): string
282		{
283	1	return $this->responder;
284		}
285
286		/**
287		* @inheritDoc
288		*/
289	7	public function scheme(): string
290		{
291	7	return $this->scheme;
292		}
293
294		/**
295		* @inheritDoc
296		*/
297		public function variables(): array
298		{
299		return $this->variables;
300		}
301
302		/**
303		* @inheritDoc
304		*/
305	3	public function secure(): RouteContract
306		{
307	3	$route = clone $this;
308	3	$route->scheme = 'https';
309
310	3	return $route;
311		}
312
313		/**
314		* @inheritDoc
315		*/
316	1	public function withDomain(string $domainClass): RouteContract
317		{
318	1	$route = clone $this;
319	1	$route->domain = $domainClass;
320
321	1	return $route;
322		}
323
324		/**
325		* @inheritDoc
326		*/
327	3	public function withHost(string $host): RouteContract
328		{
329	3	$route = clone $this;
330	3	$route->host = $host;
331
332	3	return $route;
333		}
334
335		/**
336		* @inheritDoc
337		*/
338	1	public function withInput(string $inputClass): RouteContract
339		{
340	1	$route = clone $this;
341	1	$route->input = $inputClass;
342
343	1	return $route;
344		}
345
346		/**
347		* @inheritDoc
348		*/
349	1	public function withMiddleware(string $middleware): RouteContract
350		{
351	1	$route = clone $this;
352	1	$route->middlewares[] = $middleware;
353
354	1	return $route;
355		}
356
357		/**
358		* @inheritDoc
359		*/
360	1	public function withName(string $name): RouteContract
361		{
362	1	$route = clone $this;
363	1	$route->name = $name;
364
365	1	return $route;
366		}
367
368		/**
369		* @inheritDoc
370		*/
371	5	public function withPath(string $path): RouteContract
372		{
373	5	$route = clone $this;
374	5	$route->path = $path;
375
376	5	return $route;
377		}
378
379		/**
380		* @inheritDoc
381		*/
382		public function withVariables(array $variables): RouteContract
383		{
384		$route = clone $this;
385		$route->variables = $variables;
386
387		return $route;
388		}
389
390		}

venta / framework

Push — master ( 73c405...d57cd2 )

Route C

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

28 Methods

Duplication Side-by-Side

Filter issues like