EdifactParser\EdifactParser

Complexity

Total Complexity

2

Size/Duplication

Total Lines	12
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	parse()	0	10	2

<?php

declare(strict_types=1);

namespace EdifactParser;

use EDI\Parser;
use EdifactParser\Exception\InvalidFile;

final class EdifactParser
{
    public static function parse(Parser $parser): TransactionResult
    {
        $errors = $parser->errors();

        if ($errors) {

The expression $errors 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.

            throw InvalidFile::withErrors($errors);
        }

        return TransactionResult::fromSegmentedValues(
            SegmentedValues::fromRaw($parser->get())
        );
    }
}