dlindberg\BlobChunk\Parser\RowCol::parse()

Complexity

Conditions	7
Paths	48

Size

Total Lines	20
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function parse(\DOMElement $node): array
    {
        $this->manager->setRowColParentNode($node);
        $thead = $this->manager->rowColHasHeaders() ?
            $this->stringifyList($this->collectRowColDefs($node->firstChild)) : [];
        $tbody = $this->manager->rowColHasHeaders() ?
            $this->collectRowColRowGroups($node->firstChild) : [];
        if (\method_exists($tbody, 'count')) {
            $tbody = 1 === $tbody->count() ?
                $this->stringifyList($this->collectRows($tbody->item(0)->firstChild)) : $this->stringifyList($tbody);
        }
        $tbody = 0 === \count($tbody) ? $this->stringifyList($this->collectRows($node->firstChild)) : $tbody;
        $thead = 0 === \count($thead) ? \array_shift($tbody) : $thead;

It seems like $tbody can also be of type DOMNodeList; however, parameter $array of array_shift() does only seem to accept array, maybe add an additional type check?

        return \array_map(function ($row) use ($thead) {
            return [
                'thead' => $thead,
                'tr'    => $row,
            ];
        }, $tbody);
    }