WebServCo\Framework\CsvCreator::getCsvData()

Complexity

Conditions	4
Paths	5

Size

Total Lines	26
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function getCsvData(array $data, bool $addHeader = true)
    {
        $csvData = null;

The assignment to $csvData is dead and can be removed.

        if (empty($data)) {
            return false;
        }

        // temporary memory wrapper; if bigger than 5MB will be written to temp file.
        $handle = fopen('php://temp/maxmemory: ' . (5*1024*1024), 'r+');

        if ($addHeader) {
            fputcsv($handle, array_keys(current($data)), $this->delimiter, $this->enclosure);

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

        }

        foreach ($data as $item) {
            fputcsv($handle, $item, $this->delimiter, $this->enclosure);
        }

        rewind($handle);

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

        $csvData = stream_get_contents($handle);

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

        fclose($handle);

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

        return $csvData;
    }