Completed
Push — master ( 37cc39...3535a4 )
by Johannes
16s queued 12s
created
src/JMS/Serializer/XmlDeserializationVisitor.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -346,7 +346,6 @@
 block discarded – undo
346 346
     /**
347 347
      * Retrieves internalSubset even in bugfixed php versions
348 348
      *
349
-     * @param \DOMDocumentType $child
350 349
      * @param string $data
351 350
      * @return string
352 351
      */
Please login to merge, or discard this 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,13 +266,13 @@  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
                 if (empty($nodes)) {
271 271
                     return;
272 272
                 }
273 273
                 $node = reset($nodes);
274 274
             } else {
275
-                if (!isset($data->$name)) {
275
+                if ( ! isset($data->$name)) {
276 276
                     return;
277 277
                 }
278 278
                 $node = $data->$name;
Please login to merge, or discard this patch.