Dallgoot\Yaml\NodeCompactSequence

Complexity

Total Complexity

5

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	2
A	build()	0	10	3

<?php

namespace Dallgoot\Yaml;

/**
 *
 * @author  Stéphane Rebai <stephane.rebai@gmail.com>
 * @license Apache 2.0
 * @link    TODO : url to specific online doc
 */
class NodeCompactSequence extends Node
{
    public function __construct(string $nodeString, int $line)
    {
        parent::__construct($nodeString, $line);
        preg_match_all(Regex::SEQUENCE_VALUES, trim(substr(trim($nodeString), 1,-1)), $matches);
        foreach ($matches['item'] as $key => $item) {
            $i = new NodeItem('', $line);
            $i->indent = null;
            $itemValue = NodeFactory::get(trim($item));
            $itemValue->indent = null;
            $i->add($itemValue);
            $this->add($i);
        }
    }

    public function build(&$parent = null)
    {
        if (is_null($this->value)) {
            return null;
        }
        if ($this->value instanceof Node) {
            $this->value = new NodeList($this->value);
            $this->value->type = NodeList::SEQUENCE;
        }
        return new Compact($this->value->build());

It seems like $this->value->build() can also be of type boolean and double and integer and string; however, parameter $candidate of Dallgoot\Yaml\Compact::__construct() does only seem to accept array|object, maybe add an additional type check?

    }
}