Application\Handler\AccountingDocumentHandler::handle()

Complexity

Conditions	4
Paths	4

Size

Total Lines	31
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

0

<?php
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        $id = $request->getAttribute('id');

        /** @var null|AccountingDocument $accountingDocument */
        $accountingDocument = $this->accountingDocumentRepository->findOneById($id);
        if (!$accountingDocument) {
            return $this->createError("AccountingDocument $id not found in database");
        }

        $path = $accountingDocument->getPath();
        if (!is_readable($path)) {
            return $this->createError("File for accounting document $id not found on disk, or not readable");
        }

        $resource = fopen($path, 'rb');
        if ($resource === false) {
            return $this->createError("Cannot open file for accounting document $id on disk");
        }

        $size = filesize($path);
        $type = mime_content_type($path);
        $ext = pathinfo($path, PATHINFO_EXTENSION);

        $response = new Response($resource, 200, [
            'content-type' => $type,
            'content-length' => $size,
            'content-disposition' => 'inline; filename="' . $id . '.' . $ext . '"',

Are you sure $ext of type array|string can be used in concatenation?

        ]);

        return $response;
    }