dallgoot / yaml

src/Builder.php

Spacing +4 added lines, -4 removed lines

@@ -68,7 +68,7 @@ 
      *
      * @return YamlObject the YAML document as an object
      */
-    public function buildDocument(NodeList &$list, int $docNum): YamlObject
+    public function buildDocument(NodeList&$list, int $docNum): YamlObject
     {
         $yamlObject = new YamlObject($this->_options);
         $rootNode   = new Root();
@@ -83,18 +83,18 @@ 
             }
             return $rootNode->build($yamlObject);
         } catch (\Throwable $e) {
-            throw new \ParseError(sprintf(self::INVALID_DOCUMENT, $docNum) . ':' . $e->getMessage(), 2, $e);
+            throw new \ParseError(sprintf(self::INVALID_DOCUMENT, $docNum).':'.$e->getMessage(), 2, $e);
         }
     }
 
-    public function pushAndSave(DocEnd $child, NodeList &$buffer, array &$documents)
+    public function pushAndSave(DocEnd $child, NodeList&$buffer, array &$documents)
     {
         $buffer->push($child);
         $documents[] = $this->buildDocument($buffer, count($documents) + 1);
         $buffer = new NodeList();
     }
 
-    public function saveAndPush(DocStart $child, NodeList &$buffer, array &$documents)
+    public function saveAndPush(DocStart $child, NodeList&$buffer, array &$documents)
     {
         if ($buffer->count() > 0 && $buffer->hasContent()) {
             $documents[] = $this->buildDocument($buffer, count($documents) + 1);

src/NodeFactory.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
  */
 class NodeFactory
 {
-    private const JSON_OPTIONS = \JSON_PARTIAL_OUTPUT_ON_ERROR | \JSON_UNESCAPED_SLASHES;
+    private const JSON_OPTIONS = \JSON_PARTIAL_OUTPUT_ON_ERROR|\JSON_UNESCAPED_SLASHES;
 
     final public static function get(string $nodeString, int $line = 0, bool $debug = false): NodeGeneric
     {
@@ -32,7 +32,7 @@ 
             $match => new Nodes\Key($nodeString, $line, $matches),
             default => self::onCharacter($trimmed[0], $nodeString, $line)
         };
-        if ($debug) echo $line . ":" . get_class($node) . "\n";
+        if ($debug) echo $line.":".get_class($node)."\n";
         return $node;
     }
 

Braces +3 added lines, -1 removed lines

@@ -32,7 +32,9 @@
             $match => new Nodes\Key($nodeString, $line, $matches),
             default => self::onCharacter($trimmed[0], $nodeString, $line)
         };
-        if ($debug) echo $line . ":" . get_class($node) . "\n";
+        if ($debug) {
+            echo $line . ":" . get_class($node) . "\n";
+        }
         return $node;
     }
 

src/Tag/SymfonySchema.php

Spacing +2 added lines, -2 removed lines

@@ -40,12 +40,12 @@
     public function __call($name, $arguments)
     {
         //TODO : handle 'php/object'
-        if(in_array($name, ['php/object'])) {
+        if (in_array($name, ['php/object'])) {
             return match($name) {
                 'php/object' => self::PHPobjectHandler($arguments),
                 default => null,
             };
         }
-        throw new \Exception("no handler for tag '$name' in " . self::class, 1);
+        throw new \Exception("no handler for tag '$name' in ".self::class, 1);
     }
 }

src/Tag/TagFactory.php

Indentation -1 removed lines

@@ -62,7 +62,6 @@
 %TAG !! tag:example.com,2000:app/
 %TAG !e! tag:example.com,2000:app/
 !<tag:yaml.org,2002:str> foo :
-
      */
     /**
      * Add Handlers for legacy Yaml tags

src/Tag/CoreSchema.php

Spacing +3 added lines, -3 removed lines

@@ -23,8 +23,8 @@ 
     const SCHEMA_URI = 'tag:yaml.org,2002:';
     const BUILDING_NAMESPACE = "\\";
 
-    private const ERROR_SET = 'Error : tag ' . self::class . ":'set' can NOT be a single Node : must be a NodeList";
-    private const ERROR_OMAP = 'Error : tag ' . self::class . ":'omap' MUST have Nodes\Item *with* a Nodes\Key";
+    private const ERROR_SET = 'Error : tag '.self::class.":'set' can NOT be a single Node : must be a NodeList";
+    private const ERROR_OMAP = 'Error : tag '.self::class.":'omap' MUST have Nodes\Item *with* a Nodes\Key";
 
     public function __call($name, $arguments)
     {
@@ -85,7 +85,7 @@ 
      *
      * @return string  The value considered as 'binary' Note: the difference with strHandler is that multiline have not separation
      */
-    public function binary($node, NodeGeneric &$parent = null)
+    public function binary($node, NodeGeneric&$parent = null)
     {
         return $this->str($node, $parent);
     }

src/Loader.php

Spacing +10 added lines, -10 removed lines

@@ -35,10 +35,10 @@ 
     private array $_blankBuffer = [];
 
     //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?)";
