Odiseo\SyliusReportPlugin\Response\CsvResponse::setData()

Complexity

Conditions	3
Paths	4

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function setData(array $data)
    {
        $output = fopen('php://temp', 'r+');
        foreach ($data as $row) {
            fputcsv($output, $row);

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

        }
        rewind($output);

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

        $this->data = '';
        while ($line = fgets($output)) {

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

            $this->data .= $line;
        }
        $this->data .= fgets($output);
        return $this->update();
    }
}