Total Complexity | 4 |
Total Lines | 34 |
Duplicated Lines | 0 % |
Changes | 0 |
1 | <?php |
||
13 | class FindSimilarAppsUrlScraper implements ResponseHandlerInterface |
||
14 | { |
||
15 | /** |
||
16 | * @var RequestApp |
||
17 | */ |
||
18 | private $requestApp; |
||
19 | |||
20 | /** |
||
21 | * SimilarScraper constructor. |
||
22 | * |
||
23 | * @param RequestApp $requestApp |
||
24 | */ |
||
25 | public function __construct(RequestApp $requestApp) |
||
28 | } |
||
29 | |||
30 | /** |
||
31 | * @param RequestInterface $request |
||
32 | * @param ResponseInterface $response |
||
33 | * @return string|null |
||
34 | */ |
||
35 | public function __invoke(RequestInterface $request, ResponseInterface $response): ?string |
||
49 |
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.