-    private const EXCEPTION_READ_ERROR = self::class . ": file '%s' failed to be loaded (permission denied ?)";
-    private const EXCEPTION_LINE_SPLIT = self::class . ": content is not a string (maybe a file error?)";
+    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?)";
+    private const EXCEPTION_READ_ERROR = self::class.": file '%s' failed to be loaded (permission denied ?)";
+    private const EXCEPTION_LINE_SPLIT = self::class.": content is not a string (maybe a file error?)";
 
     /**
      * Loader constructor
@@ -100,9 +100,9 @@ 
     private function getSourceGenerator(?string $strContent = null): \Generator
     {
         if (is_null($strContent)) {
-            if(is_null($this->content)) {
+            if (is_null($this->content)) {
                 throw new \Exception(self::EXCEPTION_LINE_SPLIT);
-            }else {
+            } else {
                 $source = $this->content;
             }
         } else {
@@ -130,7 +130,7 @@ 
      */
     public function parse(?string $strContent = null)
     {
-        if(!is_null($strContent)) {
+        if (!is_null($strContent)) {
             $this->content = null;
         }
         $generator = $this->getSourceGenerator($strContent);
@@ -185,7 +185,7 @@ 
     {
         $deepest = $previous->getDeepestNode();
         if ($deepest instanceof Nodes\Partial) {
-            return $deepest->specialProcess($current,  $this->_blankBuffer);
+            return $deepest->specialProcess($current, $this->_blankBuffer);
         } elseif (!($current instanceof Nodes\Partial)) {
             return $current->specialProcess($previous, $this->_blankBuffer);
         }
@@ -195,11 +195,11 @@ 
     private function onError(\Throwable $e)
     {
         $file = $this->filePath ? realpath($this->filePath) : '#YAML STRING#';
-        $message = $e->getMessage() . "\n " . $e->getFile() . ":" . $e->getLine();
+        $message = $e->getMessage()."\n ".$e->getFile().":".$e->getLine();
         if ($this->_options & self::NO_PARSING_EXCEPTIONS) {
             self::$error = $message;
             return null;
         }
-        throw new \Exception($message . " for $file:" . $e->getLine(), 1, $e);
+        throw new \Exception($message." for $file:".$e->getLine(), 1, $e);
     }
 }

Braces +4 added lines, -2 removed lines

@@ -102,7 +102,7 @@ 
         if (is_null($strContent)) {
             if(is_null($this->content)) {
                 throw new \Exception(self::EXCEPTION_LINE_SPLIT);
-            }else {
+            } else {
                 $source = $this->content;
             }
         } else {
@@ -139,7 +139,9 @@ 
         try {
             foreach ($generator as $lineNB => $lineString) {
                 $node = NodeFactory::get($lineString, $lineNB, $debugNodeFactory);
-                if ($this->needsSpecialProcess($node, $previous)) continue;
+                if ($this->needsSpecialProcess($node, $previous)) {
+                    continue;
+                }
                 $this->_attachBlankLines($previous);
                 $target = match ($node->indent <=> $previous->indent) {
                     -1 => $previous->getTargetOnLessIndent($node),

src/Nodes/CompactMapping.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         parent::__construct($nodeString, $line);
         preg_match_all(Regex::MAPPING_VALUES, trim(substr(trim($nodeString), 1, -1)), $matches);
         foreach ($matches['k'] as $index => $property) {
-            $pair = $property . ': ' . trim($matches['v'][$index]);
+            $pair = $property.': '.trim($matches['v'][$index]);
             $child = NodeFactory::get($pair, (int) $line);
             $child->indent = null;
             $this->add($child);

src/Nodes/SetKey.php

Spacing +2 added lines, -2 removed lines

@@ -34,8 +34,8 @@
     {
         $built = is_null($this->value) ? null : $this->value->build();
         $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($this->value, true));
+        $key = json_encode($stringKey, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
+        if (empty($key)) throw new \Exception("Cant serialize complex key: ".var_export($this->value, true));
         $parent->{trim($key, '\'" ')} = null;
         return null;
     }

Braces +3 added lines, -1 removed lines

@@ -35,7 +35,9 @@
         $built = is_null($this->value) ? null : $this->value->build();
         $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($this->value, true));
+        if (empty($key)) {
+            throw new \Exception("Cant serialize complex key: " . var_export($this->value, true));
+        }
         $parent->{trim($key, '\'" ')} = null;
         return null;
     }

src/Nodes/Directive.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
 class Directive extends NodeGeneric
 {
     private const ERROR_BUILDING = "Error : can not build Directive";
-    private const WARNING_LOWER_VERSION  = "The declared version '%s' is obsolete, there may be features that are deprecated and therefore not handled, minimum supported is: " . Yaml::VERSION_SUPPORT;
-    private const WARNING_HIGHER_VERSION = "The declared version '%s' is not yet supported, minimum supported is: " . Yaml::VERSION_SUPPORT;
+    private const WARNING_LOWER_VERSION  = "The declared version '%s' is obsolete, there may be features that are deprecated and therefore not handled, minimum supported is: ".Yaml::VERSION_SUPPORT;
+    private const WARNING_HIGHER_VERSION = "The declared version '%s' is not yet supported, minimum supported is: ".Yaml::VERSION_SUPPORT;
 
     /**
      * Builds a Directive : update YamlObject if applicable.