Test Failed
Push — master ( f07b2d...569899 )
by stéphane
03:01
created
yaml/Node.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
         elseif (in_array($first, ['?', ':']))      $this->onSetElement($nodeValue);
186 186
         elseif (in_array($first, ['!', '&', '*'])) $this->onNodeAction($nodeValue);
187 187
         else {
188
-            $characters = [ '#' =>  [Y::COMMENT, $v],
188
+            $characters = ['#' =>  [Y::COMMENT, $v],
189 189
                             '%' =>  [Y::DIRECTIVE, $v],
190 190
                             '>' =>  [Y::LITT_FOLDED, null],
191 191
                             '|' =>  [Y::LITT, null]
@@ -254,28 +254,28 @@  discard block
 block discarded – undo
254 254
     private function onCompact($value)
255 255
     {
256 256
         $this->value = json_decode($value, false, 512, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
257
-        if (json_last_error() === JSON_ERROR_NONE){
257
+        if (json_last_error() === JSON_ERROR_NONE) {
258 258
             $this->type = Y::JSON;
259 259
             return;
260 260
         }
261 261
         $this->value = new NodeList();
262
-        if (preg_match(R::MAPPING, $value)){
262
+        if (preg_match(R::MAPPING, $value)) {
263 263
             $this->type = Y::COMPACT_MAPPING;
264 264
             $this->value->type = Y::COMPACT_MAPPING;
265
-            preg_match_all(R::MAPPING_VALUES, trim(substr($value, 1,-1)), $matches);
265
+            preg_match_all(R::MAPPING_VALUES, trim(substr($value, 1, -1)), $matches);
266 266
             foreach ($matches['k'] as $index => $property) {
267 267
                 $n = new Node('', $this->line);
268 268
                 $n->type = Y::KEY;
269
-                $n->identifier = trim($property, '"\' ');//TODO : maybe check for proper quoting first ?
269
+                $n->identifier = trim($property, '"\' '); //TODO : maybe check for proper quoting first ?
270 270
                 $n->value = new Node($matches['v'][$index], $this->line);
271 271
                 $this->value->push($n);
272 272
             }
273 273
             return;
274 274
         }
275
-        if (preg_match(R::SEQUENCE, $value)){
275
+        if (preg_match(R::SEQUENCE, $value)) {
276 276
             $this->type = Y::COMPACT_SEQUENCE;
277 277
             $this->value->type = Y::COMPACT_SEQUENCE;
278
-            $count = preg_match_all(R::SEQUENCE_VALUES, trim(substr($value, 1,-1)), $matches);
278
+            $count = preg_match_all(R::SEQUENCE_VALUES, trim(substr($value, 1, -1)), $matches);
279 279
             foreach ($matches['item'] as $key => $item) {
280 280
                 $i = new Node('', $this->line);
281 281
                 $i->type = Y::ITEM;
Please login to merge, or discard this patch.
yaml/Builder.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -75,13 +75,13 @@  discard block
 block discarded – undo
75 75
      *
76 76
      * @return mixed    The parent (object|array) or a string representing the NodeList.
77 77
      */
78
-    private static function buildNodeList(NodeList $node, &$parent=null)
78
+    private static function buildNodeList(NodeList $node, &$parent = null)
79 79
     {
80 80
         $node->forceType();
81
-        if ($node->type & (Y::RAW | Y::LITTERALS)) {
81
+        if ($node->type & (Y::RAW|Y::LITTERALS)) {
82 82
             return self::buildLitteral($node, (int) $node->type);
83 83
         }
84
-        $action = function ($child, &$parent, &$out) {
84
+        $action = function($child, &$parent, &$out) {
85 85
             self::build($child, $out);
86 86
         };
87 87
         if ($node->type & (Y::COMPACT_MAPPING|Y::MAPPING|Y::SET)) {
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
             $out = $parent ?? [];
91 91
         } else {
92 92
             $out = '';
93
-            $action = function ($child, &$parent, &$out) {
93
+            $action = function($child, &$parent, &$out) {
94 94
                 if ($child->type & (Y::SCALAR|Y::QUOTED)) {
95 95
                     if ($parent) {
96 96
                         $parent->setText(self::build($child));
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     private static function buildNode(Node $node, &$parent)
121 121
     {
122 122
         extract((array) $node, EXTR_REFS);
123
-        if ($type & (Y::REF_DEF | Y::REF_CALL)) {
123
+        if ($type&(Y::REF_DEF|Y::REF_CALL)) {
124 124
             if (is_object($value)) {
125 125
                 $tmp = self::build($value, $parent) ?? $parent;
126 126
             } else {
@@ -129,10 +129,10 @@  discard block
 block discarded – undo
129 129
             if ($type === Y::REF_DEF) self::$_root->addReference($identifier, $tmp);
130 130
             return self::$_root->getReference($identifier);
131 131
         }
132
-        if ($type & (Y::COMPACT_MAPPING|Y::COMPACT_SEQUENCE)) {
132
+        if ($type&(Y::COMPACT_MAPPING|Y::COMPACT_SEQUENCE)) {
133 133
             return self::buildNodeList($node->value, $parent);
134 134
         }
135
-        if ($type & Y::COMMENT) self::$_root->addComment($node->line, $node->value);
135
+        if ($type&Y::COMMENT) self::$_root->addComment($node->line, $node->value);
136 136
         $typesActions = [Y::DIRECTIVE => 'buildDirective',
137 137
                          Y::ITEM      => 'buildItem',
138 138
                          Y::KEY       => 'buildKey',
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
      * @throws \ParseError if Key has no name(identifier) Note: empty string is allowed
156 156
      * @return null
157 157
      */
158
-    private static function buildKey(Node $node, &$parent=null)
158
+    private static function buildKey(Node $node, &$parent = null)
159 159
     {
160 160
         extract((array) $node, EXTR_REFS);
161 161
         if (is_null($identifier)) {
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
         $numKeys = array_filter(array_keys($ref), 'is_int');
203 203
         $key = count($numKeys) > 0 ? max($numKeys) + 1 : 0;
204 204
         if ($value instanceof Node) {
205
-            if($value->type & Y::KEY) {
205
+            if ($value->type & Y::KEY) {
206 206
                 self::buildKey($node->value, $parent);
207 207
                 return;
208 208
             } elseif ($value->type & Y::ITEM) {
@@ -231,13 +231,13 @@  discard block
 block discarded – undo
231 231
         //remove trailing blank
232 232
         while ($list->top()->type & Y::BLANK) $list->pop();
233 233
         $result = '';
234
-        $separator = [ 0 => '', Y::LITT => "\n", Y::LITT_FOLDED => ' '][(int) $type];
234
+        $separator = [0 => '', Y::LITT => "\n", Y::LITT_FOLDED => ' '][(int) $type];
235 235
         foreach ($list as $child) {
236 236
             if ($child->value instanceof NodeList) {
237 237
                 $result .= self::buildLitteral($child->value, $type).$separator;
238 238
             } else {
239 239
                 $val = $child->type & (Y::SCALAR|Y::BLANK) ? $child->value : substr($child->raw, $refIndent);
240
-                if ($type & Y::LITT_FOLDED && ($child->indent > $refIndent || ($child->type & Y::BLANK))) {
240
+                if ($type&Y::LITT_FOLDED && ($child->indent > $refIndent || ($child->type & Y::BLANK))) {
241 241
                     if ($result[-1] === $separator)
242 242
                         $result[-1] = "\n";
243 243
                     if ($result[-1] === "\n")
@@ -261,7 +261,7 @@  discard block
 block discarded – undo
261 261
     private function buildSetKey(Node $node, &$parent)
262 262
     {
263 263
         $built = is_object($node->value) ? self::build($node->value) : null;
264
-        $stringKey = is_string($built) && Regex::isProperlyQuoted($built) ? trim($built, '\'" '): $built;
264
+        $stringKey = is_string($built) && Regex::isProperlyQuoted($built) ? trim($built, '\'" ') : $built;
265 265
         $key = json_encode($stringKey, JSON_PARTIAL_OUTPUT_ON_ERROR|JSON_UNESCAPED_SLASHES);
266 266
         if (empty($key)) throw new \Exception("Cant serialize complex key: ".var_export($node->value, true), 1);
267 267
         $parent->{trim($key, '\'" ')} = null;
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
     {
278 278
         $prop = array_keys(get_object_vars($parent));
279 279
         $key = end($prop);
280
-        if ($node->value->type & (Y::ITEM|Y::KEY )) {
280
+        if ($node->value->type & (Y::ITEM|Y::KEY)) {
281 281
             $node->value = new NodeList($node->value);
282 282
         }
283 283
         $parent->{$key} = self::build($node->value);
Please login to merge, or discard this patch.