Completed
Pull Request — master (#672)
by Davidson
05:03
created
src/JMS/Serializer/XmlDeserializationVisitor.php 1 patch
Spacing   +7 added lines, -8 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,12 +147,11 @@  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 153
         $hasNode = null !== $namespace ? 
154
-                !empty($data) && !empty($data->children($namespace)) && isset($data->children($namespace)->$entryName) : 
155
-                !empty($data) && isset($data->$entryName);
154
+                ! empty($data) && ! empty($data->children($namespace)) && isset($data->children($namespace)->$entryName) : ! empty($data) && isset($data->$entryName);
156 155
         if (false === $hasNode) {
157 156
             if (null === $this->result) {
158 157
                 return $this->result = array();
@@ -245,7 +244,7 @@  discard block
 block discarded – undo
245 244
 
246 245
         if ($metadata->xmlCollection) {
247 246
             $enclosingElem = $data;
248
-            if (!$metadata->xmlCollectionInline) {
247
+            if ( ! $metadata->xmlCollectionInline) {
249 248
                 $enclosingElem = $data->children($metadata->xmlNamespace)->$name;
250 249
             }
251 250
 
@@ -259,7 +258,7 @@  discard block
 block discarded – undo
259 258
 
260 259
         if ($metadata->xmlNamespace) {
261 260
             $node = $data->children($metadata->xmlNamespace)->$name;
262
-            if (!$node->count()) {
261
+            if ( ! $node->count()) {
263 262
                 return;
264 263
             }
265 264
         } else {
@@ -269,9 +268,9 @@  discard block
 block discarded – undo
269 268
             if (isset($namespaces[''])) {
270 269
                 $prefix = uniqid('ns-');
271 270
                 $data->registerXPathNamespace($prefix, $namespaces['']);
272
-                $nodes = $data->xpath('./'.$prefix. ':'.$name );
271
+                $nodes = $data->xpath('./'.$prefix.':'.$name);
273 272
             } else {
274
-                $nodes = $data->xpath('./'. $name );
273
+                $nodes = $data->xpath('./'.$name);
275 274
             }
276 275
             if (empty($nodes)) {
277 276
                 return;
Please login to merge, or discard this patch.