Passed
Push — master ( f0c5ab...d6277d )
by stéphane
01:45
created
sources/Dumper.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -41,7 +41,9 @@  discard block
 block discarded – undo
41 41
      */
42 42
     public static function toString($dataType, int $options = null):string
43 43
     {
44
-        if (is_null($dataType)) throw new \Exception(self::class.": No content to convert to Yaml");
44
+        if (is_null($dataType)) {
45
+            throw new \Exception(self::class.": No content to convert to Yaml");
46
+        }
45 47
         self::$options = is_int($options) ? $options : self::OPTIONS;
46 48
         self::$result = new DLL;
47 49
         if ($dataType instanceof YamlObject) {
@@ -83,8 +85,12 @@  discard block
 block discarded – undo
83 85
     private static function dump($dataType, int $indent)
84 86
     {
85 87
         if (is_scalar($dataType)) {
86
-            if ($dataType === INF) return '.inf';
87
-            if ($dataType === -INF) return '-.inf';
88
+            if ($dataType === INF) {
89
+                return '.inf';
90
+            }
91
+            if ($dataType === -INF) {
92
+                return '-.inf';
93
+            }
88 94
             switch (gettype($dataType)) {
89 95
                 case 'boolean': return $dataType ? 'true' : 'false';
90 96
                 case 'float': //fall through
@@ -106,7 +112,9 @@  discard block
 block discarded – undo
106 112
      */
107 113
     private static function dumpYamlObject(YamlObject $obj)
108 114
     {
109
-        if ($obj->hasDocStart() && self::$result instanceof DLL) self::$result->push("---");
115
+        if ($obj->hasDocStart() && self::$result instanceof DLL) {
116
+            self::$result->push("---");
117
+        }
110 118
         // self::dump($obj, 0);
111 119
         if (count($obj) > 0) {
112 120
             self::dumpArray($obj->getArrayCopy(), 0);
@@ -182,9 +190,13 @@  discard block
 block discarded – undo
182 190
                 self::add(self::dump($obj->value, $indent + self::INDENT), $indent + self::INDENT);
183 191
             }
184 192
         }
185
-        if ($obj instanceof Compact) return self::dumpCompact($obj, $indent);
193
+        if ($obj instanceof Compact) {
194
+            return self::dumpCompact($obj, $indent);
195
+        }
186 196
         //TODO:  consider dumping datetime as date strings according to a format provided by user or default
187
-        if ($obj instanceof \DateTime) return $obj->format(self::DATE_FORMAT);
197
+        if ($obj instanceof \DateTime) {
198
+            return $obj->format(self::DATE_FORMAT);
199
+        }
188 200
         $propList = get_object_vars($obj);
189 201
         foreach ($propList as $property => $value) {
190 202
             if (is_scalar($value) || $value instanceof Compact || $value instanceof \DateTime) {
Please login to merge, or discard this patch.
sources/Loader.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -93,7 +93,9 @@  discard block
 block discarded – undo
93 93
     {
94 94
         $source = $this->content ?? preg_split("/\n/m", preg_replace('/(\r\n|\r)/', "\n", $strContent), 0, PREG_SPLIT_DELIM_CAPTURE);
95 95
         //TODO : be more permissive on $strContent values
96
-        if (!is_array($source) || !count($source)) throw new \Exception(self::EXCEPTION_LINE_SPLIT);
96
+        if (!is_array($source) || !count($source)) {
97
+            throw new \Exception(self::EXCEPTION_LINE_SPLIT);
98
+        }
97 99
         foreach ($source as $key => $value) {
98 100
             yield ++$key => $value;
99 101
         }
@@ -116,7 +118,9 @@  discard block
 block discarded – undo
116 118
         try {
117 119
             foreach ($generator as $lineNb => $lineString) {
118 120
                 $node = NodeFactory::get($lineString, $lineNb);
119
-                if ($this->needsSpecialProcess($node, $previous)) continue;
121
+                if ($this->needsSpecialProcess($node, $previous)) {
122
+                    continue;
123
+                }
120 124
                 $this->attachBlankLines($previous);
121 125
                 switch ($node->indent <=> $previous->indent) {
122 126
                     case -1: $target = $previous->getTargetOnLessIndent($node);
Please login to merge, or discard this patch.
sources/NodeList.php 1 patch
Braces   +11 added lines, -5 removed lines patch added patch discarded remove patch
@@ -35,7 +35,9 @@  discard block
 block discarded – undo
35 35
         $tmp->rewind();
36 36
         $fqn = __NAMESPACE__."\\$nodeType";
37 37
         foreach ($tmp as $child) {
38
-            if ($child instanceof $fqn) return true;
38
+            if ($child instanceof $fqn) {
39
+                return true;
40
+            }
39 41
         }
40 42
         return false;
41 43
     }
@@ -47,7 +49,9 @@  discard block
 block discarded – undo
47 49
         foreach ($tmp as $child) {
48 50
             if (!($child instanceof NodeComment)
49 51
                 && !($child instanceof NodeDirective)
50
-                && !($child instanceof NodeDocstart && is_null($child->value)) ) return true;
52
+                && !($child instanceof NodeDocstart && is_null($child->value)) ) {
53
+                return true;
54
+            }
51 55
         }
52 56
         return false;
53 57
     }
@@ -55,9 +59,11 @@  discard block
 block discarded – undo
55 59
     public function push($node)
56 60
     {
57 61
         $type = null;
58
-        if     ($node instanceof NodeItem )    $type = self::SEQUENCE;
59
-        elseif ($node instanceof NodeKey)      $type = self::MAPPING;
60
-        elseif ($node instanceof NodeSetKey
62
+        if     ($node instanceof NodeItem ) {
63
+            $type = self::SEQUENCE;
64
+        } elseif ($node instanceof NodeKey) {
65
+            $type = self::MAPPING;
66
+        } elseif ($node instanceof NodeSetKey
61 67
              || $node instanceof NodeSetValue) {
62 68
             $type = self::SET;
63 69
         } elseif ($node instanceof NodeScalar ){
Please login to merge, or discard this patch.
sources/nodes/NodeSetKey.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,9 @@
 block discarded – undo
31 31
         $built = is_null($this->value) ? null : $this->value->build();
32 32
         $stringKey = is_string($built) && Regex::isProperlyQuoted($built) ? trim($built, '\'" '): $built;
33 33
         $key = json_encode($stringKey, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
34
-        if (empty($key)) throw new \Exception("Cant serialize complex key: ".var_export($this->value, true));
34
+        if (empty($key)) {
35
+            throw new \Exception("Cant serialize complex key: ".var_export($this->value, true));
36
+        }
35 37
         $parent->{trim($key, '\'" ')} = null;
36 38
     }
37 39
 
Please login to merge, or discard this patch.
sources/types/YamlObject.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,8 +74,12 @@
 block discarded – undo
74 74
     {
75 75
         $prop = get_object_vars($this);
76 76
         unset($prop["__yaml__object__api"]);
77
-        if (count($prop) > 0) return $prop;
78
-        if (count($this) > 0) return iterator_to_array($this);
77
+        if (count($prop) > 0) {
78
+            return $prop;
79
+        }
80
+        if (count($this) > 0) {
81
+            return iterator_to_array($this);
82
+        }
79 83
         return $this->__yaml__object__api->value ?? "_Empty YamlObject_";
80 84
     }
81 85
 }
Please login to merge, or discard this patch.
sources/Builder.php 1 patch
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -90,8 +90,12 @@  discard block
 block discarded – undo
90 90
      */
91 91
     public static function getScalar(string $v, bool $onlyScalar = false)
92 92
     {
93
-        if (Regex::isDate($v))   return self::$dateAsObject && !$onlyScalar ? date_create($v) : $v;
94
-        if (Regex::isNumber($v)) return self::getNumber($v);
93
+        if (Regex::isDate($v)) {
94
+            return self::$dateAsObject && !$onlyScalar ? date_create($v) : $v;
95
+        }
96
+        if (Regex::isNumber($v)) {
97
+            return self::getNumber($v);
98
+        }
95 99
         $types = ['yes'   => true,
96 100
                     'no'    => false,
97 101
                     'true'  => true,
@@ -114,8 +118,12 @@  discard block
 block discarded – undo
114 118
      */
115 119
     private static function getNumber(string $v)
116 120
     {
117
-        if (preg_match(Regex::OCTAL_NUM, $v)) return intval(base_convert($v, 8, 10));
118
-        if (preg_match(Regex::HEX_NUM, $v))   return intval(base_convert($v, 16, 10));
121
+        if (preg_match(Regex::OCTAL_NUM, $v)) {
122
+            return intval(base_convert($v, 8, 10));
123
+        }
124
+        if (preg_match(Regex::HEX_NUM, $v)) {
125
+            return intval(base_convert($v, 16, 10));
126
+        }
119 127
         return is_bool(strpos($v, '.')) ? intval($v) : floatval($v);
120 128
     }
121 129
 
Please login to merge, or discard this patch.
sources/nodes/NodeLiterals.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,9 @@
 block discarded – undo
25 25
 
26 26
     public function add(Node $child):Node
27 27
     {
28
-        if (is_null($this->value)) $this->value = new NodeList();
28
+        if (is_null($this->value)) {
29
+            $this->value = new NodeList();
30
+        }
29 31
         $candidate = $child;
30 32
         if (!isOneOf($child, ['NodeScalar', 'NodeBlank', 'NodeComment', 'NodeQuoted'])) {
31 33
             $candidate = new NodeScalar((string) $child->raw, $child->line);
Please login to merge, or discard this patch.
sources/Node.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -210,13 +210,23 @@
 block discarded – undo
210 210
     {
211 211
         $props = [];
212 212
         $props['line->indent'] = "$this->line -> $this->indent";
213
-        if ($this->identifier) $props['identifier'] = "($this->identifier)";
214
-        if ($this->_anchor)    $props['_anchor']    = "($this->_anchor)";
215
-        if ($this->_tag)       $props['_tag']       = "($this->_tag)";
216
-        if ($this->value)      $props['value']       = $this->value;
213
+        if ($this->identifier) {
214
+            $props['identifier'] = "($this->identifier)";
215
+        }
216
+        if ($this->_anchor) {
217
+            $props['_anchor']    = "($this->_anchor)";
218
+        }
219
+        if ($this->_tag) {
220
+            $props['_tag']       = "($this->_tag)";
221
+        }
222
+        if ($this->value) {
223
+            $props['value']       = $this->value;
224
+        }
217 225
         // $props['value'] = $this->value;
218 226
         $props['raw']   = $this->raw;
219
-        if (!$this->_parent)  $props['parent'] = 'NO PARENT!!!';
227
+        if (!$this->_parent) {
228
+            $props['parent'] = 'NO PARENT!!!';
229
+        }
220 230
         return $props;
221 231
     }
222 232
 }
Please login to merge, or discard this patch.
sources/Yaml.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -108,7 +108,9 @@
 block discarded – undo
108 108
 {
109 109
     foreach ($comparison as $className) {
110 110
         $fqn = __NAMESPACE__."\\$className";
111
-        if ($subject instanceof $fqn) return true;
111
+        if ($subject instanceof $fqn) {
112
+            return true;
113
+        }
112 114
     }
113 115
     return false;
114 116
 }
115 117
\ No newline at end of file
Please login to merge, or discard this patch.