Swis\Laravel\Encrypted\BinaryFileResponse::sendContent()

Complexity

Conditions	5
Paths	4

Size

Total Lines	27
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	5.0592

Importance

Changes

0

<?php
    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;
    }