CsvParser\Writer\StringWriter::write()

Complexity

Conditions	4
Paths	3

Size

Total Lines	22
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4.0119

Importance

Changes	1
Bugs	1	Features	0

<?php
    public static function write(\CsvParser\Parser $parser, \CsvParser\Csv $csv)
    {
        $data = $csv->getData();

        if ($data && !empty($data)) {

The expression $data of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $output = array(
                implode($parser->fieldDelimiter, array_map(function ($value) use ($parser) {
                    return $parser->fieldEnclosure . str_replace($parser->fieldEnclosure, $parser->fieldEnclosure.$parser->fieldEnclosure, $value) . $parser->fieldEnclosure;
                }, array_keys(current($data))))
            );

            foreach ($data as $line) {
                $output[] = implode($parser->fieldDelimiter, array_map(function ($value) use ($parser) {
                    return $parser->fieldEnclosure . str_replace($parser->fieldEnclosure, $parser->fieldEnclosure.$parser->fieldEnclosure, $value) . $parser->fieldEnclosure;
                }, $line));
            }

            return implode($parser->lineDelimiter, $output);
        } else {
            return '';
        }
    }
}