Minepic\Http\Controllers\Api\DownloadTextureController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	serve()	0	15	2
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace Minepic\Http\Controllers\Api;

use Illuminate\Http\Response;
use Laravel\Lumen\Http\ResponseFactory;
use Laravel\Lumen\Routing\Controller as BaseController;
use Minepic\Helpers\Storage\Files\SkinsStorage;
use Minepic\Image\Sections\Raw;
use Minepic\Minecraft\MinecraftDefaults;
use Minepic\Resolvers\UuidResolver;

class DownloadTextureController extends BaseController
{
    public function __construct(
        protected UuidResolver $uuidResolver,
        protected ResponseFactory $responseFactory
    ) {
    }

    /**
     * Download user texture.
     *
     * @param string $uuid User UUID or Username
     *
     * @throws \Minepic\Image\Exceptions\ImageCreateFromPngFailedException
     */
    public function serve(string $uuid = ''): Response
    {
        $headers = [
            'Content-Disposition' => 'Attachment;filename='.$uuid.'.png',
            'Content-Type' => 'image/png',
        ];

        $skinPath = $this->uuidResolver->resolve($uuid) ?
            SkinsStorage::getPath($this->uuidResolver->getUuid()) :

It seems like $this->uuidResolver->getUuid() can also be of type null; however, parameter $uuid of Minepic\Helpers\Storage\Storage::getPath() does only seem to accept string, maybe add an additional type check?

            SkinsStorage::getPath(MinecraftDefaults::STEVE_DEFAULT_SKIN_NAME);

        $rawSkin = new Raw($skinPath);
        $rawSkin->render();

        return $this->responseFactory->make((string) $rawSkin, 200, $headers);
    }
}