Application\Handler\ImageHandler::handle()

Complexity

Conditions	7
Paths	7

Size

Total Lines	49
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes	1
Bugs	0	Features	0

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

        $card = $this->cardRepository->findOneById($id);

The method findOneById() does not exist on Application\Repository\CardRepository. Since you implemented __call, consider adding a @method annotation.

        if (!$card) {
            return $this->createError("Card $id not found in database");
        }

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

        $maxHeight = (int) $request->getAttribute('maxHeight');
        if ($maxHeight) {
            $accept = $request->getHeaderLine('accept');
            $useWebp = str_contains($accept, 'image/webp');

            $resizeNeeded = $this->imageResizer->isResizeNeeded($card, $maxHeight, $useWebp);
            $resizeSpecified = array_key_exists('resize', $request->getQueryParams());

            if ($resizeNeeded && !$resizeSpecified) {
                // If resize is needed, user must specify a specific query params.
                // This allow to configure server load balance, but also work
                // without any special configuration if not needed.
                // Resizing operation is ressource intensive and could lead to block
                // the server if not properly configured.
                return new RedirectResponse($this->constructRedirectURI(), 302);
            }

            $path = $this->imageResizer->resize($card, $maxHeight, $useWebp);
        }

        $queryParams = $request->getQueryParams();

        $resource = fopen($path, 'rb');
        $type = mime_content_type($path);
        $extension = pathinfo($path, PATHINFO_EXTENSION);
        $filename = $id . '.' . $extension;

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

        $disposition = isset($queryParams['inline']) ? 'inline' : 'attachment';

        $response = new Response($resource, 200, [
            'content-type' => $type,
            'content-disposition' => $disposition . '; filename=' . $filename,
            'cache-control' => 'max-age=' . (24 * 60 * 60), // 24 hours cache
        ]);

        return $response;
    }