dallgoot / yaml

Node.php

Braces +45 added lines, -15 removed lines

@@ -38,7 +38,9 @@ 
 
     public function getParent($indent = null):Node
     {
-        if (is_null($indent)) return $this->parent ?? $this;
+        if (is_null($indent)) {
+            return $this->parent ?? $this;
+        }
         $cursor = $this;
         while ($cursor->indent >= $indent) {
             $cursor = $cursor->parent;
@@ -50,7 +52,9 @@ 
     {
         $child->setParent($this);
         $current = $this->value;
-        if (in_array($this->type, T::$LITTERALS)) $child->type = T::SCALAR;
+        if (in_array($this->type, T::$LITTERALS)) {
+            $child->type = T::SCALAR;
+        }
         if (is_null($current)) {
             $this->value = $child;
             return;
@@ -114,8 +118,12 @@ 
             $type = R::isProperlyQuoted($nodeValue) ? T::QUOTED : T::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 [T::COMMENT, ltrim($v)];
             case "-": return $this->onHyphen($nodeValue);
@@ -154,9 +162,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 [T::JSON, $value];
-        if (preg_match(R::MAPPING, $value))         return [T::MAPPING_SHORT, $value];
-        if (preg_match(R::SEQUENCE, $value))        return [T::SEQUENCE_SHORT, $value];
+        if (json_last_error() === JSON_ERROR_NONE) {
+            return [T::JSON, $value];
+        }
+        if (preg_match(R::MAPPING, $value)) {
+            return [T::MAPPING_SHORT, $value];
+        }
+        if (preg_match(R::SEQUENCE, $value)) {
+            return [T::SEQUENCE_SHORT, $value];
+        }
         return [T::PARTIAL, $value];
     }
 
@@ -164,7 +178,9 @@ 
     {
         if (substr($nodeValue, 0, 3) === '---') {
             $rest = trim(substr($nodeValue, 3));
-            if (empty($rest)) return [T::DOC_START, null];
+            if (empty($rest)) {
+                return [T::DOC_START, null];
+            }
             $n = new Node($rest, $this->line);
             $n->indent = $this->indent + 4;
             return [T::DOC_START, $n->setParent($this)];
@@ -193,7 +209,9 @@ 
     public function getPhpValue()
     {
         $v = $this->value;
-        if (is_null($v)) return null;
+        if (is_null($v)) {
+            return null;
+        }
         switch ($this->type) {
             case T::JSON:   return json_decode($v, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR);
             case T::QUOTED: return substr($v, 1, -1);
@@ -222,16 +240,26 @@ 
                   '-.inf' => -INF,
                   '.nan'  => NAN
         ];
-        if (in_array(strtolower($v), array_keys($types))) return $types[strtolower($v)];
-        if (R::isDate($v))   return date_create($v);
-        if (R::isNumber($v)) return self::getNumber($v);
+        if (in_array(strtolower($v), array_keys($types))) {
+            return $types[strtolower($v)];
+        }
+        if (R::isDate($v)) {
+            return date_create($v);
+        }
+        if (R::isNumber($v)) {
+            return self::getNumber($v);
+        }
         return strval($v);
     }
 
     private static function getNumber($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));
+        }
         // if preg_match("/^([\d.]+e[-+]\d{1,2})$/", $v)://fall through
         // if preg_match("/^([-+]?(?:\d+|\d*.\d+))$/", $v):
             return is_bool(strpos($v, '.')) ? intval($v) : floatval($v);
@@ -254,7 +282,9 @@ 
                 'indent'=> $this->indent,
                 'type'  => T::getName($this->type),
                 'value' => $this->value];
-        if (!is_null($this->identifier)) $out['type'] .= "($this->identifier)";
+        if (!is_null($this->identifier)) {
+            $out['type'] .= "($this->identifier)";
+        }
         return $out;
     }
 }

Loader.php

Braces +18 added lines, -6 removed lines

@@ -66,9 +66,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 = [T::LITTERAL, T::LITTERAL_FOLDED, T::EMPTY];
@@ -85,7 +89,9 @@ 
                     $deepest->parse($deepest->value.' '.ltrim($lineString));
                 } else {
                     if (in_array($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);
@@ -96,7 +102,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;
@@ -125,8 +133,12 @@ 
         $deepest = $previous->getDeepestNode();
         switch ($n->type) {
             case T::EMPTY:
-                if ($previous->type === T::SCALAR) $emptyLines[] = $n->setParent($previous->getParent());
-                if (in_array($deepest->type, T::$LITTERALS)) $emptyLines[] = $n->setParent($deepest);
+                if ($previous->type === T::SCALAR) {
+                    $emptyLines[] = $n->setParent($previous->getParent());
+                }
+                if (in_array($deepest->type, T::$LITTERALS)) {
+                    $emptyLines[] = $n->setParent($deepest);
+                }
                 return true;
             case T::LITTERAL://fall through
             case T::LITTERAL_FOLDED://var_dump($deepest);exit();

Builder.php

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 T::REF_DEF: //fall through
             case T::REF_CALL://TODO: self::build returns what ?
                 $tmp = is_object($value) ? self::build($value, $parent) : $node->getPhpValue();
-                if ($type === T::REF_DEF) self::$root->addReference($identifier, $tmp);
+                if ($type === T::REF_DEF) {
+                    self::$root->addReference($identifier, $tmp);
+                }
                 return self::$root->getReference($identifier);
             case T::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 T::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 = T::RAW;
-                        else $value->type = T::RAW;
+                        if ($value->value instanceof DLL) {
+                            $value->value->type = T::RAW;
+                        } else {
+                            $value->type = T::RAW;
+                        }
                     }
                     $val = is_null($value) ? null : self::build($value, $node);
                     return new Tag($identifier, $val);
@@ -148,10 +156,14 @@ 
         }
         $root->value->setIteratorMode(DLL::IT_MODE_DELETE);
         foreach ($root->value as $child) {
-            if ($child->type === T::DOC_START) $totalDocStart++;
+            if ($child->type === T::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);

Dumper.php

Braces +12 added lines, -4 removed lines

@@ -21,12 +21,16 @@ 
 
     public function __construct(int $options = null)
     {
-        if (is_int($options)) self::$options = $options;
+        if (is_int($options)) {
+            self::$options = $options;
+        }
     }
 
     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);
@@ -55,8 +59,12 @@ 
         if (is_scalar($dataType)) {
             switch (gettype($dataType)) {
                 case 'boolean': return $dataType ? 'true' : 'false';break;
-                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;
             }