UrlGenerator::generate() - Code Metrics - Inspection of "Merge branch '0.9' into develop" - bytic/router - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( f50e96...1d7e99 )

by Gabriel

created 2019-11-05 18:28 UTC

UrlGenerator::generate() A

↳ Parent: UrlGenerator

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
nc	1
nop	3
dl	0
loc	5
ccs	0
cts	3
cp	0
crap	2
rs	10
c	0
b	0
f	0

<?php

namespace Nip\Router\Generator;

use Nip\Request;
use Nip\Router\Generator\Traits\FormattingTrait;
use Nip\Router\Generator\Traits\HasPreviousUrlTrait;
use Nip\Router\Generator\Traits\UrlFunctionTrait;
use Nip\Router\RequestContext;
use Nip\Router\Route\Route;

/**
 * Class UrlGenerator
 * @package Nip\Router\Generator
 * @method RequestContext getContext()
 */
class UrlGenerator extends \Symfony\Component\Routing\Generator\UrlGenerator
{
    use FormattingTrait;
    use UrlFunctionTrait;
    use HasPreviousUrlTrait;

    /**
     * Set the current request instance.
     *
     * @param  Request $request
     */
    public function setRequest(Request $request)
    {
        $context = (new RequestContext())->fromRequest($request);

        $this->cachedRoot = null;
        $this->cachedSchema = null;
        return $this->setContext($context);
    }

    public function generate($name, $parameters = [], $referenceType = self::ABSOLUTE_PATH)
    {
        $name = $this->initDefaultRoute($name, $parameters);
        return parent::generate($name, $parameters, $referenceType);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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


    /**
     * @param $name
     * @param array $params
     * @return null|Route
     */
    protected function initDefaultRoute($name, &$params = [])
    {
        $route = $this->routes->get($name);
        if ($route) {
            return $name;
        }
        $parts = explode(".", $name);
        $count = count($parts);
        if ($count <= 3) {
            if (in_array(reset($parts), app('mvc.modules')->getNames())) {
                $module = array_shift($parts);
                $params['controller'] = isset($parts[0]) ? $parts[0] : null;
                $params['action'] = isset($parts[1]) ? $parts[1] : null;
                $route = $this->routes->get($module . '.default');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
                if ($route) {
                    return $route->getName();
                }
            }
        }
        return $name;
    }
}


1			<?php
2
3			namespace Nip\Router\Generator;
4
5			use Nip\Request;
6			use Nip\Router\Generator\Traits\FormattingTrait;
7			use Nip\Router\Generator\Traits\HasPreviousUrlTrait;
8			use Nip\Router\Generator\Traits\UrlFunctionTrait;
9			use Nip\Router\RequestContext;
10			use Nip\Router\Route\Route;
11
12			/**
13			* Class UrlGenerator
14			* @package Nip\Router\Generator
15			* @method RequestContext getContext()
16			*/
17			class UrlGenerator extends \Symfony\Component\Routing\Generator\UrlGenerator
18			{
19			use FormattingTrait;
20			use UrlFunctionTrait;
21			use HasPreviousUrlTrait;
22
23			/**
24			* Set the current request instance.
25			*
26			* @param Request $request
27			*/
28			public function setRequest(Request $request)
29			{
30			$context = (new RequestContext())->fromRequest($request);
31
32			$this->cachedRoot = null;
33			$this->cachedSchema = null;
34			return $this->setContext($context);
35			}
36
37			public function generate($name, $parameters = [], $referenceType = self::ABSOLUTE_PATH)
38			{
39			$name = $this->initDefaultRoute($name, $parameters);
40			return parent::generate($name, $parameters, $referenceType);
			0 ignored issues – show Documentation introduced 2019-03-26 07:00 UTC by Report Bug Copy Issue Report `$name` is of type `null\|object<Nip\Router\Route\Route>`, 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...
41			}
42
43
44			/**
45			* @param $name
46			* @param array $params
47			* @return null\|Route
48			*/
49			protected function initDefaultRoute($name, &$params = [])
50			{
51			$route = $this->routes->get($name);
52			if ($route) {
53			return $name;
54			}
55			$parts = explode(".", $name);
56			$count = count($parts);
57			if ($count <= 3) {
58			if (in_array(reset($parts), app('mvc.modules')->getNames())) {
59			$module = array_shift($parts);
60			$params['controller'] = isset($parts[0]) ? $parts[0] : null;
61			$params['action'] = isset($parts[1]) ? $parts[1] : null;
62			$route = $this->routes->get($module . '.default');
			0 ignored issues – show Bug introduced 2019-03-26 07:00 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$route` is correct as `$this->routes->get($module . '.default')` (which targets `Symfony\Component\Routing\RouteCollection::get()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
63			if ($route) {
64			return $route->getName();
65			}
66			}
67			}
68			return $name;
69			}
70			}
71

bytic / router

Push — master ( f50e96...1d7e99 )

UrlGenerator::generate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like