PhpOffice\PhpSpreadsheetTests\Functional\AbstractFunctional::writeAndReload()

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function writeAndReload(Spreadsheet $spreadsheet, $format)
    {
        $filename = tempnam(File::sysGetTempDir(), 'phpspreadsheet-test');
        $writer = IOFactory::createWriter($spreadsheet, $format);
        $writer->save($filename);

It seems like $filename can also be of type false; however, parameter $pFilename of PhpOffice\PhpSpreadsheet\Writer\IWriter::save() does only seem to accept string, maybe add an additional type check?

        $reader = IOFactory::createReader($format);
        $reloadedSpreadsheet = $reader->load($filename);

It seems like $filename can also be of type false; however, parameter $pFilename of PhpOffice\PhpSpreadsheet\Reader\IReader::load() does only seem to accept string, maybe add an additional type check?

        unlink($filename);

It seems like $filename can also be of type false; however, parameter $filename of unlink() does only seem to accept string, maybe add an additional type check?

        return $reloadedSpreadsheet;
    }