dallgoot / yaml

yaml/Builder.php

Spacing +13 added lines, -13 removed lines

@@ -43,7 +43,7 @@ 
      */
     private static function buildNodeList(NodeList $node, &$parent)
     {
-        if ($node->type & (Y::RAW | Y::LITTERALS)) {
+        if ($node->type & (Y::RAW|Y::LITTERALS)) {
             return self::buildLitteral($node, $node->type);
         }
         $p = $parent;
@@ -51,7 +51,7 @@ 
             case Y::MAPPING: // fall through
             // case Y::SET_KEY: // fall through
             case Y::SET:      $p = new \StdClass; var_dump('in set'); break;
-            case Y::SEQUENCE: $p = [];break;
+            case Y::SEQUENCE: $p = []; break;
             // case Y::KEY: $p = $parent;break;
         }
         $out = null;
@@ -79,7 +79,7 @@ 
     private static function buildNode(Node $node, &$parent)
     {
         extract((array) $node, EXTR_REFS);
-        if ($type & (Y::REF_DEF | Y::REF_CALL)) {
+        if ($type&(Y::REF_DEF|Y::REF_CALL)) {
             if (is_object($value)) {
                 $tmp = self::build($value, $parent) ?? $parent;
             } else {
@@ -200,15 +200,15 @@ 
     {
         self::$_root = new YamlObject();
         $childTypes = $list->getTypes();
-        $isaMapping  = (bool) (Y::KEY | Y::MAPPING) & $childTypes;
+        $isaMapping  = (bool) (Y::KEY|Y::MAPPING)&$childTypes;
         $isaSequence = (bool) Y::ITEM & $childTypes;
         $isaSet      = (bool) Y::SET_VALUE & $childTypes;
         if ($isaMapping && $isaSequence) {
             throw new \ParseError(sprintf(self::INVALID_DOCUMENT, $key));
         } else {
             switch (true) {
-                case $isaSequence: $list->type = Y::SEQUENCE;break;
-                case $isaSet:      $list->type = Y::SET;break;
+                case $isaSequence: $list->type = Y::SEQUENCE; break;
+                case $isaSet:      $list->type = Y::SET; break;
                 default:           $list->type = Y::MAPPING;
             }
         }
@@ -243,22 +243,22 @@ 
                 $lines[] = self::buildLitteral($child->value, $type);
             } else {
                 $prefix = '';
-                if ($type & Y::LITT_FOLDED && ($child->indent > $refIndent || ($child->type & Y::BLANK))) {
+                if ($type&Y::LITT_FOLDED && ($child->indent > $refIndent || ($child->type & Y::BLANK))) {
                     $prefix = "\n";
                 }
                 if (!($child->type & (Y::SCALAR|Y::BLANK))) {
                     switch ($child->type) {
-                        case Y::ITEM:    $child->value = '- '.$child->value;break;
-                        case Y::COMMENT: $child->value = '# '.$child->value;break;
+                        case Y::ITEM:    $child->value = '- '.$child->value; break;
+                        case Y::COMMENT: $child->value = '# '.$child->value; break;
                         default: die(Y::getName($child->type));
                     }
                 }
                 $lines[] = $prefix.$child->value;
             }
         }
-        if ($type & Y::RAW)         return implode('',   $lines);
-        if ($type & Y::LITT)        return implode("\n", $lines);
-        if ($type & Y::LITT_FOLDED) return implode(' ',  $lines);
+        if ($type&Y::RAW)         return implode('', $lines);
+        if ($type&Y::LITT)        return implode("\n", $lines);
+        if ($type&Y::LITT_FOLDED) return implode(' ', $lines);
         return '';
     }
 
@@ -273,7 +273,7 @@ 
     private function buildSetKey(Node $node, &$parent)
     {
         $built = self::build($node->value, $parent);
-        $stringKey = is_string($built) && Regex::isProperlyQuoted($built) ? trim($built, '\'" '): $built;
+        $stringKey = is_string($built) && Regex::isProperlyQuoted($built) ? trim($built, '\'" ') : $built;
         $key = json_encode($stringKey, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
         if (empty($key)) throw new \Exception("Cant serialize complex key: ".var_export($node->value, true), 1);
         $parent->{trim($key, '\'" ')} = null;

Braces +29 added lines, -10 removed lines

@@ -29,7 +29,9 @@ 
      */
     private static function build(object $node, &$parent = null)
     {
-        if ($node instanceof NodeList) return self::buildNodeList($node, $parent);
+        if ($node instanceof NodeList) {
+            return self::buildNodeList($node, $parent);
+        }
         return self::buildNode($node, $parent);
     }
 
@@ -85,7 +87,9 @@ 
             } else {
                 $tmp = $node->getPhpValue();
             }
-            if ($type === Y::REF_DEF) self::$_root->addReference($identifier, $tmp);
+            if ($type === Y::REF_DEF) {
+                self::$_root->addReference($identifier, $tmp);
+            }
             return self::$_root->getReference($identifier);
         }
         $typesActions = [Y::COMMENT   => 'buildComment',
@@ -176,10 +180,14 @@ 
         }
         $_root->value->setIteratorMode(NodeList::IT_MODE_DELETE);
         foreach ($_root->value as $child) {
-            if ($child->type & Y::DOC_START) $totalDocStart++;
+            if ($child->type & Y::DOC_START) {
+                $totalDocStart++;
+            }
             //if 0 or 1 DOC_START = we are still in first document
             $currentDoc = $totalDocStart > 1 ? $totalDocStart - 1 : 0;
-            if (!isset($documents[$currentDoc])) $documents[$currentDoc] = new NodeList();
+            if (!isset($documents[$currentDoc])) {
+                $documents[$currentDoc] = new NodeList();
+            }
             $documents[$currentDoc]->push($child);
         }
         $content = array_map([self::class, 'buildDocument'], $documents, array_keys($documents));
@@ -256,9 +264,15 @@ 
                 $lines[] = $prefix.$child->value;
             }
         }
-        if ($type & Y::RAW)         return implode('',   $lines);
-        if ($type & Y::LITT)        return implode("\n", $lines);
-        if ($type & Y::LITT_FOLDED) return implode(' ',  $lines);
+        if ($type & Y::RAW) {
+            return implode('',   $lines);
+        }
+        if ($type & Y::LITT) {
+            return implode("\n", $lines);
+        }
+        if ($type & Y::LITT_FOLDED) {
+            return implode(' ',  $lines);
+        }
         return '';
     }
 
@@ -275,7 +289,9 @@ 
         $built = self::build($node->value, $parent);
         $stringKey = is_string($built) && Regex::isProperlyQuoted($built) ? trim($built, '\'" '): $built;
         $key = json_encode($stringKey, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
-        if (empty($key)) throw new \Exception("Cant serialize complex key: ".var_export($node->value, true), 1);
+        if (empty($key)) {
+            throw new \Exception("Cant serialize complex key: ".var_export($node->value, true), 1);
+        }
         $parent->{trim($key, '\'" ')} = null;
     }
 
@@ -314,8 +330,11 @@ 
         }
         //TODO: have somewhere a list of common tags and their treatment
         if (in_array($node->identifier, ['!binary', '!str'])) {
-            if ($node->value->value instanceof NodeList) $node->value->value->type = Y::RAW;
-            else $node->value->type = Y::RAW;
+            if ($node->value->value instanceof NodeList) {
+                $node->value->value->type = Y::RAW;
+            } else {
+                $node->value->type = Y::RAW;
+            }
         }
         $val = is_null($node->value) ? null : self::build(/** @scrutinizer ignore-type */ $node->value, $node);
         return new Tag($node->identifier, $val);

