Completed
Pull Request — master (#1138)
by Gabriel
11:36
created
src/Serializer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 
115 115
     private function getNavigator(int $direction): GraphNavigatorInterface
116 116
     {
117
-        if (!isset($this->graphNavigators[$direction])) {
117
+        if ( ! isset($this->graphNavigators[$direction])) {
118 118
             throw new RuntimeException(
119 119
                 sprintf(
120 120
                     'Can not find a graph navigator for the direction "%s".',
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
             ? $this->serializationVisitors
133 133
             : $this->deserializationVisitors;
134 134
 
135
-        if (!isset($factories[$format])) {
135
+        if ( ! isset($factories[$format])) {
136 136
             throw new UnsupportedFormatException(
137 137
                 sprintf(
138 138
                     'The format "%s" is not supported for %s.',
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
         $result = $this->visit($navigator, $visitor, $context, $data, 'json', $type);
197 197
         $result = $this->convertArrayObjects($result);
198 198
 
199
-        if (!\is_array($result)) {
199
+        if ( ! \is_array($result)) {
200 200
             throw new RuntimeException(sprintf(
201 201
                 'The input data of type "%s" did not convert to an array, but got a result of type "%s".',
202 202
                 \is_object($data) ? \get_class($data) : \gettype($data),
Please login to merge, or discard this patch.
src/XmlSerializationVisitor.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 
105 105
     public function createRoot(?ClassMetadata $metadata = null, ?string $rootName = null, ?string $rootNamespace = null, ?string $rootPrefix = null): \DOMElement
106 106
     {
107
-        if (null !== $metadata && !empty($metadata->xmlRootName)) {
107
+        if (null !== $metadata && ! empty($metadata->xmlRootName)) {
108 108
             $rootPrefix = $metadata->xmlRootPrefix;
109 109
             $rootName = $metadata->xmlRootName;
110 110
             $rootNamespace = $metadata->xmlRootNamespace ?: $this->getClassDefaultNamespace($metadata);
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 
117 117
         $document = $this->getDocument();
118 118
         if ($rootNamespace) {
119
-            $rootNode = $document->createElementNS($rootNamespace, (null !== $rootPrefix ? ($rootPrefix . ':') : '') . $rootName);
119
+            $rootNode = $document->createElementNS($rootNamespace, (null !== $rootPrefix ? ($rootPrefix.':') : '').$rootName);
120 120
         } else {
121 121
             $rootNode = $document->createElement($rootName);
122 122
         }
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
             $node = $this->navigator->accept($v, $metadata->type);
243 243
             $this->revertCurrentMetadata();
244 244
 
245
-            if (!$node instanceof \DOMCharacterData) {
245
+            if ( ! $node instanceof \DOMCharacterData) {
246 246
                 throw new RuntimeException(sprintf('Unsupported value for XML attribute for %s. Expected character data, but got %s.', $metadata->name, json_encode($v)));
247 247
             }
248 248
 
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
         }
253 253
 
254 254
         if (($metadata->xmlValue && $this->currentNode->childNodes->length > 0)
255
-            || (!$metadata->xmlValue && $this->hasValue)
255
+            || ( ! $metadata->xmlValue && $this->hasValue)
256 256
         ) {
257 257
             throw new RuntimeException(sprintf('If you make use of @XmlValue, all other properties in the class must have the @XmlAttribute annotation. Invalid usage detected in class %s.', $metadata->class));
258 258
         }
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
             $node = $this->navigator->accept($v, $metadata->type);
265 265
             $this->revertCurrentMetadata();
266 266
 
267
-            if (!$node instanceof \DOMCharacterData) {
267
+            if ( ! $node instanceof \DOMCharacterData) {
268 268
                 throw new RuntimeException(sprintf('Unsupported value for property %s::$%s. Expected character data, but got %s.', $metadata->reflection->class, $metadata->reflection->name, \is_object($node) ? \get_class($node) : \gettype($node)));
269 269
             }
270 270
 
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
         }
275 275
 
276 276
         if ($metadata->xmlAttributeMap) {
277
-            if (!\is_array($v)) {
277
+            if ( ! \is_array($v)) {
278 278
                 throw new RuntimeException(sprintf('Unsupported value type for XML attribute map. Expected array but got %s.', \gettype($v)));
279 279
             }
280 280
 
@@ -283,7 +283,7 @@  discard block
 block discarded – undo
283 283
                 $node = $this->navigator->accept($value, null);
284 284
                 $this->revertCurrentMetadata();
285 285
 
286
-                if (!$node instanceof \DOMCharacterData) {
286
+                if ( ! $node instanceof \DOMCharacterData) {
287 287
                     throw new RuntimeException(sprintf('Unsupported value for a XML attribute map value. Expected character data, but got %s.', json_encode($v)));
288 288
                 }
289 289
 
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
             return;
294 294
         }
295 295
 
296
-        if ($addEnclosingElement = !$this->isInLineCollection($metadata) && !$metadata->inline) {
296
+        if ($addEnclosingElement = ! $this->isInLineCollection($metadata) && ! $metadata->inline) {
297 297
             $namespace = null !== $metadata->xmlNamespace
298 298
                 ? $metadata->xmlNamespace
299 299
                 : $this->getClassDefaultNamespace($this->objectMetadataStack->top());
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
337 337
 
338 338
     private function isSkippableEmptyObject(?\DOMElement $node, PropertyMetadata $metadata): bool
339 339
     {
340
-        return null === $node && !$metadata->xmlCollection && $metadata->skipWhenEmpty;
340
+        return null === $node && ! $metadata->xmlCollection && $metadata->skipWhenEmpty;
341 341
     }
342 342
 
343 343
     private function isSkippableCollection(PropertyMetadata $metadata): bool
@@ -347,7 +347,7 @@  discard block
 block discarded – undo
347 347
 
348 348
     private function isElementEmpty(\DOMElement $element): bool
349 349
     {
350
-        return !$element->hasChildNodes() && !$element->hasAttributes();
350
+        return ! $element->hasChildNodes() && ! $element->hasAttributes();
351 351
     }
352 352
 
353 353
     public function endVisitingObject(ClassMetadata $metadata, object $data, array $type): void
@@ -453,7 +453,7 @@  discard block
 block discarded – undo
453 453
         foreach ($metadata->xmlNamespaces as $prefix => $uri) {
454 454
             $attribute = 'xmlns';
455 455
             if ('' !== $prefix) {
456
-                $attribute .= ':' . $prefix;
456
+                $attribute .= ':'.$prefix;
457 457
             } elseif ($element->namespaceURI === $uri) {
458 458
                 continue;
459 459
             }
@@ -478,19 +478,19 @@  discard block
 block discarded – undo
478 478
         if ($this->currentNode->isDefaultNamespace($namespace)) {
479 479
             return $this->document->createElementNS($namespace, $tagName);
480 480
         }
481
-        if (!($prefix = $this->currentNode->lookupPrefix($namespace)) && !($prefix = $this->document->lookupPrefix($namespace))) {
482
-            $prefix = 'ns-' . substr(sha1($namespace), 0, 8);
481
+        if ( ! ($prefix = $this->currentNode->lookupPrefix($namespace)) && ! ($prefix = $this->document->lookupPrefix($namespace))) {
482
+            $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
483 483
         }
484
-        return $this->document->createElementNS($namespace, $prefix . ':' . $tagName);
484
+        return $this->document->createElementNS($namespace, $prefix.':'.$tagName);
485 485
     }
486 486
 
487 487
     private function setAttributeOnNode(\DOMElement $node, string $name, string $value, ?string $namespace = null): void
488 488
     {
489 489
         if (null !== $namespace) {
490
-            if (!$prefix = $node->lookupPrefix($namespace)) {
491
-                $prefix = 'ns-' . substr(sha1($namespace), 0, 8);
490
+            if ( ! $prefix = $node->lookupPrefix($namespace)) {
491
+                $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
492 492
             }
493
-            $node->setAttributeNS($namespace, $prefix . ':' . $name, $value);
493
+            $node->setAttributeNS($namespace, $prefix.':'.$name, $value);
494 494
         } else {
495 495
             $node->setAttribute($name, $value);
496 496
         }
Please login to merge, or discard this patch.