Doctrs\SonataImportBundle\Loaders\CsvFileLoader

Complexity

Total Complexity

4

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setFile()	0	3	1
A	getIteration()	0	8	3

<?php

namespace Doctrs\SonataImportBundle\Loaders;


use Symfony\Component\HttpFoundation\File\File;
use Symfony\Component\Process\Exception\InvalidArgumentException;

The type Symfony\Component\Process\Exception\InvalidArgumentException was not found. Maybe you did not declare it correctly or list all dependencies?

class CsvFileLoader implements FileLoaderInterface {

    /** @var File $file  */
    protected $file = null;

    public function setFile(File $file) : FileLoaderInterface {
        $this->file = $file;
        return $this;
    }

    public function getIteration() {
        if (!$this->file) {
            throw new InvalidArgumentException('File not found');
        }

        $file = fopen($this->file->getRealPath(), 'r');
        while (($line = fgetcsv($file, 0, ',')) !== false) {

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

            yield $line;
        }
    }

}