RequestRouteCollection::all() - 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

RequestRouteCollection::all() B

↳ Parent: RequestRouteCollection

Complexity

Conditions	6
Paths	3

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	6

Importance

Changes

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

<?php declare(strict_types = 1);

namespace Venta\Routing;

use Psr\Http\Message\ServerRequestInterface;
use Venta\Contracts\Routing\ImmutableRouteCollection as RouteCollectionContract;

/**
 * Class RequestRouteCollection
 *
 * @package Venta\Routing
 */
final class RequestRouteCollection implements RouteCollectionContract
{

    /**
     * @var ServerRequestInterface
     */
    private $request;

    /**
     * @var RouteCollectionContract
     */
    private $routes;

    /**
     * RequestRouteCollection constructor.
     *
     * @param ServerRequestInterface $request
     * @param RouteCollectionContract $routes
     */
    public function __construct(ServerRequestInterface $request, RouteCollectionContract $routes)
    {
        $this->request = $request;
        $this->routes = $routes;
    }

    /**
     * @inheritDoc
     */
    public function findByName(string $routeName)
    {
        return $this->routes->findByName($routeName);
    }

    /**
     * @inheritDoc
     */
    public function all(): array
    {
        $routes = [];
        foreach ($this->routes->all() as $route) {
            if ((!$route->host() || $route->host() === $this->request->getUri()->getHost())
                && (!$route->scheme() || $route->scheme() === $this->request->getUri()->getScheme())
            ) {
                $routes[] = $route;
            }
        }

        return $routes;
    }

}


1		<?php declare(strict_types = 1);
2
3		namespace Venta\Routing;
4
5		use Psr\Http\Message\ServerRequestInterface;
6		use Venta\Contracts\Routing\ImmutableRouteCollection as RouteCollectionContract;
7
8		/**
9		* Class RequestRouteCollection
10		*
11		* @package Venta\Routing
12		*/
13		final class RequestRouteCollection implements RouteCollectionContract
14		{
15
16		/**
17		* @var ServerRequestInterface
18		*/
19		private $request;
20
21		/**
22		* @var RouteCollectionContract
23		*/
24		private $routes;
25
26		/**
27		* RequestRouteCollection constructor.
28		*
29		* @param ServerRequestInterface $request
30		* @param RouteCollectionContract $routes
31		*/
32	4	public function __construct(ServerRequestInterface $request, RouteCollectionContract $routes)
33		{
34	4	$this->request = $request;
35	4	$this->routes = $routes;
36	4	}
37
38		/**
39		* @inheritDoc
40		*/
41		public function findByName(string $routeName)
42		{
43		return $this->routes->findByName($routeName);
44		}
45
46		/**
47		* @inheritDoc
48		*/
49	2	public function all(): array
50		{
51	2	$routes = [];
52	2	foreach ($this->routes->all() as $route) {
53	1	if ((!$route->host() \|\| $route->host() === $this->request->getUri()->getHost())
54	1	&& (!$route->scheme() \|\| $route->scheme() === $this->request->getUri()->getScheme())
55		) {
56	1	$routes[] = $route;
57		}
58		}
59
60	2	return $routes;
61		}
62
63		}
64

venta / framework

Push — master ( 73c405...d57cd2 )

RequestRouteCollection::all() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like