Total Complexity | 4 |
Total Lines | 35 |
Duplicated Lines | 0 % |
Coverage | 90.91% |
Changes | 1 | ||
Bugs | 0 | Features | 0 |
1 | <?php |
||
24 | class FindSimilarAppsUrlScraper implements ResponseHandlerInterface |
||
25 | { |
||
26 | /** @var AppId */ |
||
27 | private $appId; |
||
28 | |||
29 | /** |
||
30 | * SimilarScraper constructor. |
||
31 | * |
||
32 | * @param AppId $appId |
||
33 | */ |
||
34 | 1 | public function __construct(AppId $appId) |
|
37 | 1 | } |
|
38 | |||
39 | /** |
||
40 | * @param RequestInterface $request |
||
41 | * @param ResponseInterface $response |
||
42 | * |
||
43 | * @return string|null |
||
44 | */ |
||
45 | 1 | public function __invoke(RequestInterface $request, ResponseInterface $response): ?string |
|
61 |
The
break
statement is not necessary if it is preceded for example by areturn
statement:If you would like to keep this construct to be consistent with other
case
statements, you can safely mark this issue as a false-positive.