AbterPhp\Admin\Http\Controllers\Api\Editor

Complexity

Total Complexity

5

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	sendJson()	0	16	2
A	fileUpload()	0	25	3

<?php

declare(strict_types=1);

namespace AbterPhp\Admin\Http\Controllers\Api;

use AbterPhp\Admin\Config\Routes;
use AbterPhp\Admin\Constant\Env;
use AbterPhp\Framework\Constant\Env as FrameworkEnv;
use Opulence\Environments\Environment;
use Opulence\Http\Responses\JsonResponse;
use Opulence\Http\Responses\Response;
use Opulence\Http\Responses\ResponseHeaders;
use Opulence\Routing\Controller;

class Editor extends Controller
{
    /**
     * @return Response
     */
    public function fileUpload(): Response
    {
        $actualClientId = explode(' ', $this->request->getHeaders()->get('authorization'))[1];

        $expectedClientId = Environment::getVar(FrameworkEnv::CRYPTO_CLIENT_ID);
        if ($actualClientId != $expectedClientId) {
            return $this->sendJson(ResponseHeaders::HTTP_FORBIDDEN);
        }

        $image = $this->request->getFiles()->get('image');
        if ($image->hasErrors()) {
            return $this->sendJson(ResponseHeaders::HTTP_INTERNAL_SERVER_ERROR);
        }

        $pathinfo  = pathinfo($image->getTempFilename());
        $basePath  = mb_substr(basename($image->getPathname()), 3);
        $extension = $pathinfo['extension'] ?? '';
        $filename  = sprintf('%s.%s', $basePath, $extension);

        $path = Environment::getVar(FrameworkEnv::DIR_UPLOAD);

The constant AbterPhp\Framework\Constant\Env::DIR_UPLOAD was not found. Maybe you did not declare it correctly or list all dependencies?

        $url  = Environment::getVar(Env::UPLOAD_BASE_URL) . '/' . $filename;

        $image->move($path, $filename);

It seems like $path can also be of type null; however, parameter $targetDirectory of Opulence\Http\Requests\UploadedFile::move() does only seem to accept string, maybe add an additional type check?

        return $this->sendJson(ResponseHeaders::HTTP_OK, ['url' => $url]);
    }

    /**
     * @param int        $status
     * @param array|null $data
     *
     * @return Response
     */
    protected function sendJson($status = ResponseHeaders::HTTP_OK, array $data = null): Response
    {
        $body = [
            'success' => $status == ResponseHeaders::HTTP_OK,
            'status'  => $status,
        ];

        if ($data) {
            $body['data'] = $data;
        }

        $response = new JsonResponse($body, $status);

        $response->send();

        return $response;
    }
}