TestController::foo() - Code Metrics - Inspection of "Make sure FormExtracotrs are disabled unless we ex..." - php-translation/extractor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b64f8f...06557d )

by Tobias

created 2018-01-06 12:26 UTC

TestController::foo() A

↳ Parent: TestController

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	7
nc	1
nop	0

<?php


namespace App\Controller\Test;


use Symfony\Bundle\FrameworkBundle\Controller\Controller;

class TestController extends Controller
{
    public function foo()
    {
        $foo = [
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            ['label' => $this->get('translator')->trans('this.should.be.translated.without.errors')], //ERROR: Form label is not a scalar string
            [/** @Ignore */'label' => $this->get('translator')->trans('this.should.be.translated.without.errors.and.now.it.works')],
            ['label_' => $this->get('translator')->trans('this.works.correctly')],
            ['label' => $this->shouldBeIgnored()], //ERROR: Form label is not a scalar string
            [/** @Ignore */'label' => $this->shouldBeIgnoredAndGoesWithoutError()],
        ];
    }

    private function shouldBeIgnored()
    {
    }

    private function shouldBeIgnoredAndGoesWithoutError()
    {
    }
}


1			<?php
2
3
4			namespace App\Controller\Test;
5
6
7			use Symfony\Bundle\FrameworkBundle\Controller\Controller;
8
9			class TestController extends Controller
10			{
11			public function foo()
12			{
13			$foo = [
			0 ignored issues – show Unused Code introduced 2018-01-06 12:29 UTC by Report Bug Copy Issue Report `$foo` 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...
14			['label' => $this->get('translator')->trans('this.should.be.translated.without.errors')], //ERROR: Form label is not a scalar string
15			[/** @Ignore */'label' => $this->get('translator')->trans('this.should.be.translated.without.errors.and.now.it.works')],
16			['label_' => $this->get('translator')->trans('this.works.correctly')],
17			['label' => $this->shouldBeIgnored()], //ERROR: Form label is not a scalar string
18			[/** @Ignore */'label' => $this->shouldBeIgnoredAndGoesWithoutError()],
19			];
20			}
21
22			private function shouldBeIgnored()
23			{
24			}
25
26			private function shouldBeIgnoredAndGoesWithoutError()
27			{
28			}
29			}
30

php-translation / extractor

Push — master ( b64f8f...06557d )

TestController::foo() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like