Completed
Pull Request — master (#640)
by Ayrton
07:37
created
src/JMS/Serializer/EventDispatcher/EventDispatcher.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@
 block discarded – undo
80 80
             $this->classListeners[$eventName][$loweredClass][$format] = $this->initializeListeners($eventName, $loweredClass, $format);
81 81
         }
82 82
 
83
-        return !!$this->classListeners[$eventName][$loweredClass][$format];
83
+        return ! ! $this->classListeners[$eventName][$loweredClass][$format];
84 84
     }
85 85
 
86 86
     public function dispatch($eventName, $class, $format, Event $event)
Please login to merge, or discard this patch.
src/JMS/Serializer/Util/Writer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -83,16 +83,16 @@
 block discarded – undo
83 83
         $addition = '';
84 84
 
85 85
         $lines = explode("\n", $content);
86
-        for ($i=0,$c=count($lines); $i<$c; $i++) {
86
+        for ($i = 0, $c = count($lines); $i < $c; $i++) {
87 87
             if ($this->indentationLevel > 0
88
-                && !empty($lines[$i])
88
+                && ! empty($lines[$i])
89 89
                 && ((empty($addition) && "\n" === substr($this->content, -1)) || "\n" === substr($addition, -1))) {
90 90
                 $addition .= str_repeat(' ', $this->indentationLevel * $this->indentationSpaces);
91 91
             }
92 92
 
93 93
             $addition .= $lines[$i];
94 94
 
95
-            if ($i+1 < $c) {
95
+            if ($i + 1 < $c) {
96 96
                 $addition .= "\n";
97 97
             }
98 98
         }
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/PropertyMetadata.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -166,10 +166,10 @@
 block discarded – undo
166 166
             $parentStr
167 167
         ) = $unserialized;
168 168
 
169
-        if (isset($unserialized['xmlEntryNamespace'])){
169
+        if (isset($unserialized['xmlEntryNamespace'])) {
170 170
             $this->xmlEntryNamespace = $unserialized['xmlEntryNamespace'];
171 171
         }
172
-        if (isset($unserialized['xmlCollectionSkipWhenEmpty'])){
172
+        if (isset($unserialized['xmlCollectionSkipWhenEmpty'])) {
173 173
             $this->xmlCollectionSkipWhenEmpty = $unserialized['xmlCollectionSkipWhenEmpty'];
174 174
         }
175 175
         
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/Driver/YamlDriver.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -113,15 +113,15 @@
 block discarded – undo
113 113
 
114 114
                         $colConfig = $pConfig['xml_list'];
115 115
                         if (isset($colConfig['inline'])) {
116
-                            $pMetadata->xmlCollectionInline = (Boolean)$colConfig['inline'];
116
+                            $pMetadata->xmlCollectionInline = (Boolean) $colConfig['inline'];
117 117
                         }
118 118
 
119 119
                         if (isset($colConfig['entry_name'])) {
120
-                            $pMetadata->xmlEntryName = (string)$colConfig['entry_name'];
120
+                            $pMetadata->xmlEntryName = (string) $colConfig['entry_name'];
121 121
                         }
122 122
 
123 123
                         if (isset($colConfig['skip_when_empty'])) {
124
-                            $pMetadata->xmlCollectionSkipWhenEmpty = (Boolean)$colConfig['skip_when_empty'];
124
+                            $pMetadata->xmlCollectionSkipWhenEmpty = (Boolean) $colConfig['skip_when_empty'];
125 125
                         } else {
126 126
                             $pMetadata->xmlCollectionSkipWhenEmpty = true;
127 127
                         }
Please login to merge, or discard this patch.
src/JMS/Serializer/XmlDeserializationVisitor.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
         if (false !== stripos($data, '<!doctype')) {
65 65
             $internalSubset = $this->getDomDocumentTypeEntitySubset($data);
66
-            if (!in_array($internalSubset, $this->doctypeWhitelist, true)) {
66
+            if ( ! in_array($internalSubset, $this->doctypeWhitelist, true)) {
67 67
                 throw new InvalidArgumentException(sprintf(
68 68
                     'The document type "%s" is not allowed. If it is safe, you may add it to the whitelist configuration.',
69 69
                     $internalSubset
@@ -147,10 +147,10 @@  discard block
 block discarded – undo
147 147
 
148 148
         if ($namespace === null && $this->objectMetadataStack->count()) {
149 149
             $classMetadata = $this->objectMetadataStack->top();
150
-            $namespace = isset($classMetadata->xmlNamespaces[''])?$classMetadata->xmlNamespaces['']:$namespace;
150
+            $namespace = isset($classMetadata->xmlNamespaces['']) ? $classMetadata->xmlNamespaces[''] : $namespace;
151 151
         }
152 152
 
153
-        if ( ! isset($data->$entryName) ) {
153
+        if ( ! isset($data->$entryName)) {
154 154
             if (null === $this->result) {
155 155
                 return $this->result = array();
156 156
             }
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
 
243 243
         if ($metadata->xmlCollection) {
244 244
             $enclosingElem = $data;
245
-            if (!$metadata->xmlCollectionInline) {
245
+            if ( ! $metadata->xmlCollectionInline) {
246 246
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
247 247
             }
248 248
 
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
 
257 257
         if ($metadata->xmlNamespace) {
258 258
             $node = $data->children($metadata->xmlNamespace)->$name;
259
-            if (!$node->count()) {
259
+            if ( ! $node->count()) {
260 260
                 return;
261 261
             }
262 262
         } else {
@@ -266,9 +266,9 @@  discard block
 block discarded – undo
266 266
             if (isset($namespaces[''])) {
267 267
                 $prefix = uniqid('ns-');
268 268
                 $data->registerXPathNamespace($prefix, $namespaces['']);
269
-                $nodes = $data->xpath('./'.$prefix. ':'.$name );
269
+                $nodes = $data->xpath('./'.$prefix.':'.$name);
270 270
             } else {
271
-                $nodes = $data->xpath('./'. $name );
271
+                $nodes = $data->xpath('./'.$name);
272 272
             }
273 273
             if (empty($nodes)) {
274 274
                 return;
Please login to merge, or discard this patch.
src/JMS/Serializer/XmlSerializationVisitor.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
             $this->document = $this->createDocument(null, null, false);
198 198
             if ($metadata->xmlRootName) {
199 199
                 $rootName = $metadata->xmlRootName;
200
-                $rootNamespace = $metadata->xmlRootNamespace?:$this->getClassDefaultNamespace($metadata);
200
+                $rootNamespace = $metadata->xmlRootNamespace ?: $this->getClassDefaultNamespace($metadata);
201 201
             } else {
202 202
                 $rootName = $this->defaultRootName;
203 203
                 $rootNamespace = $this->defaultRootNamespace;
@@ -303,7 +303,7 @@  discard block
 block discarded – undo
303 303
         if ($addEnclosingElement) {
304 304
             $this->revertCurrentNode();
305 305
 
306
-            if ($this->nodeNotEmpty($element) || ((!$metadata->xmlCollection || !$metadata->xmlCollectionSkipWhenEmpty) && $node === null && $v !== null && !$context->isVisiting($v))) {
306
+            if ($this->nodeNotEmpty($element) || (( ! $metadata->xmlCollection || ! $metadata->xmlCollectionSkipWhenEmpty) && $node === null && $v !== null && ! $context->isVisiting($v))) {
307 307
                 $this->currentNode->appendChild($element);
308 308
             }
309 309
         }
@@ -449,18 +449,18 @@  discard block
 block discarded – undo
449 449
         if ($this->currentNode->isDefaultNamespace($namespace)) {
450 450
             return $this->document->createElementNS($namespace, $tagName);
451 451
         }
452
-        if (!($prefix = $this->currentNode->lookupPrefix($namespace)) && !($prefix = $this->document->lookupPrefix($namespace))) {
453
-            $prefix = 'ns-'.  substr(sha1($namespace), 0, 8);
454
-            return $this->document->createElementNS($namespace, $prefix . ':' . $tagName);
452
+        if ( ! ($prefix = $this->currentNode->lookupPrefix($namespace)) && ! ($prefix = $this->document->lookupPrefix($namespace))) {
453
+            $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
454
+            return $this->document->createElementNS($namespace, $prefix.':'.$tagName);
455 455
         }
456
-        return $this->document->createElement($prefix . ':' . $tagName);
456
+        return $this->document->createElement($prefix.':'.$tagName);
457 457
     }
458 458
 
459 459
     private function setAttributeOnNode(\DOMElement $node, $name, $value, $namespace = null)
460 460
     {
461 461
         if (null !== $namespace) {
462
-            if (!$prefix = $node->lookupPrefix($namespace)) {
463
-                $prefix = 'ns-'.  substr(sha1($namespace), 0, 8);
462
+            if ( ! $prefix = $node->lookupPrefix($namespace)) {
463
+                $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
464 464
             }
465 465
             $node->setAttributeNS($namespace, $prefix.':'.$name, $value);
466 466
         } else {
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
 
471 471
     private function getClassDefaultNamespace(ClassMetadata $metadata)
472 472
     {
473
-        return (isset($metadata->xmlNamespaces[''])?$metadata->xmlNamespaces['']:null);
473
+        return (isset($metadata->xmlNamespaces['']) ? $metadata->xmlNamespaces[''] : null);
474 474
     }
475 475
 
476 476
 }
Please login to merge, or discard this patch.