Test Failed
Branch master (44c6e4)
by stéphane
09:11
created
yaml/YamlObject.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,8 +52,12 @@
 block discarded – undo
52 52
     {
53 53
         $prop = get_object_vars($this);
54 54
         unset($prop["__yaml__object__api"]);
55
-        if (count($prop) > 0) return $prop;
56
-        if (count($this) > 0) return iterator_to_array($this);
55
+        if (count($prop) > 0) {
56
+            return $prop;
57
+        }
58
+        if (count($this) > 0) {
59
+            return iterator_to_array($this);
60
+        }
57 61
         return $this->__yaml__object__api->value ?? "Empty YamlObject";
58 62
     }
59 63
 }
Please login to merge, or discard this patch.
yaml/Loader.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -88,7 +88,9 @@  discard block
 block discarded – undo
88 88
     {
89 89
         $source = $this->content ?? preg_split("/\n/m", preg_replace('/(\r\n|\r)/', "\n", $strContent), 0, PREG_SPLIT_DELIM_CAPTURE);
90 90
         //TODO : be more permissive on $strContent values
91
-        if (!is_array($source) || !count($source)) throw new \Exception(self::EXCEPTION_LINE_SPLIT);
91
+        if (!is_array($source) || !count($source)) {
92
+            throw new \Exception(self::EXCEPTION_LINE_SPLIT);
93
+        }
92 94
         $previous = $root = new Node();
93 95
         $emptyLines = [];
94 96
         try { //var_dump($source);
@@ -101,7 +103,9 @@  discard block
 block discarded – undo
101 103
                 $n = new Node($lineString, $lineNb);
102 104
                 $deepest = $previous->getDeepestNode();
103 105
                 if ($n->type & (Y::LITTERALS|Y::BLANK|Y::COMMENT|Y::TAG) || $deepest->type & Y::PARTIAL) {
104
-                    if ($this->onSpecialType($n, $previous, $emptyLines, $lineString)) continue;
106
+                    if ($this->onSpecialType($n, $previous, $emptyLines, $lineString)) {
107
+                        continue;
108
+                    }
105 109
                 }
106 110
                 //Note: 6.6 comments: Note that outside scalar content, a line containing only white space characters is taken to be a comment line.
107 111
                     foreach ($emptyLines as $blankNode) {
@@ -130,7 +134,9 @@  discard block
 block discarded – undo
130 134
                             }
131 135
                         }
132 136
                 }
133
-                if ($this->onContextType($n, $target, $lineString)) continue;
137
+                if ($this->onContextType($n, $target, $lineString)) {
138
+                    continue;
139
+                }
134 140
                 $target->add($n);
135 141
                 $previous = $n;
136 142
             }
@@ -167,8 +173,12 @@  discard block
 block discarded – undo
167 173
             return true;
168 174
         }