yaml/Loader.php

Spacing +7 added lines, -7 removed lines

@@ -18,17 +18,17 @@ 
     /* @var null|string */
     public $error;
 
-    public const EXCLUDE_DIRECTIVES = 1;//DONT include_directive
-    public const IGNORE_COMMENTS    = 2;//DONT include_comments
-    public const NO_PARSING_EXCEPTIONS = 4;//THROW Exception on parsing Errors
-    public const NO_OBJECT_FOR_DATE = 8;//DONT import date strings as dateTime Object
+    public const EXCLUDE_DIRECTIVES = 1; //DONT include_directive
+    public const IGNORE_COMMENTS    = 2; //DONT include_comments
+    public const NO_PARSING_EXCEPTIONS = 4; //THROW Exception on parsing Errors
+    public const NO_OBJECT_FOR_DATE = 8; //DONT import date strings as dateTime Object
     //privates
     /* @var null|string */
     private $content;
     /* @var null|string */
     private $filePath;
     /* @var integer */
-    private $debug = 0;///TODO: determine levels
+    private $debug = 0; ///TODO: determine levels
     /* @var integer */
     private $options = 0;
     //Exceptions messages
@@ -90,7 +90,7 @@ 
         $previous = $root = new Node();
         $emptyLines = [];
         try {
-            $gen = function () use($source) {
+            $gen = function() use($source) {
                 foreach ($source as $key => $value) {
                     yield ++$key => $value;
                 }
@@ -109,7 +109,7 @@ 
                         break;
                     case 0:  $target = $previous->getParent();
                         break;
-                    default: $target = $previous->type & Y::ITEM && $n->type != Y::SET_VALUE? $previous->getDeepestNode()->getParent($n->indent) : $previous;
+                    default: $target = $previous->type & Y::ITEM && $n->type != Y::SET_VALUE ? $previous->getDeepestNode()->getParent($n->indent) : $previous;
                 }
                 if ($this->onContextType($n, $target, $lineString)) continue;
                 $target->add($n);

Braces +15 added lines, -5 removed lines

@@ -86,7 +86,9 @@ 
     {
         $source = $this->content ?? preg_split("/\n/m", preg_replace('/(\r\n|\r)/', "\n", $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);
+        if (!is_array($source) || !count($source)) {
+            throw new \Exception(self::EXCEPTION_LINE_SPLIT);
+        }
         $previous = $root = new Node();
         $emptyLines = [];
         try {
@@ -98,7 +100,9 @@ 
             foreach ($gen() as $lineNb => $lineString) {
                 $n = new Node($lineString, $lineNb);
                 if ($n->type & (Y::LITTERALS|Y::BLANK|Y::COMMENT)) {
-                    if ($this->onSpecialType($n, $previous, $emptyLines)) continue;
+                    if ($this->onSpecialType($n, $previous, $emptyLines)) {
+                        continue;
+                    }
                 }
                 foreach ($emptyLines as $blankNode) {
                     $blankNode->getParent()->add($blankNode);
@@ -111,7 +115,9 @@ 
                         break;
                     default: $target = $previous->type & Y::ITEM && $n->type != Y::SET_VALUE? $previous->getDeepestNode()->getParent($n->indent) : $previous;
                 }
-                if ($this->onContextType($n, $target, $lineString)) continue;
+                if ($this->onContextType($n, $target, $lineString)) {
+                    continue;
+                }
                 $target->add($n);
                 $previous = $n;
             }
@@ -140,8 +146,12 @@ 
     {
         $deepest = $previous->getDeepestNode();
         if ($n->type & Y::BLANK) {
-            if ($previous->type & Y::SCALAR) $emptyLines[] = $n->setParent($previous->getParent());
-            if ($deepest->type & Y::LITTERALS) $emptyLines[] = $n->setParent($deepest);
+            if ($previous->type & Y::SCALAR) {
+                $emptyLines[] = $n->setParent($previous->getParent());
+            }
+            if ($deepest->type & Y::LITTERALS) {
+                $emptyLines[] = $n->setParent($deepest);
+            }
             return true;
         }
         //comment is fullline : forces 'root' as parent if not inside a LITTERAL

yaml/Node.php

Spacing +6 added lines, -6 removed lines

@@ -86,7 +86,7 @@ 
      */
     public function add(Node $child)
     {
-        if($this->type & (Y::SCALAR|Y::QUOTED)) {
+        if ($this->type & (Y::SCALAR|Y::QUOTED)) {
             $this->getParent()->add($child);
             return;
         }
@@ -171,8 +171,8 @@ 
         if (in_array($first, ['{', '[']))      return $this->onObject($nodeValue);
         if (in_array($first, ['!', '&', '*'])) return $this->onNodeAction($nodeValue);
         // Note : php don't like '?' as an array key -_-
-        if($first === '?') return [Y::SET_KEY, empty(trim($v)) ? null : new Node(ltrim($v), $this->line)];
-        $characters = [ '#' =>  [Y::COMMENT, ltrim($v)],
+        if ($first === '?') return [Y::SET_KEY, empty(trim($v)) ? null : new Node(ltrim($v), $this->line)];
+        $characters = ['#' =>  [Y::COMMENT, ltrim($v)],
                         "-" =>  $this->onHyphen($nodeValue),
                         '%' =>  [Y::DIRECTIVE, ltrim($v)],
                         ':' =>  [Y::SET_VALUE, empty(trim($v)) ? null : new Node(ltrim($v), $this->line)],
@@ -291,8 +291,8 @@ 
     {
         $v = &$this->value;
         if (is_null($v)) return null;
-        if ($this->type & (Y::REF_CALL | Y::SCALAR)) return self::getScalar($v);
-        if ($this->type & (Y::COMPACT_MAPPING | Y::COMPACT_SEQUENCE)) return self::getCompact(substr($v, 1, -1), $this->type);
+        if ($this->type & (Y::REF_CALL|Y::SCALAR)) return self::getScalar($v);
+        if ($this->type & (Y::COMPACT_MAPPING|Y::COMPACT_SEQUENCE)) return self::getCompact(substr($v, 1, -1), $this->type);
         $expected = [Y::JSON   => json_decode($v, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR),
                      Y::QUOTED => substr($v, 1, -1),
                      Y::RAW    => strval($v)];
@@ -354,7 +354,7 @@ 
         //TODO : this should handle references present inside the string
         $out = new Compact();
         if ($type === Y::COMPACT_SEQUENCE) {
-            $f = function ($e) { return self::getScalar(trim($e));};
+            $f = function($e) { return self::getScalar(trim($e)); };
             //TODO : that's not robust enough, improve it
             foreach (array_map($f, explode(",", $mappingOrSeqString)) as $key => $value) {
                 $out[$key] = $value;

Braces +60 added lines, -20 removed lines

@@ -68,7 +68,9 @@ 
         if ($this->type === Y::ROOT) {
             return $this;
         }
-        if (!is_int($indent)) return $this->parent ?? $this;
+        if (!is_int($indent)) {
+            return $this->parent ?? $this;
+        }
         $cursor = $this;
         while ($cursor instanceof Node && $cursor->indent >= $indent) {
             $cursor = $cursor->parent;
@@ -103,13 +105,21 @@ 
                     $this->value->push($current);
                 }
                 //modify type according to child
-                if ($current->type & Y::SET_KEY)   $this->value->type = Y::SET;
-                if ($current->type & Y::KEY)       $this->value->type = Y::MAPPING;
-                if ($current->type & Y::ITEM)      $this->value->type = Y::SEQUENCE;
+                if ($current->type & Y::SET_KEY) {
+                    $this->value->type = Y::SET;
+                }
+                if ($current->type & Y::KEY) {
+                    $this->value->type = Y::MAPPING;
+                }
+                if ($current->type & Y::ITEM) {
+                    $this->value->type = Y::SEQUENCE;
+                }
             }
             $this->value->push($child);
 
-            if ($this->type & Y::LITTERALS)  $this->value->type = $this->type;
+            if ($this->type & Y::LITTERALS) {
+                $this->value->type = $this->type;
+            }
         }
     }
 
@@ -168,10 +178,16 @@ 
             $type = R::isProperlyQuoted($nodeValue) ? Y::QUOTED : Y::PARTIAL;
             return [$type, $nodeValue];
         }
-        if (in_array($first, ['{', '[']))      return $this->onObject($nodeValue);
-        if (in_array($first, ['!', '&', '*'])) return $this->onNodeAction($nodeValue);
+        if (in_array($first, ['{', '['])) {
+            return $this->onObject($nodeValue);
+        }
+        if (in_array($first, ['!', '&', '*'])) {
+            return $this->onNodeAction($nodeValue);
+        }
         // Note : php don't like '?' as an array key -_-
-        if($first === '?') return [Y::SET_KEY, empty(trim($v)) ? null : new Node(ltrim($v), $this->line)];
+        if($first === '?') {
+            return [Y::SET_KEY, empty(trim($v)) ? null : new Node(ltrim($v), $this->line)];
+        }
         $characters = [ '#' =>  [Y::COMMENT, ltrim($v)],
                         "-" =>  $this->onHyphen($nodeValue),
                         '%' =>  [Y::DIRECTIVE, ltrim($v)],
@@ -224,9 +240,15 @@ 
     private function onObject($value):array
     {
         json_decode($value, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
-        if (json_last_error() === JSON_ERROR_NONE)  return [Y::JSON, $value];
-        if (preg_match(R::MAPPING, $value))         return [Y::COMPACT_MAPPING, $value];
-        if (preg_match(R::SEQUENCE, $value))        return [Y::COMPACT_SEQUENCE, $value];
+        if (json_last_error() === JSON_ERROR_NONE) {
+            return [Y::JSON, $value];
+        }
+        if (preg_match(R::MAPPING, $value)) {
+            return [Y::COMPACT_MAPPING, $value];
+        }
+        if (preg_match(R::SEQUENCE, $value)) {
+            return [Y::COMPACT_SEQUENCE, $value];
+        }
         return [Y::PARTIAL, $value];
     }
 
@@ -242,7 +264,9 @@ 
     {
         if (substr($nodeValue, 0, 3) === '---') {
             $rest = trim(substr($nodeValue, 3));
-            if (empty($rest)) return [Y::DOC_START, null];
+            if (empty($rest)) {
+                return [Y::DOC_START, null];
+            }
             $n = new Node($rest, $this->line);
             $n->indent = $this->indent + 4;
             return [Y::DOC_START, $n->setParent($this)];
@@ -290,9 +314,15 @@ 
     public function getPhpValue()
     {
         $v = &$this->value;
-        if (is_null($v)) return null;
-        if ($this->type & (Y::REF_CALL | Y::SCALAR)) return self::getScalar($v);
-        if ($this->type & (Y::COMPACT_MAPPING | Y::COMPACT_SEQUENCE)) return self::getCompact(substr($v, 1, -1), $this->type);
+        if (is_null($v)) {
+            return null;
+        }
+        if ($this->type & (Y::REF_CALL | Y::SCALAR)) {
+            return self::getScalar($v);
+        }
+        if ($this->type & (Y::COMPACT_MAPPING | Y::COMPACT_SEQUENCE)) {
+            return self::getCompact(substr($v, 1, -1), $this->type);
+        }
         $expected = [Y::JSON   => json_decode($v, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR),
                      Y::QUOTED => substr($v, 1, -1),
                      Y::RAW    => strval($v)];
@@ -321,9 +351,15 @@ 
                     '-.inf' => -INF,
                     '.nan'  => NAN
         ];
-        if (isset($types[strtolower($v)])) return $types[strtolower($v)];
-        if (R::isDate($v))   return date_create($v);
-        if (R::isNumber($v)) return self::getNumber($v);
+        if (isset($types[strtolower($v)])) {
+            return $types[strtolower($v)];
+        }
+        if (R::isDate($v)) {
+            return date_create($v);
+        }
+        if (R::isNumber($v)) {
+            return self::getNumber($v);
+        }
         return strval($v);
     }
 
@@ -336,8 +372,12 @@ 
      */
     private static function getNumber(string $v)
     {
-        if (preg_match("/^(0o\d+)$/i", $v))      return intval(base_convert($v, 8, 10));
-        if (preg_match("/^(0x[\da-f]+)$/i", $v)) return intval(base_convert($v, 16, 10));
+        if (preg_match("/^(0o\d+)$/i", $v)) {
+            return intval(base_convert($v, 8, 10));
+        }
+        if (preg_match("/^(0x[\da-f]+)$/i", $v)) {
+            return intval(base_convert($v, 16, 10));
+        }
         return is_bool(strpos($v, '.')) ? intval($v) : floatval($v);
     }
 

example.php

Spacing +2 added lines, -2 removed lines

@@ -6,14 +6,14 @@
 /**
  * Display some use cases for Yaml library
  */
-const JSON_OPTIONS = JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_LINE_TERMINATORS | JSON_UNESCAPED_UNICODE | JSON_PRESERVE_ZERO_FRACTION | JSON_PARTIAL_OUTPUT_ON_ERROR;
+const JSON_OPTIONS = JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_LINE_TERMINATORS|JSON_UNESCAPED_UNICODE|JSON_PRESERVE_ZERO_FRACTION|JSON_PARTIAL_OUTPUT_ON_ERROR;
 
 /* USE CASE 1
 * load and parse if file exists
 */
 ini_set("auto_detect_line_endings", 1);
 // $yaml = Y::parseFile('./tests/cases/parsing/blockChompingWithInsideBlank.yml', null, 0); //->parse();
-$content = file_get_contents('./tests/cases/parsing/complex_mapping_in_item.yml');//var_dump($content);
+$content = file_get_contents('./tests/cases/parsing/complex_mapping_in_item.yml'); //var_dump($content);
 $yaml = Y::parse($content, null, 2); //->parse();
 // $yaml = Y::parseFile('./references/Example 2.27.yml', null, 1);
 var_dump($yaml);