Passed
Push — feat/manage-settings ( 112b09...dbdc5f )
by
unknown
01:41
created

GetCompany::__construct()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 8
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 2
c 0
b 0
f 0
nc 1
nop 3
dl 0
loc 8
rs 10
1
<?php
2
3
declare(strict_types=1);
4
5
/*
6
 * This file is part of the G.L.S.R. Apps package.
7
 *
8
 * (c) Dev-Int Création <[email protected]>.
9
 *
10
 * For the full copyright and license information, please view the LICENSE
11
 * file that was distributed with this source code.
12
 */
13
14
namespace Company\UI\Controller\Company;
15
16
use Company\Application\Company\DTO\OutputCompany;
17
use Company\Application\Company\Query\GetCompany as GetCompanyGuery;
18
use Core\Controller\AbstractController;
19
use Symfony\Component\HttpFoundation\Response;
0 ignored issues
show
Bug introduced by
The type Symfony\Component\HttpFoundation\Response was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
20
21
class GetCompany extends AbstractController
22
{
23
    public function __invoke(string $uuid): Response
24
    {
25
        /** @var OutputCompany $company */
26
        $company = $this->handle(new GetCompanyGuery());
0 ignored issues
show
Bug introduced by
The call to Company\Application\Comp...tCompany::__construct() has too few arguments starting with company. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

26
        $company = $this->handle(/** @scrutinizer ignore-call */ new GetCompanyGuery());

This check compares calls to functions or methods with their respective definitions. If the call has less arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
27
28
        return $this->response($this->serialize($company));
29
    }
30
}
31