RouteExistsValidator::validate() - Code Metrics - Inspection of "Merge pull request #3 from alpixel/feature/admin-s..." - alpixel/AlpixelMenuBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 02de5b...d3085d )

by Benjamin

created 2016-03-21 15:50 UTC

RouteExistsValidator::validate() C

↳ Parent: RouteExistsValidator

Complexity

Conditions	7
Paths	10

Size

Total Lines	27
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	27
rs	6.7272
cc	7
eloc	18
nc	10
nop	2

<?php

namespace Alpixel\Bundle\MenuBundle\Validator\Constraints;

use Symfony\Bundle\FrameworkBundle\Routing\Router;
use Symfony\Component\Validator\Constraint;
use Symfony\Component\Validator\ConstraintValidator;

class RouteExistsValidator extends ConstraintValidator
{
    private $router;

    public function __construct(Router $router)
    {
        $this->router = $router;
    }

    public function validate($value, Constraint $constraint)
    {
        $match = false;
        if (strpos($value, '/') === 0) {
            $routeCollection = $this->router->getRouteCollection()->all();
            foreach ($routeCollection as $name => $route) {
                if ($match = $route->getPath() === $value) {
                    break;
                }
            }
        } else {
            $handle = curl_init($value);
            curl_setopt($handle,  CURLOPT_RETURNTRANSFER, TRUE);
            $response = curl_exec($handle);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            $httpCode = curl_getinfo($handle, CURLINFO_HTTP_CODE);

            if ($httpCode >= 200 && $httpCode < 300) {
                $match = true;
            }
        }

        if ($match === false) {
            $this->context->buildViolation($constraint->message)
                ->setParameter('%string%', $value)
                ->addViolation();
        }
    }
}


1			<?php
2
3			namespace Alpixel\Bundle\MenuBundle\Validator\Constraints;
4
5			use Symfony\Bundle\FrameworkBundle\Routing\Router;
6			use Symfony\Component\Validator\Constraint;
7			use Symfony\Component\Validator\ConstraintValidator;
8
9			class RouteExistsValidator extends ConstraintValidator
10			{
11			private $router;
12
13			public function __construct(Router $router)
14			{
15			$this->router = $router;
16			}
17
18			public function validate($value, Constraint $constraint)
19			{
20			$match = false;
21			if (strpos($value, '/') === 0) {
22			$routeCollection = $this->router->getRouteCollection()->all();
23			foreach ($routeCollection as $name => $route) {
24			if ($match = $route->getPath() === $value) {
25			break;
26			}
27			}
28			} else {
29			$handle = curl_init($value);
30			curl_setopt($handle, CURLOPT_RETURNTRANSFER, TRUE);
31			$response = curl_exec($handle);
			0 ignored issues – show Unused Code introduced 2016-03-18 17:27 UTC by Report Bug Copy Issue Report `$response` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
32			$httpCode = curl_getinfo($handle, CURLINFO_HTTP_CODE);
33
34			if ($httpCode >= 200 && $httpCode < 300) {
35			$match = true;
36			}
37			}
38
39			if ($match === false) {
40			$this->context->buildViolation($constraint->message)
41			->setParameter('%string%', $value)
42			->addViolation();
43			}
44			}
45			}
46

alpixel / AlpixelMenuBundle

Push — master ( 02de5b...d3085d )

RouteExistsValidator::validate() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like