dallgoot / yaml

yaml/Tag.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         if (is_null($tagName)) {
             throw new \Exception(self::class.": a tag MUST have a name", 1);
         }
-        $this->tagName  = $tagName;
+        $this->tagName = $tagName;
         $this->value = $value;
     }
 

yaml/Regex.php

Indentation +5 added lines, -5 removed lines

@@ -14,10 +14,10 @@ 
     const AN = "[\w ]+";
     const NUM = "-?[\d.e]+";
     const SIMPLE = "(?P<sv>".self::NULL."|".
-                                  self::FALSE."|".
-                                  self::TRUE."|".
-                                  self::AN."|".
-                                  self::NUM.")";
+                                    self::FALSE."|".
+                                    self::TRUE."|".
+                                    self::AN."|".
+                                    self::NUM.")";
     private const seqForMap = "(?P<seq>\[(?:(?:(?P>sv)|(?P>seq)|(?P>map)),?\s*)+\])";
     private const mapForSeq = "(?P<map>{\s*(?:".self::AN."\s*:\s*(?:(?P>sv)|(?P>seq)|(?P>map)),?\s*)+})";
 
@@ -41,7 +41,7 @@ 
         $matchSpaced    = preg_match($spaced, $v);
         $matchIso       = preg_match($iso8601, $v);
         if (is_bool($matchDate) || is_bool($matchCanonical) || is_bool($matchSpaced) || is_bool($matchIso))
-          throw new \Exception("Regex date error");
+            throw new \Exception("Regex date error");
 
         return $matchDate || $matchCanonical || $matchSpaced || $matchIso;
     }

Spacing +4 added lines, -4 removed lines

@@ -32,10 +32,10 @@
     {
         $d         = "\\d{4}([-\\/])\\d{2}\\1\\d{2}";
         $h         = "\\d{2}(:)\\d{2}\\2\\d{2}";
-        $date      =  "/^$d$/";// 2002-12-14, 2002/12/14
-        $canonical =  "/^$d(?:t| )$h\\.\\dz?$/im";// 2001-12-15T02:59:43.1Z
-        $spaced    =  "/^$d(?:t| )$h\\.\\d{2} [-+]\\d$/im";// 2001-12-14 21:59:43.10 -5
-        $iso8601   =  "/^$d(?:t| )$h\\.\\d{2}[-+]\\d{2}\\2\\d{2}/im";// 2001-12-14t21:59:43.10-05:00
+        $date      = "/^$d$/"; // 2002-12-14, 2002/12/14
+        $canonical = "/^$d(?:t| )$h\\.\\dz?$/im"; // 2001-12-15T02:59:43.1Z
+        $spaced    = "/^$d(?:t| )$h\\.\\d{2} [-+]\\d$/im"; // 2001-12-14 21:59:43.10 -5
+        $iso8601   = "/^$d(?:t| )$h\\.\\d{2}[-+]\\d{2}\\2\\d{2}/im"; // 2001-12-14t21:59:43.10-05:00
         $matchDate      = preg_match($date, $v);
         $matchCanonical = preg_match($canonical, $v);
         $matchSpaced    = preg_match($spaced, $v);

Braces +3 added lines, -2 removed lines

@@ -40,8 +40,9 @@
         $matchCanonical = preg_match($canonical, $v);
         $matchSpaced    = preg_match($spaced, $v);
         $matchIso       = preg_match($iso8601, $v);
-        if (is_bool($matchDate) || is_bool($matchCanonical) || is_bool($matchSpaced) || is_bool($matchIso))
-          throw new \Exception("Regex date error");
+        if (is_bool($matchDate) || is_bool($matchCanonical) || is_bool($matchSpaced) || is_bool($matchIso)) {
+                  throw new \Exception("Regex date error");
+        }
 
         return $matchDate || $matchCanonical || $matchSpaced || $matchIso;
     }

yaml/YamlObject.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
 
     public function __construct()
     {
-        parent::__construct([], 1);//1 = Array indices can be accessed as properties in read/write.
+        parent::__construct([], 1); //1 = Array indices can be accessed as properties in read/write.
         $this->__yaml__object__api = new API();
     }
 
