dlindberg\BlobChunk\Parser\PairNode::collectBNodes()

Complexity

Conditions	4
Paths	6

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    private function collectBNodes(?\DOMNode $node, array $nodes = []): array
    {
        if ($this->manager->isPairSideBNode($node)) {

It seems like $node can also be of type null; however, parameter $input of dlindberg\BlobChunk\Manager\CheckNode::isPairSideBNode() does only seem to accept DOMNode, maybe add an additional type check?

            $nodes[] = $this->stringify($node);

It seems like $node can also be of type null; however, parameter $node of dlindberg\BlobChunk\Parser\BaseParser::stringify() does only seem to accept DOMNode, maybe add an additional type check?

        }
        if ($this->manager->isPairSideANode($node)) {

It seems like $node can also be of type null; however, parameter $input of dlindberg\BlobChunk\Manager\CheckNode::isPairSideANode() does only seem to accept DOMNode, maybe add an additional type check?

            $node = $node->previousSibling;
        } elseif (null !== $node->nextSibling) {
            return $this->collectBNodes($node->nextSibling, $nodes);
        }

        return [
            'currentNode' => $node,
            'bNodes'      => $nodes,
        ];