Recca0120\Upload\FileAPI

Complexity

Total Complexity

7

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

93.75%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getOriginalName()	0	12	2
A	getMimeType()	0	9	2
B	doReceive()	0	24	3

<?php

namespace Recca0120\Upload;

class FileAPI extends Api
{
    /**
     * getOriginalName.
     *
     * @return string
     */
    protected function getOriginalName($contentDisposition)
    {
        $originalName = $this->request->get('name');
        if (empty($originalName) === true) {
            list($originalName) = sscanf(
                $contentDisposition,
                'attachment; filename=%s'
            );
        }

        return preg_replace('/[\'"]/', '', $originalName);
    }

    /**
     * getMimeType.
     *
     * @param string $originalName
     * @return string
     */
    protected function getMimeType($originalName)
    {
        $mimeType = $this->request->header('content-type');
        if (empty($mimeType) === true) {
            $mimeType = $this->filesystem->mimeType($originalName);
        }

        return $mimeType;
    }

    /**
     * receive.
     *
     * @param string $inputName
     * @return \Symfony\Component\HttpFoundation\File\UploadedFile
     *
     * @throws \Recca0120\Upload\Exceptions\ChunkedResponseException
     */
    protected function doReceive($inputName)
    {
        $contentDisposition = $this->request->header('content-disposition');
        if (empty($contentDisposition) === true) {
            return $this->request->file($inputName);
        }

        $contentRange = $this->request->header('content-range');
        if (empty($contentRange) === false) {
            list($start, $end, $total) = sscanf($contentRange, 'bytes %d-%d/%d');
        } else {
            $start = 0;
            $end = $this->request->header('content-length');
            $total = $end;
        }

        return $this->receiveChunkedFile(
            $originalName = $this->getOriginalName($contentDisposition),

It seems like $originalName = $this->getOriginalName($contentDisposition) can also be of type array<integer,string>; however, Recca0120\Upload\Api::receiveChunkedFile() does only seem to accept string, maybe add an additional type check?

            'php://input',
            $start,
            $end >= $total - 1,
            ['mimeType' => $this->getMimeType($originalName), 'headers' => ['X-Last-Known-Byte' => $end]]

It seems like $originalName defined by $this->getOriginalName($contentDisposition) on line 66 can also be of type array<integer,string>; however, Recca0120\Upload\FileAPI::getMimeType() does only seem to accept string, maybe add an additional type check?

        );
    }
}