dallgoot / yaml

yaml/YamlObject.php

Braces +6 added lines, -2 removed lines

@@ -40,8 +40,12 @@
     {
         $prop = get_object_vars($this);
         unset($prop["__yaml__object__api"]);
-        if (count($prop) > 0) return $prop;
-        if (count($this) > 0) return iterator_to_array($this);
+        if (count($prop) > 0) {
+            return $prop;
+        }
+        if (count($this) > 0) {
+            return iterator_to_array($this);
+        }
         return $this->__yaml__object__api->value;
     }
 }

yaml/Builder.php

Braces +24 added lines, -8 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);
     }
 
@@ -83,11 +85,15 @@ 
             } else {
                 $tmp = Node2PHP::get($node);
             }
-            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);
         }
 
-        if ($type & Y::COMMENT) self::$_root->addComment($node->line, $node->value);
+        if ($type & Y::COMMENT) {
+            self::$_root->addComment($node->line, $node->value);
+        }
         $typesActions = [Y::DIRECTIVE => 'buildDirective',
                          Y::ITEM      => 'buildItem',
                          Y::KEY       => 'buildKey',
@@ -179,10 +185,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));
@@ -280,9 +290,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 preg_replace(['/ +(\n)/','/(\n+) +/'], "$1", implode(' ',  $lines));
+        if ($type & Y::RAW) {
+            return implode('',   $lines);
+        }
+        if ($type & Y::LITT) {
+            return implode("\n", $lines);
+        }
+        if ($type & Y::LITT_FOLDED) {
+            return preg_replace(['/ +(\n)/','/(\n+) +/'], "$1", implode(' ',  $lines));
+        }
         return '';
     }
 

yaml/Compact.php

Braces +6 added lines, -2 removed lines

@@ -25,8 +25,12 @@
     public function jsonSerialize():array
     {
         $prop = get_object_vars($this);
-        if (count($prop) > 0) return $prop;
-        if (count($this) > 0) return iterator_to_array($this);
+        if (count($prop) > 0) {
+            return $prop;
+        }
+        if (count($this) > 0) {
+            return iterator_to_array($this);
+        }
     }
 
     /**

yaml/Loader.php

Braces +15 added lines, -5 removed lines

@@ -88,7 +88,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 { //var_dump($source);
@@ -101,7 +103,9 @@ 
                 $n = new Node($lineString, $lineNb);
                 $deepest = $previous->getDeepestNode();
                 if ($n->type & (Y::LITTERALS|Y::BLANK|Y::COMMENT) || $deepest->type & Y::PARTIAL) {
-                    if ($this->onSpecialType($n, $previous, $emptyLines, $lineString)) continue;
+                    if ($this->onSpecialType($n, $previous, $emptyLines, $lineString)) {
+                        continue;
+                    }
                 }
                 //Note: 6.6 comments: Note that outside scalar content, a line containing only white space characters is taken to be a comment line.
                     foreach ($emptyLines as $blankNode) {
@@ -135,7 +139,9 @@ 
                             }
                         }
                 }
-                if ($this->onContextType($n, $target, $lineString)) continue;
+                if ($this->onContextType($n, $target, $lineString)) {
+                    continue;
+                }
                 $target->add($n);
                 $previous = $n;
             }
@@ -172,8 +178,12 @@ 
             return true;
         }
         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/Dumper.php

Braces +15 added lines, -5 removed lines

@@ -39,7 +39,9 @@ 
      */
     public static function toString($dataType, int $options = null):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;
         if ($dataType instanceof YamlObject) {
@@ -73,9 +75,13 @@ 
         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 'float': if (is_infinite($dataType)) {
+                    return $dataType > 0 ? '.inf' : '-.inf';
+                }
                               return sprintf('%.2F', $dataType);
-                case 'double': if (is_nan((float) $dataType)) return '.nan';
+                case 'double': if (is_nan((float) $dataType)) {
+                    return '.nan';
+                }
                 default:
                     return $dataType;
             }
@@ -139,9 +145,13 @@ 
                 self::add(self::dump($obj->value, $indent + self::INDENT), $indent + self::INDENT);
             }
         }
-        if ($obj instanceof Compact) return self::dumpCompact($obj, $indent);
+        if ($obj instanceof Compact) {
+            return self::dumpCompact($obj, $indent);
+        }
         //TODO:  consider dumping datetime as date strings according to a format provided by user or default
-        if ($obj instanceof \DateTime) return $obj->format('Y-m-d');
+        if ($obj instanceof \DateTime) {
+            return $obj->format('Y-m-d');
+        }
         $propList = get_object_vars($obj);
         foreach ($propList as $property => $value) {
             if (is_scalar($value)) {

yaml/Node.php

Braces +15 added lines, -5 removed lines

@@ -67,7 +67,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) {
             if ($cursor->indent === $indent && $cursor->type !== $type) {
@@ -106,13 +108,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;
+            }
         }
     }
 

yaml/Node2PHP.php

Braces +21 added lines, -8 removed lines

@@ -21,10 +21,15 @@ 
      */
     public static function get(Node $n)
     {
-        if (is_null($n->value)) return null;
-        if ($n->type & (Y::REF_CALL | Y::SCALAR)) return self::getScalar($n->value);
-        if ($n->type & (Y::COMPACT_MAPPING | Y::COMPACT_SEQUENCE))
-            return self::getCompact(substr($n->value, 1, -1), $n->type);
+        if (is_null($n->value)) {
+            return null;
+        }
+        if ($n->type & (Y::REF_CALL | Y::SCALAR)) {
+            return self::getScalar($n->value);
+        }
+        if ($n->type & (Y::COMPACT_MAPPING | Y::COMPACT_SEQUENCE)) {
+                    return self::getCompact(substr($n->value, 1, -1), $n->type);
+        }
         $expected = [Y::JSON   => json_decode($n->value, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR),
                      Y::QUOTED => trim($n->value, "\"'"),
                      Y::RAW    => strval($n->value)];
@@ -40,8 +45,12 @@ 
      */
     private static function getScalar(string $v)
     {
-        if (R::isDate($v))   return date_create($v);
-        if (R::isNumber($v)) return self::getNumber($v);
+        if (R::isDate($v)) {
+            return date_create($v);
+        }
+        if (R::isNumber($v)) {
+            return self::getNumber($v);
+        }
         $types = ['yes'   => true,
                     'no'    => false,
                     'true'  => true,
@@ -63,8 +72,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);
     }