dallgoot / yaml

example.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-require_once __DIR__ . '/vendor/autoload.php';
+require_once __DIR__.'/vendor/autoload.php';
 
 use \Dallgoot\Yaml as Y;
 
@@ -7,7 +7,7 @@ 
 * load and parse if file exists
 */
 // $yaml = Y::loadFile('./dummy.yml');//->parse();
-$yaml = Y::parseFile('./references/Example 2.18.yml');//->parse();
+$yaml = Y::parseFile('./references/Example 2.18.yml'); //->parse();
 var_dump($yaml);
 exit(0);
 // USE CASE 2

yaml/Yaml.php

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
             'SEQUENCE_SHORT',
             'PARTIAL',
             'LITT', //litteral
-            'LITT_FOLDED',//litteral
+            'LITT_FOLDED', //litteral
             'SCALAR',
             'TAG',
             'JSON',
@@ -34,7 +34,7 @@ 
     define(__NAMESPACE__."\\$name", 2**$power);
 }
 
-const LITTERALS = Y\LITT | Y\LITT_FOLDED;
+const LITTERALS = Y\LITT|Y\LITT_FOLDED;
 // print_r(get_defined_constants(true)['user']);
 
 namespace Dallgoot;
@@ -52,8 +52,8 @@ 
      */
     public static function getName($typeInteger)
     {
-        if(is_null(self::$TYPE_NAMES)) {
-            $f = function ($v) { return str_replace('Dallgoot\Yaml\\', '', $v);};
+        if (is_null(self::$TYPE_NAMES)) {
+            $f = function($v) { return str_replace('Dallgoot\Yaml\\', '', $v); };
             self::$TYPE_NAMES = array_map($f, array_flip(get_defined_constants(true)['user']));
         }
         return self::$TYPE_NAMES[$typeInteger];

yaml/Loader.php

Spacing +17 added lines, -17 removed lines

@@ -10,15 +10,15 @@ 
     //public
     public $errors = [];
 
-    public const EXCLUDE_DIRECTIVES = 1;//DONT include_directive
-    public const IGNORE_COMMENTS    = 2;//DONT include_comments
-    public const EXCEPTIONS_PARSING = 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 EXCEPTIONS_PARSING = 4; //THROW Exception on parsing Errors
+    public const NO_OBJECT_FOR_DATE = 8; //DONT import date strings as dateTime Object
     //privates
-    private $content;/* @var null|string */
-    private $filePath;/* @var null|string */
-    private $debug = 0;//TODO: determine levels
-    private $options = 0;/* @var int */
+    private $content; /* @var null|string */
+    private $filePath; /* @var null|string */
+    private $debug = 0; //TODO: determine levels
+    private $options = 0; /* @var int */
     //Exceptions messages
     private const INVALID_VALUE        = self::class.": at line %d";
     private const EXCEPTION_NO_FILE    = self::class.": file '%s' does not exists (or path is incorrect?)";
@@ -27,8 +27,8 @@ 
 
     public function __construct($absolutePath = null, $options = null, $debug = 0)
     {
-        $this->debug   = is_int($debug)   ? min($debug, 3) : 1;
-        $this->options = is_int($options) ? $options       : $this->options;
+        $this->debug   = is_int($debug) ? min($debug, 3) : 1;
+        $this->options = is_int($options) ? $options : $this->options;
         if (is_string($absolutePath)) {
             $this->load($absolutePath);
         }
@@ -79,10 +79,10 @@ 
         if (!is_array($source)) throw new \Exception(self::EXCEPTION_LINE_SPLIT);
         $previous = $root = new Node();
         $emptyLines = [];
-        $specialTypes = Y\LITTERALS | Y\BLANK;
+        $specialTypes = Y\LITTERALS|Y\BLANK;
         try {
             foreach ($source as $lineNb => $lineString) {
-                $n = new Node($lineString, $lineNb + 1);//TODO: useful???-> $this->debug && var_dump($n);
+                $n = new Node($lineString, $lineNb + 1); //TODO: useful???-> $this->debug && var_dump($n);
                 $parent  = $previous;
                 $deepest = $previous->getDeepestNode();
                 if ($deepest->type & Y\PARTIAL) {
@@ -114,7 +114,7 @@ 
                 var_dump("\033[33mParsed Structure\033[0m\n", $root);
                 die("Debug of root structure requested (remove debug level to suppress these)");
             }
-            $out = Builder::buildContent($root, $this->debug);//var_dump($out);exit();
+            $out = Builder::buildContent($root, $this->debug); //var_dump($out);exit();
             return $out;
         } catch (\ParseError $pe) {
             $message = $pe->getMessage()." on line ".$pe->getLine()." for '$this->filePath'\n";
@@ -150,19 +150,19 @@ 
     {
         $deepest = $previous->getDeepestNode();
         if ($deepest->type & Y\LITTERALS) {
-            $n->value = trim($lineString);//fall through
+            $n->value = trim($lineString); //fall through
         }
-        if ($deepest->type & (Y\LITTERALS | Y\REF_DEF | Y\SET_VALUE | Y\TAG)) {
+        if ($deepest->type & (Y\LITTERALS|Y\REF_DEF|Y\SET_VALUE|Y\TAG)) {
             $parent = $deepest;
             return false;
         }
-        if ($deepest->type & (Y\BLANK | Y\SCALAR) ) {
+        if ($deepest->type & (Y\BLANK|Y\SCALAR)) {
             if ($n->type === Y\SCALAR && ($deepest->getParent()->type & Y\LITTERALS)) {
                 $deepest->type = Y\SCALAR;
                 $deepest->value .= "\n".$n->value;
                 return true;
             } else {
-                if ($previous->type & (Y\ITEM | Y\SET_KEY)) {
+                if ($previous->type & (Y\ITEM|Y\SET_KEY)) {
                     $parent = $deepest->getParent();
                 }
             }

Braces +18 added lines, -6 removed lines

@@ -74,9 +74,13 @@ 
     public function parse($strContent = null)
     {
         $source = $this->content;
-        if (is_null($source)) $source = preg_split("/([^\n\r]+)/um", $strContent, 0, PREG_SPLIT_DELIM_CAPTURE);
+        if (is_null($source)) {
+            $source = preg_split("/([^\n\r]+)/um", $strContent, 0, PREG_SPLIT_DELIM_CAPTURE);
+        }
         //TODO : be more permissive on $strContent values
-        if (!is_array($source)) throw new \Exception(self::EXCEPTION_LINE_SPLIT);
+        if (!is_array($source)) {
+            throw new \Exception(self::EXCEPTION_LINE_SPLIT);
+        }
         $previous = $root = new Node();
         $emptyLines = [];
         $specialTypes = Y\LITTERALS | Y\BLANK;
@@ -93,7 +97,9 @@ 
                     $deepest->parse($deepest->value.' '.ltrim($lineString));
                 } else {
                     if ($n->type & $specialTypes) {
-                        if ($this->onSpecialType($n, $parent, $previous, $emptyLines)) continue;
+                        if ($this->onSpecialType($n, $parent, $previous, $emptyLines)) {
+                            continue;
+                        }
                     }
                     foreach ($emptyLines as $key => $node) {
                         $node->getParent()->add($node);
@@ -104,7 +110,9 @@ 
                     } elseif ($n->indent === $previous->indent) {
                         $parent = $previous->getParent();
                     } elseif ($n->indent > $previous->indent) {
-                        if ($this->onDeepestType($n, $parent, $previous, $lineString)) continue;
+                        if ($this->onDeepestType($n, $parent, $previous, $lineString)) {
+                            continue;
+                        }
                     }
                     $parent->add($n);
                     $previous = $n;
@@ -139,8 +147,12 @@ 
             }
         }
         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;
         }
         return false;

yaml/YamlObject.php

Indentation +9 added lines, -9 removed lines

@@ -2,15 +2,15 @@
 namespace Dallgoot\Yaml;
 
 /**
-  *  The returned object representing a YAML file content
-  *
-  * @method void addReference(string $name, $value)
-  * @method mixed getReference(string $name)
-  * @method array getAllReferences()
-  * @method void addComment($index, $value)
-  * @method string|array getComment($lineNumber)
-  * @method void setText(string $value)
-  * @method void addTag(string $value)
+ *  The returned object representing a YAML file content
+ *
+ * @method void addReference(string $name, $value)
+ * @method mixed getReference(string $name)
+ * @method array getAllReferences()
+ * @method void addComment($index, $value)
+ * @method string|array getComment($lineNumber)
+ * @method void setText(string $value)
+ * @method void addTag(string $value)
  */
 class YamlObject extends \ArrayIterator implements \JsonSerializable
 {

yaml/Builder.php

Spacing +17 added lines, -17 removed lines

@@ -26,14 +26,14 @@ 
     private static function buildDLL(DLL $node, &$parent)
     {
         $type = property_exists($node, "type") ? $node->type : null;
-        if ($type & (Y\RAW | Y\LITTERALS)) {
+        if ($type&(Y\RAW|Y\LITTERALS)) {
             return self::litteral($node, $type);
         }
         $p = $parent;
         switch ($type) {
             case Y\MAPPING: //fall through
-            case Y\SET:      $p = new \StdClass;break;
-            case Y\SEQUENCE: $p = [];break;
+            case Y\SET:      $p = new \StdClass; break;
+            case Y\SEQUENCE: $p = []; break;
             // case Y\KEY: $p = $parent;break;
         }
         $out = null;
@@ -54,10 +54,10 @@ 
     {
         list($line, $type, $value, $identifier) = [$node->line, $node->type, $node->value, $node->identifier];
         switch ($type) {
-            case Y\COMMENT: self::$_root->addComment($line, $value);return;
-            case Y\DIRECTIVE: return;//TODO
-            case Y\ITEM: self::buildItem($value, $parent);return;
-            case Y\KEY:  self::buildKey($node, $parent);return;
+            case Y\COMMENT: self::$_root->addComment($line, $value); return;
+            case Y\DIRECTIVE: return; //TODO
+            case Y\ITEM: self::buildItem($value, $parent); return;
+            case Y\KEY:  self::buildKey($node, $parent); return;
             case Y\REF_DEF: //fall through
             case Y\REF_CALL://TODO: self::build returns what ?
                 $tmp = is_object($value) ? self::build($value, $parent) : $node->getPhpValue();
@@ -72,8 +72,8 @@ 
             case Y\SET_VALUE:
                 $prop = array_keys(get_object_vars($parent));
                 $key = end($prop);
-                if (property_exists($value, "type") && ($value->type & (Y\ITEM | Y\MAPPING))) {
-                    $p = $value->type === Y\ITEM  ? [] : new \StdClass;
+                if (property_exists($value, "type") && ($value->type & (Y\ITEM|Y\MAPPING))) {
+                    $p = $value->type === Y\ITEM ? [] : new \StdClass;
                     self::build($value, $p);
                 } else {
                     $p = self::build($value, $parent->{$key});
@@ -82,7 +82,7 @@ 
                 return;
             case Y\TAG:
                 if ($parent === self::$_root) {
-                    $parent->addTag($identifier);return;
+                    $parent->addTag($identifier); return;
                 } else {
                     if (in_array($identifier, ['!binary', '!str'])) {
                         if ($value->value instanceof DLL) $value->value->type = Y\RAW;
@@ -110,7 +110,7 @@ 
         if (is_null($identifier)) {
             throw new \ParseError(sprintf(self::ERROR_NO_KEYNAME, $node->line));
         } else {
-            if ($value instanceof Node && ($value->type & (Y\KEY | Y\ITEM)) {
+            if ($value instanceof Node && ($value->type & (Y\KEY|Y\ITEM)) {
                 $parent->{$identifier} = $value->type === Y\KEY ? new \StdClass : [];
                 self::build($value, $parent->{$identifier});
             } elseif (is_object($value)) {
@@ -157,7 +157,7 @@ 
             if (!isset($documents[$currentDoc])) $documents[$currentDoc] = new DLL();
             $documents[$currentDoc]->push($child);
         }
-        $_debug >= 2 && var_dump($documents);//var_dump($documents);die("documents");
+        $_debug >= 2 && var_dump($documents); //var_dump($documents);die("documents");
         $content = array_map([self::class, 'buildDocument'], $documents, array_keys($documents));
         return count($content) === 1 ? $content[0] : $content;
     }
@@ -173,8 +173,8 @@ 
             throw new \ParseError(sprintf(self::INVALID_DOCUMENT, $key));
         } else {
             switch (true) {
-                case $isSequence: $list->type = Y\SEQUENCE;break;
-                case $isSet:      $list->type = Y\SET;break;
+                case $isSequence: $list->type = Y\SEQUENCE; break;
+                case $isSet:      $list->type = Y\SET; break;
                 case $isMapping://fall through
                 default:          $list->type = Y\MAPPING;
             }
@@ -198,10 +198,10 @@ 
         $children->rewind();
         $refIndent = $children->current()->indent;
         $separator = $type === Y\RAW ? '' : "\n";
-        $action = function ($c) { return $c->value; };
-        if ($type & Y\LITT_FOLDED) {
+        $action = function($c) { return $c->value; };
+        if ($type&Y\LITT_FOLDED) {
             $separator = ' ';
-            $action = function ($c) use ($refIndent) {
+            $action = function($c) use ($refIndent) {
                 return $c->indent > $refIndent || ($c->type & Y\EMPTY) ? "\n".$c->value : $c->value;
             };
         }

Braces +20 added lines, -8 removed lines

@@ -19,7 +19,9 @@ 
 
     private static function build(object $node, &$parent = null)
     {
-        if ($node instanceof DLL) return self::buildDLL($node, $parent);
+        if ($node instanceof DLL) {
+            return self::buildDLL($node, $parent);
+        }
         return self::buildNode($node, $parent);
     }
 
@@ -61,12 +63,15 @@ 
             case Y\REF_DEF: //fall through
             case Y\REF_CALL://TODO: self::build returns what ?
                 $tmp = is_object($value) ? self::build($value, $parent) : $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);
             case Y\SET_KEY:
                 $key = json_encode(self::build($value, $parent), JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
-                if (empty($key))
-                    throw new \Exception("Cant serialize complex key: ".var_export($value, true), 1);
+                if (empty($key)) {
+                                    throw new \Exception("Cant serialize complex key: ".var_export($value, true), 1);
+                }
                 $parent->{$key} = null;
                 return;
             case Y\SET_VALUE:
@@ -85,8 +90,11 @@ 
                     $parent->addTag($identifier);return;
                 } else {
                     if (in_array($identifier, ['!binary', '!str'])) {
-                        if ($value->value instanceof DLL) $value->value->type = Y\RAW;
-                        else $value->type = Y\RAW;
+                        if ($value->value instanceof DLL) {
+                            $value->value->type = Y\RAW;
+                        } else {
+                            $value->type = Y\RAW;
+                        }
                     }
                     $val = is_null($value) ? null : self::build($value, $node);
                     return new Tag($identifier, $val);
@@ -151,10 +159,14 @@ 
         }
         $_root->value->setIteratorMode(DLL::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 DLL();
+            if (!isset($documents[$currentDoc])) {
+                $documents[$currentDoc] = new DLL();
+            }
             $documents[$currentDoc]->push($child);
         }
         $_debug >= 2 && var_dump($documents);//var_dump($documents);die("documents");

yaml/Dumper.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
         if (is_null($dataType)) throw new \Exception(self::class.": No content to convert to Yaml", 1);
         self::$options = is_int($options) ? $options : self::OPTIONS;
         self::$result = new DLL;
-        self::$result->setIteratorMode(DLL::IT_MODE_FIFO | DLL::IT_MODE_DELETE);
+        self::$result->setIteratorMode(DLL::IT_MODE_FIFO|DLL::IT_MODE_DELETE);
         if ($dataType instanceof YamlObject) {
             self::dumpYamlObject($dataType);
         } elseif (is_array($dataType) && $dataType[0] instanceof YamlObject) {
@@ -108,9 +108,9 @@ 
         foreach ($array as $key => $item) {
             $lineStart = current($refKeys) === $key ? "- " : "- $key: ";
             if (is_scalar($item)) {
-                self::add($lineStart.$item, $indent );
+                self::add($lineStart.$item, $indent);
             } else {
-                self::add($lineStart, $indent );
+                self::add($lineStart, $indent);
                 self::dump($item, $indent + self::INDENT);
             }
             next($refKeys);
@@ -129,7 +129,7 @@ 
         if ($object instanceof Tag) {
             if (is_scalar($object->value)) {
                 return "!".$object->tagName.' '.$object->value;
-            } else{
+            } else {
                 yield "!".$object->tagName;
                 self::dump($object->value, $indent + self::INDENT);
             }

Braces +12 added lines, -4 removed lines

@@ -21,7 +21,9 @@ 
 
     public function __construct(int $options = null)
     {
-        if (is_int($options)) self::$options = $options;
+        if (is_int($options)) {
+            self::$options = $options;
+        }
     }
 
     /**
@@ -36,7 +38,9 @@ 
      */
     public static function toString($dataType, int $options):string
     {
-        if (is_null($dataType)) throw new \Exception(self::class.": No content to convert to Yaml", 1);
+        if (is_null($dataType)) {
+            throw new \Exception(self::class.": No content to convert to Yaml", 1);
+        }
         self::$options = is_int($options) ? $options : self::OPTIONS;
         self::$result = new DLL;
         self::$result->setIteratorMode(DLL::IT_MODE_FIFO | DLL::IT_MODE_DELETE);
@@ -74,8 +78,12 @@ 
         if (is_scalar($dataType)) {
             switch (gettype($dataType)) {
                 case 'boolean': return $dataType ? 'true' : 'false';
-                case 'float': if (is_infinite($dataType)) return $dataType > 0 ? '.inf' : '-.inf';
-                case 'double': if (is_nan($dataType)) return '.nan';
+                case 'float': if (is_infinite($dataType)) {
+                    return $dataType > 0 ? '.inf' : '-.inf';
+                }
+                case 'double': if (is_nan($dataType)) {
+                    return '.nan';
+                }
                 default:
                     return $dataType;
             }

yaml/Node.php

Indentation +7 added lines, -7 removed lines

@@ -288,13 +288,13 @@
     private static function getScalar(string $v)
     {
         $types = ['yes'   => true,
-                  'no'    => false,
-                  'true'  => true,
-                  'false' => false,
-                  'null'  => null,
-                  '.inf'  => INF,
-                  '-.inf' => -INF,
-                  '.nan'  => NAN
+                    'no'    => false,
+                    'true'  => true,
+                    'false' => false,
+                    'null'  => null,
+                    '.inf'  => INF,
+                    '-.inf' => -INF,
+                    '.nan'  => NAN
         ];
         if (isset($types[strtolower($v)])) return $types[strtolower($v)];
         if (R::isDate($v))   return date_create($v);

Spacing +8 added lines, -8 removed lines

@@ -92,9 +92,9 @@ 
         //modify type according to child
         if ($this->value instanceof DLL && !property_exists($this->value, "type")) {
             switch ($child->type) {
-                case Y\KEY:    $this->value->type = Y\MAPPING;break;
-                case Y\ITEM:   $this->value->type = Y\SEQUENCE;break;
-                case Y\SCALAR: $this->value->type = $this->type;break;
+                case Y\KEY:    $this->value->type = Y\MAPPING; break;
+                case Y\ITEM:   $this->value->type = Y\SEQUENCE; break;
+                case Y\SCALAR: $this->value->type = $this->type; break;
             }
         }
     }
@@ -117,7 +117,7 @@ 
      */
     public function parse(string $nodeString):Node
     {
-        $nodeValue = preg_replace("/^\t+/m", " ", $nodeString);//permissive to tabs but replacement
+        $nodeValue = preg_replace("/^\t+/m", " ", $nodeString); //permissive to tabs but replacement
         $this->indent = strspn($nodeValue, ' ');
         $nodeValue = ltrim($nodeValue);
         if ($nodeValue === '') {
@@ -153,7 +153,7 @@ 
             case '#': return [Y\COMMENT, ltrim($v)];
             case "-": return $this->onHyphen($nodeValue);
             case '%': return [Y\DIRECTIVE, ltrim($v)];
-            case '?': return [Y\SET_KEY,   empty($v) ? null : new Node(ltrim($v), $this->line)];
+            case '?': return [Y\SET_KEY, empty($v) ? null : new Node(ltrim($v), $this->line)];
             case ':': return [Y\SET_VALUE, empty($v) ? null : new Node(ltrim($v), $this->line)];
             case '>': return [Y\LITT_FOLDED, null];
             case '|': return [Y\LITT, null];
@@ -178,7 +178,7 @@ 
             if (!is_bool($hasComment)) {
                 $tmpNode = new Node(trim(substr($keyValue, 0, $hasComment)), $this->line);
                 if ($tmpNode->type !== Y\PARTIAL) {
-                    $comment = new Node(trim(substr($keyValue, $hasComment+1)), $this->line);
+                    $comment = new Node(trim(substr($keyValue, $hasComment + 1)), $this->line);
                     //TODO: modify "identifier" to specify if fullline comment or not
                     $this->add($comment);
                     $n = $tmpNode;
@@ -247,7 +247,7 @@ 
         $type = ['!' => Y\TAG, '&' => Y\REF_DEF, '*' => Y\REF_CALL][$nodeValue[0]];
         $pos = strpos($v, ' ');
         $this->identifier = is_bool($pos) ? $v : strstr($v, ' ', true);
-        $n = is_bool($pos) ? null : (new Node(trim(substr($nodeValue, $pos+1)), $this->line))->setParent($this);
+        $n = is_bool($pos) ? null : (new Node(trim(substr($nodeValue, $pos + 1)), $this->line))->setParent($this);
         return [$type, $n];
     }
 
@@ -270,7 +270,7 @@ 
             case Y\MAPPING_SHORT:  return self::getShortMapping(substr($v, 1, -1));
             //TODO : that's not robust enough, improve it
             case Y\SEQUENCE_SHORT:
-                $f = function ($e) { return self::getScalar(trim($e));};
+                $f = function($e) { return self::getScalar(trim($e)); };
                 return array_map($f, explode(",", substr($v, 1, -1)));
             default:
                 trigger_error("Error can not get PHP type for ".Y\getName($this->type), E_USER_WARNING);

Braces +42 added lines, -14 removed lines

@@ -56,7 +56,9 @@ 
      */
     public function getParent(int $indent = null):Node
     {
-        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;
@@ -76,7 +78,9 @@ 
     {
         $child->setParent($this);
         $current = $this->value;
-        if ($this->type & Y\LITTERALS) $child->type = Y\SCALAR;
+        if ($this->type & Y\LITTERALS) {
+            $child->type = Y\SCALAR;
+        }
         if (is_null($current)) {
             $this->value = $child;
             return;
@@ -147,8 +151,12 @@ 
             $type = R::isProperlyQuoted($nodeValue) ? Y\QUOTED : Y\PARTIAL;
             return [$type, $nodeValue];
         }
-        if (in_array($nodeValue[0], ['{', '[']))      return $this->onObject($nodeValue);
-        if (in_array($nodeValue[0], ['!', '&', '*'])) return $this->onNodeAction($nodeValue);
+        if (in_array($nodeValue[0], ['{', '['])) {
+            return $this->onObject($nodeValue);
+        }
+        if (in_array($nodeValue[0], ['!', '&', '*'])) {
+            return $this->onNodeAction($nodeValue);
+        }
         switch ($nodeValue[0]) {
             case '#': return [Y\COMMENT, ltrim($v)];
             case "-": return $this->onHyphen($nodeValue);
@@ -200,9 +208,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\MAPPING_SHORT, $value];
-        if (preg_match(R::SEQUENCE, $value))        return [Y\SEQUENCE_SHORT, $value];
+        if (json_last_error() === JSON_ERROR_NONE) {
+            return [Y\JSON, $value];
+        }
+        if (preg_match(R::MAPPING, $value)) {
+            return [Y\MAPPING_SHORT, $value];
+        }
+        if (preg_match(R::SEQUENCE, $value)) {
+            return [Y\SEQUENCE_SHORT, $value];
+        }
         return [Y\PARTIAL, $value];
     }
 
@@ -217,7 +231,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)];
@@ -260,7 +276,9 @@ 
     public function getPhpValue()
     {
         $v = $this->value;
-        if (is_null($v)) return null;
+        if (is_null($v)) {
+            return null;
+        }
         switch ($this->type) {
             case Y\JSON:   return json_decode($v, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR);
             case Y\QUOTED: return substr($v, 1, -1);
@@ -296,9 +314,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);
     }
 
@@ -311,8 +335,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));
+        }
         // TODO: remove these if not needed
         // if preg_match("/^([\d.]+e[-+]\d{1,2})$/", $v)://fall through
         // if preg_match("/^([-+]?(?:\d+|\d*.\d+))$/", $v):