Jackal\Copycat\Reader\CSVFileReader::__construct()

Complexity

Conditions	5
Paths	8

Size

Total Lines	27
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	2

<?php
    public function __construct(SplFileObject $fileObject, array $options = [])
    {
        $resolver = new OptionsResolver();
        $resolver->setDefaults([
            'delimiter' => ',',
            'enclosure' => '"',
            'header' => true,
        ]);

        $this->options = $resolver->resolve($options);

        $this->fileObject = $fileObject;

        $headers = [];
        if ($this->options['header']) {
            $headers = $this->readCurrentRow();
        }

        while (($data = $this->readCurrentRow()) != false) {
            //skip empty rows
            if($data == [null]){
                continue;
            }
            if ($headers) {
                $this->addItem(array_combine($headers, $data));

It seems like array_combine($headers, $data) can also be of type false; however, parameter $item of Jackal\Copycat\Reader\BaseReader::addItem() does only seem to accept array, maybe add an additional type check?

            } else {
                $this->addItem($data);
            }