169 175
         if ($n->type & Y::BLANK) {
170
-            if ($previous->type & Y::SCALAR)   $emptyLines[] = $n->setParent($previous->getParent());
171
-            if ($deepest->type & Y::LITTERALS) $emptyLines[] = $n->setParent($deepest);
176
+            if ($previous->type & Y::SCALAR) {
177
+                $emptyLines[] = $n->setParent($previous->getParent());
178
+            }
179
+            if ($deepest->type & Y::LITTERALS) {
180
+                $emptyLines[] = $n->setParent($deepest);
181
+            }
172 182
             return true;
173 183
         } elseif ($n->type & Y::COMMENT
174 184
                   && !($previous->getParent()->value->type & Y::LITTERALS)
Please login to merge, or discard this patch.
yaml/Node.php 1 patch
Braces   +16 added lines, -6 removed lines patch added patch discarded remove patch
@@ -68,7 +68,9 @@  discard block
 block discarded – undo
68 68
         if ($this->type === Y::ROOT) {
69 69
             return $this;
70 70
         }
71
-        if (!is_int($indent)) return $this->parent ?? $this;
71
+        if (!is_int($indent)) {
72
+            return $this->parent ?? $this;
73
+        }
72 74
         $cursor = $this;
73 75
         while ($cursor instanceof Node && $cursor->indent >= $indent) {
74 76
             if ($cursor->indent === $indent && $cursor->type !== $type) {
@@ -99,7 +101,7 @@  discard block
 block discarded – undo
99 101
         if (is_null($current)) {
100 102
             $this->value = $child;
101 103
             return;
102
-        }elseif ($current instanceof Node) {
104
+        } elseif ($current instanceof Node) {
103 105
             $this->value = new NodeList();
104 106
             if ($current->type & Y::LITTERALS) {
105 107
                 $this->value->type = $current->type;
@@ -109,13 +111,21 @@  discard block
 block discarded – undo
109 111
         }
110 112
         //modify type according to child
111 113
         if (is_null($this->value->type)) {
112
-            if ($child->type & Y::SET_KEY)   $this->value->type = Y::SET;
113
-            if ($child->type & Y::KEY)       $this->value->type = Y::MAPPING;
114
-            if ($child->type & Y::ITEM)      $this->value->type = Y::SEQUENCE;
114
+            if ($child->type & Y::SET_KEY) {
115
+                $this->value->type = Y::SET;
116
+            }
117
+            if ($child->type & Y::KEY) {
118
+                $this->value->type = Y::MAPPING;
119
+            }
120
+            if ($child->type & Y::ITEM) {
121
+                $this->value->type = Y::SEQUENCE;
122
+            }
115 123
         }
116 124
         $this->value->push($child);
117 125
 
118
-        if ($this->type & Y::LITTERALS)  $this->value->type = $this->type;
126
+        if ($this->type & Y::LITTERALS) {
127
+            $this->value->type = $this->type;
128
+        }
119 129
     }
120 130
 
121 131
     /**
Please login to merge, or discard this patch.
yaml/Dumper.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -40,7 +40,9 @@  discard block
 block discarded – undo
40 40
      */
41 41
     public static function toString($dataType, int $options = null):string
42 42
     {
43
-        if (is_null($dataType)) throw new \Exception(self::class.": No content to convert to Yaml", 1);
43
+        if (is_null($dataType)) {
44
+            throw new \Exception(self::class.": No content to convert to Yaml", 1);
45
+        }
44 46
         self::$options = is_int($options) ? $options : self::OPTIONS;
45 47
         self::$result = new DLL;
46 48
         if ($dataType instanceof YamlObject) {
@@ -76,9 +78,13 @@  discard block
 block discarded – undo
76 78
         if (is_scalar($dataType)) {
77 79
             switch (gettype($dataType)) {
78 80
                 case 'boolean': return $dataType ? 'true' : 'false';
79
-                case 'float': if (is_infinite($dataType)) return $dataType > 0 ? '.inf' : '-.inf';
81
+                case 'float': if (is_infinite($dataType)) {
82
+                    return $dataType > 0 ? '.inf' : '-.inf';
83
+                }
80 84
                               return sprintf('%.2F', $dataType);
81
-                case 'double': if (is_nan((float) $dataType)) return '.nan';
85
+                case 'double': if (is_nan((float) $dataType)) {
86
+                    return '.nan';
87
+                }
82 88
                 default:
83 89
                     return $dataType;
84 90
             }
@@ -91,7 +97,9 @@  discard block
 block discarded – undo
91 97
 
92 98
     private static function dumpYamlObject(YamlObject $dataType)
93 99
     {
94
-        if ($dataType->hasDocStart() && self::$result instanceof DLL) self::$result->push("---");
100
+        if ($dataType->hasDocStart() && self::$result instanceof DLL) {
101
+            self::$result->push("---");
102
+        }
95 103
         // self::dump($dataType, 0);
96 104
         if (count($dataType) > 0) {
97 105
             self::dumpSequence($dataType->getArrayCopy(), 0);
@@ -142,9 +150,13 @@  discard block
 block discarded – undo
142 150
                 self::add(self::dump($obj->value, $indent + self::INDENT), $indent + self::INDENT);
143 151
             }
144 152
         }
145
-        if ($obj instanceof Compact) return self::dumpCompact($obj, $indent);
153
+        if ($obj instanceof Compact) {
154
+            return self::dumpCompact($obj, $indent);
155
+        }
146 156
         //TODO:  consider dumping datetime as date strings according to a format provided by user or default
147
-        if ($obj instanceof \DateTime) return $obj->format('Y-m-d');
157
+        if ($obj instanceof \DateTime) {
158
+            return $obj->format('Y-m-d');
159
+        }
148 160
         // if ($obj instanceof \SplString) {var_dump('splstrin',$obj);return '"'.$obj.'"';}
149 161
         $propList = get_object_vars($obj);
150 162
         foreach ($propList as $property => $value) {
Please login to merge, or discard this patch.
yaml/Node2PHP.php 1 patch
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -23,8 +23,12 @@  discard block
 block discarded – undo
23 23
      */
24 24
     public static function get(Node $n)
25 25
     {
26
-        if (is_null($n->value)) return null;
27
-        if ($n->type & (Y::REF_CALL|Y::SCALAR)) return self::getScalar((string) $n->value);
26
+        if (is_null($n->value)) {
27
+            return null;
28
+        }
29
+        if ($n->type & (Y::REF_CALL|Y::SCALAR)) {
30
+            return self::getScalar((string) $n->value);
31
+        }
28 32
         if ($n->type & Y::JSON) {
29 33
             return $n->value;
30 34
         }
@@ -43,8 +47,12 @@  discard block
 block discarded – undo
43 47
      */
44 48
     private static function getScalar(string $v)
45 49
     {
46
-        if (R::isDate($v))   return date_create($v);
47
-        if (R::isNumber($v)) return self::getNumber($v);
50
+        if (R::isDate($v)) {
51
+            return date_create($v);
52
+        }
53
+        if (R::isNumber($v)) {
54
+            return self::getNumber($v);
55
+        }
48 56
         $types = ['yes'   => true,
49 57
                     'no'    => false,
50 58
                     'true'  => true,
@@ -66,8 +74,12 @@  discard block
 block discarded – undo
66 74
      */
67 75
     private static function getNumber(string $v)
68 76
     {
69
-        if (preg_match("/^(0o\d+)$/i", $v))      return intval(base_convert($v, 8, 10));
70
-        if (preg_match("/^(0x[\da-f]+)$/i", $v)) return intval(base_convert($v, 16, 10));
77
+        if (preg_match("/^(0o\d+)$/i", $v)) {
78
+            return intval(base_convert($v, 8, 10));
79
+        }
80
+        if (preg_match("/^(0x[\da-f]+)$/i", $v)) {
81
+            return intval(base_convert($v, 16, 10));
82
+        }
71 83
         return is_bool(strpos($v, '.')) ? intval($v) : floatval($v);
72 84
     }
73 85
 }
74 86
\ No newline at end of file
Please login to merge, or discard this patch.
yaml/Compact.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,8 +33,12 @@
 block discarded – undo
33 33
     public function jsonSerialize():array
34 34
     {
35 35
         $prop = get_object_vars($this);
36
-        if (count($prop) > 0) return $prop;
37
-        if (count($this) > 0) return iterator_to_array($this);
36
+        if (count($prop) > 0) {
37
+            return $prop;
38
+        }
39
+        if (count($this) > 0) {
40
+            return iterator_to_array($this);
41
+        }
38 42
         return new \Stdclass;
39 43
     }
40 44
 
Please login to merge, or discard this patch.
yaml/Builder.php 1 patch
Braces   +27 added lines, -11 removed lines patch added patch discarded remove patch
@@ -42,10 +42,14 @@  discard block
 block discarded – undo
42 42
         }
43 43
         $_root->value instanceof NodeList && $_root->value->setIteratorMode(NodeList::IT_MODE_DELETE);
44 44
         foreach ($_root->value as $child) {
45
-            if ($child->type & Y::DOC_START) $totalDocStart++;
45
+            if ($child->type & Y::DOC_START) {
46
+                $totalDocStart++;
47
+            }
46 48
             //if 0 or 1 DOC_START = we are still in first document
47 49
             $currentDoc = $totalDocStart > 1 ? $totalDocStart - 1 : 0;
48
-            if (!isset($documents[$currentDoc])) $documents[$currentDoc] = new NodeList();
50
+            if (!isset($documents[$currentDoc])) {
51
+                $documents[$currentDoc] = new NodeList();
52
+            }
49 53
             $documents[$currentDoc]->push($child);
50 54
         }
51 55
         $content = [];
@@ -70,7 +74,9 @@  discard block
 block discarded – undo
70 74
      */
71 75
     private static function build(object $node, &$parent = null)
72 76
     {
73
-        if ($node instanceof NodeList) return self::buildNodeList($node, $parent);
77
+        if ($node instanceof NodeList) {
78
+            return self::buildNodeList($node, $parent);
79
+        }
74 80
         return self::buildNode($node, $parent);
75 81
     }
76 82
 
@@ -133,13 +139,17 @@  discard block
 block discarded – undo
133 139
             } else {
134 140
                 $tmp = Node2PHP::get($node);
135 141
             }
136
-            if ($type === Y::REF_DEF) self::$_root->addReference($identifier, $tmp);
142
+            if ($type === Y::REF_DEF) {
143
+                self::$_root->addReference($identifier, $tmp);
144
+            }
137 145
             return self::$_root->getReference($identifier);
138 146
         }
139 147
         if ($type & (Y::COMPACT_MAPPING|Y::COMPACT_SEQUENCE)) {
140 148
             return self::buildNodeList($node->value, $parent);
141 149
         }
142
-        if ($type & Y::COMMENT) self::$_root->addComment($node->line, $node->value);
150
+        if ($type & Y::COMMENT) {
151
+            self::$_root->addComment($node->line, $node->value);
152
+        }
143 153
         $typesActions = [Y::DIRECTIVE => 'buildDirective',
144 154
                          Y::ITEM      => 'buildItem',
145 155
                          Y::KEY       => 'buildKey',
@@ -254,18 +264,24 @@  discard block
 block discarded – undo
254 264
         }
255 265
         $result = '';
256 266
         $separator = '';
257
-        if ($type & Y::LITT)         $separator = "\n";
258
-        if ($type & Y::LITT_FOLDED)  $separator = ' ';
267
+        if ($type & Y::LITT) {
268
+            $separator = "\n";
269
+        }
270
+        if ($type & Y::LITT_FOLDED) {
271
+            $separator = ' ';
272
+        }
259 273
         foreach ($list as $child) {
260 274
             if ($child->value instanceof NodeList) {
261 275
                 $result .= self::buildLitteral($child->value, $type).$separator;
262 276
             } else {
263 277
                 $val = $child->type & (Y::SCALAR|Y::BLANK) ? $child->value : substr($child->raw, $refIndent);
264 278
                 if ($type & Y::LITT_FOLDED && ($child->indent > $refIndent || ($child->type & Y::BLANK))) {
265
-                    if ($result[-1] === $separator)
266
-                        $result[-1] = "\n";
267
-                    if ($result[-1] === "\n")
268
-                        $result .= $val;
279
+                    if ($result[-1] === $separator) {
280
+                                            $result[-1] = "\n";
281
+                    }
282
+                    if ($result[-1] === "\n") {
283
+                                            $result .= $val;
284
+                    }
269 285
                     continue;
270 286
                 }
271 287
                 $result .= $val.$separator;
Please login to merge, or discard this patch.