@@ -28,7 +28,7 @@ 
     public function __call($funcName, $arguments)
     {
         $reflectAPI = new \ReflectionClass(get_class($this->__yaml__object__api));
-        $getName = function ($o) { return $o->name; };
+        $getName = function($o) { return $o->name; };
         $publicApi  = array_map($getName, $reflectAPI->getMethods(\ReflectionMethod::IS_PUBLIC));
         $privateApi = array_map($getName, $reflectAPI->getMethods(\ReflectionMethod::IS_PRIVATE));
         if (!in_array($funcName, $publicApi) && !in_array($funcName, $privateApi)) {

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;
     }
 }

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 namespace Dallgoot\Yaml;
 
 /**
-  *  The returned object representing a YAML file content
+ *  The returned object representing a YAML file content
  * @category tag in class comment
  * @package tag in class comment
  * @author tag in class comment

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);

Braces +45 added lines, -15 removed lines

@@ -55,7 +55,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;
@@ -146,8 +148,12 @@ 
             $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);
+        }
         switch ($first) {
             case '#': return [Y\COMMENT, ltrim($v)];
             case "-": return $this->onHyphen($nodeValue);
@@ -199,9 +205,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];
     }
 
@@ -216,7 +228,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)];
@@ -259,9 +273,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);
+        }
         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);
@@ -290,9 +310,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);
     }
 
@@ -305,8 +331,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):

Spacing +10 added lines, -10 removed lines

@@ -94,8 +94,8 @@ 
             //modify type according to child
             switch ($child->type) {
                 case Y\COMMENT: //fall through
-                case Y\KEY:     $this->value->type = Y\MAPPING;break;
-                case Y\ITEM:    $this->value->type = Y\SEQUENCE;break;
+                case Y\KEY:     $this->value->type = Y\MAPPING; break;
+                case Y\ITEM:    $this->value->type = Y\SEQUENCE; break;
             }
             $this->type & Y\LITTERALS && $this->value->type = $this->type;
         }
@@ -119,7 +119,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 === '') {
@@ -156,7 +156,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];
@@ -181,8 +181,8 @@ 
             if (!is_bool($hasComment)) {
                 $tmpNode = new Node(trim(substr($value, 0, $hasComment)), $this->line);
                 if ($tmpNode->type !== Y\PARTIAL) {
-                    $comment = new Node(trim(substr($value, $hasComment+1)), $this->line);
-                    $comment->identifier = true;//to specify it is NOT a fullline comment
+                    $comment = new Node(trim(substr($value, $hasComment + 1)), $this->line);
+                    $comment->identifier = true; //to specify it is NOT a fullline comment
                     $this->add($comment);
                     $n = $tmpNode;
                 }
@@ -250,7 +250,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];
     }
 
@@ -264,8 +264,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);
         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);
@@ -321,7 +321,7 @@ 
     {
         $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;

example.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-require_once __DIR__ . '/vendor/autoload.php';
+require_once __DIR__.'/vendor/autoload.php';
 
 use \Dallgoot\Yaml as Y;
 

yaml/Compact.php

Braces +6 added lines, -2 removed lines

@@ -27,8 +27,12 @@
     public function jsonSerialize()
     {
         $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);
+        }
     }
 
     public static function wrap($arrayOrObject)

yaml/Yaml.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     public static function getName($typeInteger)
     {
         if (is_null(self::$TYPE_NAMES)) {
-            $f = function ($v) { return str_replace('Dallgoot\Yaml\\', '', $v);};
+            $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/Dumper.php

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 = 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;
         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((float) $dataType)) return $dataType > 0 ? '.inf' : '-.inf';
-                case 'double': if (is_nan((float) $dataType)) return '.nan';
+                case 'float': if (is_infinite((float) $dataType)) {
+                    return $dataType > 0 ? '.inf' : '-.inf';
+                }
+                case 'double': if (is_nan((float) $dataType)) {
+                    return '.nan';
+                }
                 default:
                     return $dataType;
             }

yaml/Loader.php

Spacing +7 added lines, -7 removed lines

@@ -17,17 +17,17 @@ 
     //public
     public $errors = [];
 
-    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
@@ -112,8 +112,8 @@ 
                     }
                     $emptyLines = [];
                     switch ($n->indent <=> $previous->indent) {
-                        case -1: $previous->getParent($n->indent)->add($n);break;
-                        case 0:  $previous->getParent()->add($n);break;
+                        case -1: $previous->getParent($n->indent)->add($n); break;
+                        case 0:  $previous->getParent()->add($n); break;
                         default:
                             if ($this->onDeepestType($n, $previous, $lineString)) continue 2;
                             $previous->add($n);

Braces +15 added lines, -5 removed lines

@@ -84,9 +84,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;
@@ -115,7 +119,9 @@ 
                         case -1: $previous->getParent($n->indent)->add($n);break;
                         case 0:  $previous->getParent()->add($n);break;
                         default:
-                            if ($this->onDeepestType($n, $previous, $lineString)) continue 2;
+                            if ($this->onDeepestType($n, $previous, $lineString)) {
+                                continue 2;
+                            }
                             $previous->add($n);
                     }
                     $previous = $n;
@@ -155,8 +161,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;