RouterTrait - Code Metrics - Inspection of "update travis" - bytic/router - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( df0330...a615ca )

by Gabriel

created 2020-05-15 10:41 UTC

RouterTrait A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

94.44%

Importance

Changes

Metric	Value
dl	0
loc	41
ccs	17
cts	18
cp	0.9444
rs	10
c	0
b	0
f	0
wmc	3
lcom	0
cbo	3

2 Methods

Rating	Name	Duplication	Size	Complexity
A	registerRouter()	0	12	1
A	newRouter()	0	22	2

<?php

namespace Nip\Router\ServiceProvider\Traits;

use Nip\Http\Request;
use Nip\Router\RequestContext;
use Nip\Router\Router;
use Symfony\Component\Routing\RouterInterface;

/**
 * Trait RouterTrait
 * @package Nip\Router\ServiceProvider\Traits
 */
trait RouterTrait
{
    public function registerRouter()
    {
        $this->getContainer()->share('router', function () {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            return $this->newRouter();
        });

        $this->getContainer()->add(RouterInterface::class, Router::class);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $this->getContainer()->share(Router::class, function () {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            return $this->getContainer()->get('router');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        });
    }

    /**
     * @return Router
     */
    public function newRouter()
    {
        $app = $this->getContainer()->get('app');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        /** @var Router $router */
        $router = $this->getContainer()->get(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            RouterInterface::class,
            [
                'loader' => $this->getContainer()->get('routing.loader'),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
                'resource' => 'routes.php',
                'options' => [
                    'cache_dir' => $app->getCachedRoutesPath(),
                ]
            ]
        );

        $request = Request::instance();
        if ($request instanceof Request) {
            $router->setContext((new RequestContext())->fromRequest($request));
        }
        return $router;
    }
}


1		<?php
2
3		namespace Nip\Router\ServiceProvider\Traits;
4
5		use Nip\Http\Request;
6		use Nip\Router\RequestContext;
7		use Nip\Router\Router;
8		use Symfony\Component\Routing\RouterInterface;
9
10		/**
11		* Trait RouterTrait
12		* @package Nip\Router\ServiceProvider\Traits
13		*/
14		trait RouterTrait
15		{
16	3	public function registerRouter()
17		{
18		$this->getContainer()->share('router', function () {
		0 ignored issues – show Bug introduced 2019-03-27 07:21 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
19	3	return $this->newRouter();
20	3	});
21
22	3	$this->getContainer()->add(RouterInterface::class, Router::class);
		0 ignored issues – show Bug introduced 2019-11-05 19:54 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
23
24		$this->getContainer()->share(Router::class, function () {
		0 ignored issues – show Bug introduced 2019-11-05 19:54 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
25		return $this->getContainer()->get('router');
		0 ignored issues – show Bug introduced 2019-11-05 19:54 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
26	3	});
27	3	}
28
29		/**
30		* @return Router
31		*/
32	3	public function newRouter()
33		{
34	3	$app = $this->getContainer()->get('app');
		0 ignored issues – show Bug introduced 2020-05-15 10:41 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
35
36		/** @var Router $router */
37	3	$router = $this->getContainer()->get(
		0 ignored issues – show Bug introduced 2020-05-15 10:41 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
38	3	RouterInterface::class,
39		[
40	3	'loader' => $this->getContainer()->get('routing.loader'),
		0 ignored issues – show Bug introduced 2020-05-15 10:41 UTC by Report Bug Copy Issue Report It seems like `getContainer()` 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...
41	3	'resource' => 'routes.php',
42		'options' => [
43	3	'cache_dir' => $app->getCachedRoutesPath(),
44		]
45		]
46		);
47
48	3	$request = Request::instance();
49	3	if ($request instanceof Request) {
50	3	$router->setContext((new RequestContext())->fromRequest($request));
51		}
52	3	return $router;
53		}
54		}
55

bytic / router

Push — master ( df0330...a615ca )

RouterTrait A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

2 Methods

Duplication Side-by-Side

Filter issues like