| 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
breakstatement is not necessary if it is preceded for example by areturnstatement:If you would like to keep this construct to be consistent with other
casestatements, you can safely mark this issue as a false-positive.