StefanoTree\NestedSet\Utilities::_flatToNested()

Complexity

Conditions	8
Paths	17

Size

Total Lines	35
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	8

Importance

Changes	1
Bugs	0	Features	0

<?php
    private static function _flatToNested(
        array $flatTree,
        string $levelName = 'level',
        ?int $level = null,
        int $startPos = 0,
        array &$result = array()
    ): array {
        $total = count($flatTree);
        $first = true;

        for ($pos = $startPos; $pos < $total; ++$pos) {
            $item = $flatTree[$pos];

            if (null === $level) {
                $level = $item[$levelName];
            }

            if (!array_key_exists('_children', $item)) {
                $item['_children'] = array();
            }

            if ($level == $item[$levelName]) {
                $result[] = $item;
                $first = true;
            } elseif (($level + 1) == $item[$levelName] && true == $first) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

                $children = array($item);
                self::_flatToNested($flatTree, $levelName, (int) $item[$levelName], $pos + 1, $children);
                $result[count($result) - 1]['_children'] = $children;
                $first = false;
            } elseif ($level > $item[$levelName]) {
                break;
            }
        }

        return $result;
    }