Dallgoot\Yaml\Loader::parse()

Complexity

Conditions	15
Paths	65

Size

Total Lines	47
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function parse($strContent = null)
    {
        $source = $this->content ?? preg_split("/([^\n\r]+)/um", $strContent, 0, PREG_SPLIT_DELIM_CAPTURE);
        //TODO : be more permissive on $strContent values
        if (!is_array($source) || !count($source)) throw new \Exception(self::EXCEPTION_LINE_SPLIT);
        $previous = $root = new Node();
        $emptyLines = [];
        try {
            $gen = function() use($source) {

Expected 1 space after FUNCTION keyword; 0 found

Expected 1 space after USE keyword; found 0

                foreach ($source as $key => $value) {
                    yield ++$key => $value;
                }
            };
            foreach ($gen() as $lineNb => $lineString) {
                $n = new Node($lineString, $lineNb);
                if ($n->type & (Y::LITTERALS|Y::BLANK)) {
                    if ($this->onSpecialType($n, $previous, $emptyLines)) continue;
                } else {
                    foreach ($emptyLines as $blankNode) {
                        $blankNode->getParent()->add($blankNode);
                    }
                }
                $emptyLines = [];
                switch ($n->indent <=> $previous->indent) {
                    case -1: $target = $previous->getParent($n->indent);

Line indented incorrectly; expected 16 spaces, found 20

                        break;
                    case 0:  $target = $previous->getParent();

Line indented incorrectly; expected 16 spaces, found 20

                        break;
                    default: $target = $previous;

Line indented incorrectly; expected 16 spaces, found 20

                }
                if ($this->onContextType($n, $target, $lineString)) continue;
                $target->add($n);
                $previous = $n;
            }
            if ($this->debug === 2) echo "\033[33mParsed Structure\033[0m\n",var_export($root, true);
            $out = Builder::buildContent($root, $this->debug);
            return $out;
        } catch (\Error|\Exception|\ParseError $e) {
            $file = basename($this->filePath);
            $message = basename($e->getFile())."@".$e->getLine().":".$e->getMessage()." in '$file' @".($lineNb)."\n";

The variable $lineNb seems to be defined by a foreach iteration on line 99. Are you sure the iterator is never empty, otherwise this variable is not defined?

            if ($e instanceof \ParseError && ($this->options & self::NO_PARSING_EXCEPTIONS)) {
                trigger_error($message, E_USER_WARNING);
                $this->error = $message;
                return null;
            }
            var_dump($root);

var_dump($root) looks like debug code. Are you sure you do not want to remove it?

            throw new \Exception($message, 3);
        }