Swis\Filesystem\Encrypted\BinaryFileResponse

Complexity

Total Complexity

8

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Test Coverage

Coverage

90.48%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setFile()	0	12	3
A	sendContent()	0	27	5

<?php

namespace Swis\Filesystem\Encrypted;

use Symfony\Component\HttpFoundation\BinaryFileResponse as BaseBinaryFileResponse;

class BinaryFileResponse extends BaseBinaryFileResponse
{
    /**
     * @var \Swis\Filesystem\Encrypted\File
     */
    protected $file;

    /**
     * {@inheritdoc}
     */
    public function setFile($file, string $contentDisposition = null, bool $autoEtag = false, bool $autoLastModified = true)
    {
        // Wrap the file in our own \Swis\Filesystem\Encrypted\File class.
        if (!$file instanceof File) {
            if ($file instanceof \SplFileInfo) {
                $file = new File($file->getPathname());
            } else {
                $file = new File((string) $file);
            }
        }

        return parent::setFile($file, $contentDisposition, $autoEtag, $autoLastModified);
    }

    /**
     * {@inheritdoc}
     */
    public function sendContent()
    {
        if (!$this->isSuccessful()) {
            return parent::sendContent();
        }

        if (0 === $this->maxlen) {
            return $this;
        }

        $out = fopen('php://output', 'wb');

        // Create a temporary file stream with the decrypted contents.
        $file = tmpfile();
        fwrite($file, $this->file->getContents());

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

        fseek($file, 0);

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

        stream_copy_to_stream($file, $out, $this->maxlen, $this->offset);

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

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

        fclose($out);

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

        fclose($file);

        if ($this->deleteFileAfterSend && file_exists($this->file->getPathname())) {
            unlink($this->file->getPathname());
        }

        return $this;
    }
}