FiController::__construct() - Code Metrics - Inspection of "Merge branch 'release/v0.1.41'" - ComuneFI/BiCoreBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 1d619d...8f091f )

by Andrea

created 2018-12-15 21:19 UTC

FiController::__construct() A

↳ Parent: FiController

Complexity

Conditions	3
Paths	4

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes

Metric	Value
cc	3
eloc	9
nc	4
nop	2
dl	0
loc	14
ccs	10
cts	10
cp	1
crap	3
rs	9.9666
c	0
b	0
f	0

<?php

namespace Cdf\BiCoreBundle\Controller;

use Cdf\BiCoreBundle\Utils\Permessi\PermessiUtils;
use Doctrine\Common\Persistence\ObjectManager;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;

class FiController extends AbstractController
{
    use FiCoreControllerTrait, FiCoreCrudControllerTrait, FiCoreTabellaControllerTrait;


    protected $bundle;
    protected $controller;
    protected $permessi;

    public function __construct(TokenStorageInterface $user, PermessiUtils $permessi)
    {
        $matches = array();
$a = "a";
$ab = "ab";
$abc = "abc";
        $controllo = new \ReflectionClass(get_class($this));

        preg_match('/(.*)\\\(.*)\\\Controller\\\(.*)Controller/', $controllo->name, $matches);
        if (count($matches) == 0) {
            preg_match('/(.*)(.*)\\\Controller\\\(.*)Controller/', $controllo->name, $matches);
        }

        $this->bundle = ($matches[count($matches) - 2] ? $matches[count($matches) - 2] : $matches[count($matches) - 3]);
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->controller = $matches[count($matches) - 1];
        $this->permessi = $permessi;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->user = $user;
$a = "a";
$ab = "ab";
$abc = "abc";
    }
    protected function getBundle()
    {
        return $this->bundle;
    }
    protected function getController()
    {
        return $this->controller;
    }
    protected function getPermessi()
    {
        return $this->permessi;
    }
    protected function getUser()
    {
        return $this->user->getToken()->getUser();
    }
}


1		<?php
2
3		namespace Cdf\BiCoreBundle\Controller;
4
5		use Cdf\BiCoreBundle\Utils\Permessi\PermessiUtils;
6		use Doctrine\Common\Persistence\ObjectManager;
7		use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
8		use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
9
10		class FiController extends AbstractController
11		{
12		use FiCoreControllerTrait, FiCoreCrudControllerTrait, FiCoreTabellaControllerTrait;
		0 ignored issues – show Bug introduced 2018-12-02 21:15 UTC by Report Bug Copy Issue Report The trait `Cdf\BiCoreBundle\Control...CoreCrudControllerTrait` requires the property `$request` which is not provided by `Cdf\BiCoreBundle\Controller\FiController`. Loading history...
13
14		protected $bundle;
15		protected $controller;
16		protected $permessi;
17
18	17	public function __construct(TokenStorageInterface $user, PermessiUtils $permessi)
19		{
20	17	$matches = array();
		0 ignored issues – show Coding Style introduced 2018-12-15 21:07 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
21	17	$controllo = new \ReflectionClass(get_class($this));
22
23	17	preg_match('/(.)\\\(.)\\\Controller\\\(.*)Controller/', $controllo->name, $matches);
24	17	if (count($matches) == 0) {
25	10	preg_match('/(.)(.)\\\Controller\\\(.*)Controller/', $controllo->name, $matches);
26		}
27
28	17	$this->bundle = ($matches[count($matches) - 2] ? $matches[count($matches) - 2] : $matches[count($matches) - 3]);
		0 ignored issues – show Coding Style introduced 2018-11-17 22:42 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
29	17	$this->controller = $matches[count($matches) - 1];
30	17	$this->permessi = $permessi;
		0 ignored issues – show Coding Style introduced 2018-12-15 21:07 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
31	17	$this->user = $user;
		0 ignored issues – show Bug Best Practice introduced 2018-12-15 21:07 UTC by Report Bug Copy Issue Report The property `user` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history... Coding Style introduced 2018-12-15 21:07 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
32	17	}
33	13	protected function getBundle()
34		{
35	13	return $this->bundle;
36		}
37	17	protected function getController()
38		{
39	17	return $this->controller;
40		}
41	17	protected function getPermessi()
42		{
43	17	return $this->permessi;
44		}
45	7	protected function getUser()
46		{
47	7	return $this->user->getToken()->getUser();
48		}
49		}
50

ComuneFI / BiCoreBundle

Push — master ( 1d619d...8f091f )

FiController::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like