Hhxsv5\LaravelS\Swoole\StaticResponse::sendContent()

Complexity

Conditions	8
Paths	23

Size

Total Lines	45
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function sendContent()
    {
        /**@var File $file */
        $file = $this->laravelResponse->getFile();
        $this->swooleResponse->header('Content-Type', $file->getMimeType());
        if ($this->laravelResponse->getStatusCode() == BinaryFileResponse::HTTP_NOT_MODIFIED) {
            $this->swooleResponse->end();
        } else {
            $path = $file->getRealPath();
            $size = filesize($path);
            if ($size > 0) {
                if (version_compare(swoole_version(), '1.7.21', '<')) {
                    throw new \RuntimeException('sendfile() require Swoole >= 1.7.21');
                }

                // Support deleteFileAfterSend: https://github.com/symfony/http-foundation/blob/5.0/BinaryFileResponse.php#L305
                $reflection = new \ReflectionObject($this->laravelResponse);
                try {
                    $deleteFileAfterSend = $reflection->getProperty('deleteFileAfterSend');
                    $deleteFileAfterSend->setAccessible(true);
                    $deleteFile = $deleteFileAfterSend->getValue($this->laravelResponse);
                } catch (\Exception $e) {
                    $deleteFile = false;
                }

                if ($deleteFile) {
                    $fp = fopen($file->getPathname(), 'rb');

                    for ($offset = 0, $limit = floor(1.99 * 1024 * 1024); $offset < $size; $offset += $limit) {
                        fseek($fp, $offset, SEEK_SET);

It seems like $offset can also be of type double; however, parameter $offset of fseek() does only seem to accept integer, maybe add an additional type check?

It seems like $fp can also be of type false; however, parameter $handle of fseek() does only seem to accept resource, maybe add an additional type check?

                        $chunk = fread($fp, $limit);

$limit of type double is incompatible with the type integer expected by parameter $length of fread().

It seems like $fp can also be of type false; however, parameter $handle of fread() does only seem to accept resource, maybe add an additional type check?

                        $this->swooleResponse->write($chunk);
                    }
                    $this->swooleResponse->end();

                    fclose($fp);

It seems like $fp can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

                    if (file_exists($file->getPathname())) {
                        unlink($file->getPathname());
                    }
                } else {
                    $this->swooleResponse->sendfile($path);
                }
            } else {
                $this->swooleResponse->end();
            }
}