Completed
Pull Request — master (#886)
by Florent
10:43
created
src/JMS/Serializer/Handler/ConstraintViolationHandler.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
                     'direction' => GraphNavigator::DIRECTION_SERIALIZATION,
41 41
                     'type' => $type,
42 42
                     'format' => $format,
43
-                    'method' => $method . 'To' . $format,
43
+                    'method' => $method.'To'.$format,
44 44
                 );
45 45
             }
46 46
         }
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/Driver/XmlDriver.php 1 patch
Spacing   +60 added lines, -60 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
         }
45 45
 
46 46
         $metadata = new ClassMetadata($name = $class->name);
47
-        if (!$elems = $elem->xpath("./class[@name = '" . $name . "']")) {
47
+        if ( ! $elems = $elem->xpath("./class[@name = '".$name."']")) {
48 48
             throw new RuntimeException(sprintf('Could not find class %s inside XML element.', $name));
49 49
         }
50 50
         $elem = reset($elems);
@@ -53,88 +53,88 @@  discard block
 block discarded – undo
53 53
         $metadata->fileResources[] = $class->getFileName();
54 54
         $exclusionPolicy = strtoupper($elem->attributes()->{'exclusion-policy'}) ?: 'NONE';
55 55
         $excludeAll = null !== ($exclude = $elem->attributes()->exclude) ? 'true' === strtolower($exclude) : false;
56
-        $classAccessType = (string)($elem->attributes()->{'access-type'} ?: PropertyMetadata::ACCESS_TYPE_PROPERTY);
56
+        $classAccessType = (string) ($elem->attributes()->{'access-type'} ?: PropertyMetadata::ACCESS_TYPE_PROPERTY);
57 57
 
58 58
         $propertiesMetadata = array();
59 59
         $propertiesNodes = array();
60 60
 
61 61
         if (null !== $accessorOrder = $elem->attributes()->{'accessor-order'}) {
62
-            $metadata->setAccessorOrder((string)$accessorOrder, preg_split('/\s*,\s*/', (string)$elem->attributes()->{'custom-accessor-order'}));
62
+            $metadata->setAccessorOrder((string) $accessorOrder, preg_split('/\s*,\s*/', (string) $elem->attributes()->{'custom-accessor-order'}));
63 63
         }
64 64
 
65 65
         if (null !== $xmlRootName = $elem->attributes()->{'xml-root-name'}) {
66
-            $metadata->xmlRootName = (string)$xmlRootName;
66
+            $metadata->xmlRootName = (string) $xmlRootName;
67 67
         }
68 68
 
69 69
         if (null !== $xmlRootNamespace = $elem->attributes()->{'xml-root-namespace'}) {
70
-            $metadata->xmlRootNamespace = (string)$xmlRootNamespace;
70
+            $metadata->xmlRootNamespace = (string) $xmlRootNamespace;
71 71
         }
72 72
 
73 73
         $readOnlyClass = 'true' === strtolower($elem->attributes()->{'read-only'});
74 74
 
75
-        $discriminatorFieldName = (string)$elem->attributes()->{'discriminator-field-name'};
75
+        $discriminatorFieldName = (string) $elem->attributes()->{'discriminator-field-name'};
76 76
         $discriminatorMap = array();
77 77
         foreach ($elem->xpath('./discriminator-class') as $entry) {
78
-            if (!isset($entry->attributes()->value)) {
78
+            if ( ! isset($entry->attributes()->value)) {
79 79
                 throw new RuntimeException('Each discriminator-class element must have a "value" attribute.');
80 80
             }
81 81
 
82
-            $discriminatorMap[(string)$entry->attributes()->value] = (string)$entry;
82
+            $discriminatorMap[(string) $entry->attributes()->value] = (string) $entry;
83 83
         }
84 84
 
85
-        if ('true' === (string)$elem->attributes()->{'discriminator-disabled'}) {
85
+        if ('true' === (string) $elem->attributes()->{'discriminator-disabled'}) {
86 86
             $metadata->discriminatorDisabled = true;
87
-        } elseif (!empty($discriminatorFieldName) || !empty($discriminatorMap)) {
87
+        } elseif ( ! empty($discriminatorFieldName) || ! empty($discriminatorMap)) {
88 88
 
89 89
             $discriminatorGroups = array();
90 90
             foreach ($elem->xpath('./discriminator-groups/group') as $entry) {
91
-                $discriminatorGroups[] = (string)$entry;
91
+                $discriminatorGroups[] = (string) $entry;
92 92
             }
93 93
             $metadata->setDiscriminator($discriminatorFieldName, $discriminatorMap, $discriminatorGroups);
94 94
         }
95 95
 
96 96
         foreach ($elem->xpath('./xml-namespace') as $xmlNamespace) {
97
-            if (!isset($xmlNamespace->attributes()->uri)) {
97
+            if ( ! isset($xmlNamespace->attributes()->uri)) {
98 98
                 throw new RuntimeException('The prefix attribute must be set for all xml-namespace elements.');
99 99
             }
100 100
 
101 101
             if (isset($xmlNamespace->attributes()->prefix)) {
102
-                $prefix = (string)$xmlNamespace->attributes()->prefix;
102
+                $prefix = (string) $xmlNamespace->attributes()->prefix;
103 103
             } else {
104 104
                 $prefix = null;
105 105
             }
106 106
 
107
-            $metadata->registerNamespace((string)$xmlNamespace->attributes()->uri, $prefix);
107
+            $metadata->registerNamespace((string) $xmlNamespace->attributes()->uri, $prefix);
108 108
         }
109 109
 
110 110
         foreach ($elem->xpath('./xml-discriminator') as $xmlDiscriminator) {
111 111
             if (isset($xmlDiscriminator->attributes()->attribute)) {
112
-                $metadata->xmlDiscriminatorAttribute = (string)$xmlDiscriminator->attributes()->attribute === 'true';
112
+                $metadata->xmlDiscriminatorAttribute = (string) $xmlDiscriminator->attributes()->attribute === 'true';
113 113
             }
114 114
             if (isset($xmlDiscriminator->attributes()->cdata)) {
115
-                $metadata->xmlDiscriminatorCData = (string)$xmlDiscriminator->attributes()->cdata === 'true';
115
+                $metadata->xmlDiscriminatorCData = (string) $xmlDiscriminator->attributes()->cdata === 'true';
116 116
             }
117 117
             if (isset($xmlDiscriminator->attributes()->namespace)) {
118
-                $metadata->xmlDiscriminatorNamespace = (string)$xmlDiscriminator->attributes()->namespace;
118
+                $metadata->xmlDiscriminatorNamespace = (string) $xmlDiscriminator->attributes()->namespace;
119 119
             }
120 120
         }
121 121
 
122 122
         foreach ($elem->xpath('./virtual-property') as $method) {
123 123
 
124 124
             if (isset($method->attributes()->expression)) {
125
-                $virtualPropertyMetadata = new ExpressionPropertyMetadata($name, (string)$method->attributes()->name, (string)$method->attributes()->expression);
125
+                $virtualPropertyMetadata = new ExpressionPropertyMetadata($name, (string) $method->attributes()->name, (string) $method->attributes()->expression);
126 126
             } else {
127
-                if (!isset($method->attributes()->method)) {
127
+                if ( ! isset($method->attributes()->method)) {
128 128
                     throw new RuntimeException('The method attribute must be set for all virtual-property elements.');
129 129
                 }
130
-                $virtualPropertyMetadata = new VirtualPropertyMetadata($name, (string)$method->attributes()->method);
130
+                $virtualPropertyMetadata = new VirtualPropertyMetadata($name, (string) $method->attributes()->method);
131 131
             }
132 132
 
133 133
             $propertiesMetadata[] = $virtualPropertyMetadata;
134 134
             $propertiesNodes[] = $method;
135 135
         }
136 136
 
137
-        if (!$excludeAll) {
137
+        if ( ! $excludeAll) {
138 138
 
139 139
             foreach ($class->getProperties() as $property) {
140 140
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
                 }
143 143
 
144 144
                 $propertiesMetadata[] = new PropertyMetadata($name, $pName = $property->getName());
145
-                $pElems = $elem->xpath("./property[@name = '" . $pName . "']");
145
+                $pElems = $elem->xpath("./property[@name = '".$pName."']");
146 146
 
147 147
                 $propertiesNodes[] = $pElems ? reset($pElems) : null;
148 148
             }
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
                     || $pMetadata instanceof ExpressionPropertyMetadata;
155 155
 
156 156
                 $pElem = $propertiesNodes[$propertyKey];
157
-                if (!empty($pElem)) {
157
+                if ( ! empty($pElem)) {
158 158
 
159 159
                     if (null !== $exclude = $pElem->attributes()->exclude) {
160 160
                         $isExclude = 'true' === strtolower($exclude);
@@ -173,26 +173,26 @@  discard block
 block discarded – undo
173 173
                     }
174 174
 
175 175
                     if (null !== $excludeIf = $pElem->attributes()->{'expose-if'}) {
176
-                        $pMetadata->excludeIf = "!(" . $excludeIf . ")";
176
+                        $pMetadata->excludeIf = "!(".$excludeIf.")";
177 177
                         $isExpose = true;
178 178
                     }
179 179
 
180 180
                     if (null !== $version = $pElem->attributes()->{'since-version'}) {
181
-                        $pMetadata->sinceVersion = (string)$version;
181
+                        $pMetadata->sinceVersion = (string) $version;
182 182
                     }
183 183
 
184 184
                     if (null !== $version = $pElem->attributes()->{'until-version'}) {
185
-                        $pMetadata->untilVersion = (string)$version;
185
+                        $pMetadata->untilVersion = (string) $version;
186 186
                     }
187 187
 
188 188
                     if (null !== $serializedName = $pElem->attributes()->{'serialized-name'}) {
189
-                        $pMetadata->serializedName = (string)$serializedName;
189
+                        $pMetadata->serializedName = (string) $serializedName;
190 190
                     }
191 191
 
192 192
                     if (null !== $type = $pElem->attributes()->type) {
193
-                        $pMetadata->setType((string)$type);
193
+                        $pMetadata->setType((string) $type);
194 194
                     } elseif (isset($pElem->type)) {
195
-                        $pMetadata->setType((string)$pElem->type);
195
+                        $pMetadata->setType((string) $pElem->type);
196 196
                     }
197 197
 
198 198
                     if (null !== $groups = $pElem->attributes()->groups) {
@@ -207,21 +207,21 @@  discard block
 block discarded – undo
207 207
 
208 208
                         $colConfig = $pElem->{'xml-list'};
209 209
                         if (isset($colConfig->attributes()->inline)) {
210
-                            $pMetadata->xmlCollectionInline = 'true' === (string)$colConfig->attributes()->inline;
210
+                            $pMetadata->xmlCollectionInline = 'true' === (string) $colConfig->attributes()->inline;
211 211
                         }
212 212
 
213 213
                         if (isset($colConfig->attributes()->{'entry-name'})) {
214
-                            $pMetadata->xmlEntryName = (string)$colConfig->attributes()->{'entry-name'};
214
+                            $pMetadata->xmlEntryName = (string) $colConfig->attributes()->{'entry-name'};
215 215
                         }
216 216
 
217 217
                         if (isset($colConfig->attributes()->{'skip-when-empty'})) {
218
-                            $pMetadata->xmlCollectionSkipWhenEmpty = 'true' === (string)$colConfig->attributes()->{'skip-when-empty'};
218
+                            $pMetadata->xmlCollectionSkipWhenEmpty = 'true' === (string) $colConfig->attributes()->{'skip-when-empty'};
219 219
                         } else {
220 220
                             $pMetadata->xmlCollectionSkipWhenEmpty = true;
221 221
                         }
222 222
 
223 223
                         if (isset($colConfig->attributes()->namespace)) {
224
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig->attributes()->namespace;
224
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig->attributes()->namespace;
225 225
                         }
226 226
                     }
227 227
 
@@ -230,51 +230,51 @@  discard block
 block discarded – undo
230 230
 
231 231
                         $colConfig = $pElem->{'xml-map'};
232 232
                         if (isset($colConfig->attributes()->inline)) {
233
-                            $pMetadata->xmlCollectionInline = 'true' === (string)$colConfig->attributes()->inline;
233
+                            $pMetadata->xmlCollectionInline = 'true' === (string) $colConfig->attributes()->inline;
234 234
                         }
235 235
 
236 236
                         if (isset($colConfig->attributes()->{'entry-name'})) {
237
-                            $pMetadata->xmlEntryName = (string)$colConfig->attributes()->{'entry-name'};
237
+                            $pMetadata->xmlEntryName = (string) $colConfig->attributes()->{'entry-name'};
238 238
                         }
239 239
 
240 240
                         if (isset($colConfig->attributes()->namespace)) {
241
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig->attributes()->namespace;
241
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig->attributes()->namespace;
242 242
                         }
243 243
 
244 244
                         if (isset($colConfig->attributes()->{'key-attribute-name'})) {
245
-                            $pMetadata->xmlKeyAttribute = (string)$colConfig->attributes()->{'key-attribute-name'};
245
+                            $pMetadata->xmlKeyAttribute = (string) $colConfig->attributes()->{'key-attribute-name'};
246 246
                         }
247 247
                     }
248 248
 
249 249
                     if (isset($pElem->{'xml-element'})) {
250 250
                         $colConfig = $pElem->{'xml-element'};
251 251
                         if (isset($colConfig->attributes()->cdata)) {
252
-                            $pMetadata->xmlElementCData = 'true' === (string)$colConfig->attributes()->cdata;
252
+                            $pMetadata->xmlElementCData = 'true' === (string) $colConfig->attributes()->cdata;
253 253
                         }
254 254
 
255 255
                         if (isset($colConfig->attributes()->namespace)) {
256
-                            $pMetadata->xmlNamespace = (string)$colConfig->attributes()->namespace;
256
+                            $pMetadata->xmlNamespace = (string) $colConfig->attributes()->namespace;
257 257
                         }
258 258
                     }
259 259
 
260 260
                     if (isset($pElem->attributes()->{'xml-attribute'})) {
261
-                        $pMetadata->xmlAttribute = 'true' === (string)$pElem->attributes()->{'xml-attribute'};
261
+                        $pMetadata->xmlAttribute = 'true' === (string) $pElem->attributes()->{'xml-attribute'};
262 262
                     }
263 263
 
264 264
                     if (isset($pElem->attributes()->{'xml-attribute-map'})) {
265
-                        $pMetadata->xmlAttributeMap = 'true' === (string)$pElem->attributes()->{'xml-attribute-map'};
265
+                        $pMetadata->xmlAttributeMap = 'true' === (string) $pElem->attributes()->{'xml-attribute-map'};
266 266
                     }
267 267
 
268 268
                     if (isset($pElem->attributes()->{'xml-value'})) {
269
-                        $pMetadata->xmlValue = 'true' === (string)$pElem->attributes()->{'xml-value'};
269
+                        $pMetadata->xmlValue = 'true' === (string) $pElem->attributes()->{'xml-value'};
270 270
                     }
271 271
 
272 272
                     if (isset($pElem->attributes()->{'xml-key-value-pairs'})) {
273
-                        $pMetadata->xmlKeyValuePairs = 'true' === (string)$pElem->attributes()->{'xml-key-value-pairs'};
273
+                        $pMetadata->xmlKeyValuePairs = 'true' === (string) $pElem->attributes()->{'xml-key-value-pairs'};
274 274
                     }
275 275
 
276 276
                     if (isset($pElem->attributes()->{'max-depth'})) {
277
-                        $pMetadata->maxDepth = (int)$pElem->attributes()->{'max-depth'};
277
+                        $pMetadata->maxDepth = (int) $pElem->attributes()->{'max-depth'};
278 278
                     }
279 279
 
280 280
                     //we need read-only before setter and getter set, because that method depends on flag being set
@@ -287,9 +287,9 @@  discard block
 block discarded – undo
287 287
                     $getter = $pElem->attributes()->{'accessor-getter'};
288 288
                     $setter = $pElem->attributes()->{'accessor-setter'};
289 289
                     $pMetadata->setAccessor(
290
-                        (string)($pElem->attributes()->{'access-type'} ?: $classAccessType),
291
-                        $getter ? (string)$getter : null,
292
-                        $setter ? (string)$setter : null
290
+                        (string) ($pElem->attributes()->{'access-type'} ?: $classAccessType),
291
+                        $getter ? (string) $getter : null,
292
+                        $setter ? (string) $setter : null
293 293
                     );
294 294
 
295 295
                     if (null !== $inline = $pElem->attributes()->inline) {
@@ -298,8 +298,8 @@  discard block
 block discarded – undo
298 298
 
299 299
                 }
300 300
 
301
-                if ((ExclusionPolicy::NONE === (string)$exclusionPolicy && !$isExclude)
302
-                    || (ExclusionPolicy::ALL === (string)$exclusionPolicy && $isExpose)
301
+                if ((ExclusionPolicy::NONE === (string) $exclusionPolicy && ! $isExclude)
302
+                    || (ExclusionPolicy::ALL === (string) $exclusionPolicy && $isExpose)
303 303
                 ) {
304 304
 
305 305
                     $metadata->addPropertyMetadata($pMetadata);
@@ -308,37 +308,37 @@  discard block
 block discarded – undo
308 308
         }
309 309
 
310 310
         foreach ($elem->xpath('./callback-method') as $method) {
311
-            if (!isset($method->attributes()->type)) {
311
+            if ( ! isset($method->attributes()->type)) {
312 312
                 throw new RuntimeException('The type attribute must be set for all callback-method elements.');
313 313
             }
314
-            if (!isset($method->attributes()->name)) {
314
+            if ( ! isset($method->attributes()->name)) {
315 315
                 throw new RuntimeException('The name attribute must be set for all callback-method elements.');
316 316
             }
317 317
 
318
-            switch ((string)$method->attributes()->type) {
318
+            switch ((string) $method->attributes()->type) {
319 319
                 case 'pre-serialize':
320
-                    $metadata->addPreSerializeMethod(new MethodMetadata($name, (string)$method->attributes()->name));
320
+                    $metadata->addPreSerializeMethod(new MethodMetadata($name, (string) $method->attributes()->name));
321 321
                     break;
322 322
 
323 323
                 case 'post-serialize':
324
-                    $metadata->addPostSerializeMethod(new MethodMetadata($name, (string)$method->attributes()->name));
324
+                    $metadata->addPostSerializeMethod(new MethodMetadata($name, (string) $method->attributes()->name));
325 325
                     break;
326 326
 
327 327
                 case 'post-deserialize':
328
-                    $metadata->addPostDeserializeMethod(new MethodMetadata($name, (string)$method->attributes()->name));
328
+                    $metadata->addPostDeserializeMethod(new MethodMetadata($name, (string) $method->attributes()->name));
329 329
                     break;
330 330
 
331 331
                 case 'handler':
332
-                    if (!isset($method->attributes()->format)) {
332
+                    if ( ! isset($method->attributes()->format)) {
333 333
                         throw new RuntimeException('The format attribute must be set for "handler" callback methods.');
334 334
                     }
335
-                    if (!isset($method->attributes()->direction)) {
335
+                    if ( ! isset($method->attributes()->direction)) {
336 336
                         throw new RuntimeException('The direction attribute must be set for "handler" callback methods.');
337 337
                     }
338 338
 
339
-                    $direction = GraphNavigator::parseDirection((string)$method->attributes()->direction);
340
-                    $format = (string)$method->attributes()->format;
341
-                    $metadata->addHandlerCallback($direction, $format, (string)$method->attributes()->name);
339
+                    $direction = GraphNavigator::parseDirection((string) $method->attributes()->direction);
340
+                    $format = (string) $method->attributes()->format;
341
+                    $metadata->addHandlerCallback($direction, $format, (string) $method->attributes()->name);
342 342
 
343 343
                     break;
344 344
 
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/Driver/DoctrinePHPCRTypeDriver.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
         $propertyName = $propertyMetadata->name;
44 44
         if ($doctrineMetadata->hasField($propertyName) && $fieldType = $this->normalizeFieldType($doctrineMetadata->getTypeOfField($propertyName))) {
45 45
             $field = $doctrineMetadata->getFieldMapping($propertyName);
46
-            if (!empty($field['multivalue'])) {
46
+            if ( ! empty($field['multivalue'])) {
47 47
                 $fieldType = 'array';
48 48
             }
49 49
 
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
                 return;
60 60
             }
61 61
 
62
-            if (!$doctrineMetadata->isSingleValuedAssociation($propertyName)) {
62
+            if ( ! $doctrineMetadata->isSingleValuedAssociation($propertyName)) {
63 63
                 $targetEntity = "ArrayCollection<{$targetEntity}>";
64 64
             }
65 65
 
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/Driver/PhpDriver.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
     {
29 29
         $metadata = require $file;
30 30
 
31
-        if (!$metadata instanceof ClassMetadata) {
31
+        if ( ! $metadata instanceof ClassMetadata) {
32 32
             throw new RuntimeException(sprintf('The file %s was expected to return an instance of ClassMetadata, but returned %s.', $file, json_encode($metadata)));
33 33
         }
34 34
         if ($metadata->name !== $class->name) {
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/Driver/AbstractDoctrineTypeDriver.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
         $classMetadata = $this->delegate->loadMetadataForClass($class);
84 84
 
85 85
         // Abort if the given class is not a mapped entity
86
-        if (!$doctrineMetadata = $this->tryLoadingDoctrineMetadata($class->name)) {
86
+        if ( ! $doctrineMetadata = $this->tryLoadingDoctrineMetadata($class->name)) {
87 87
             return $classMetadata;
88 88
         }
89 89
 
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
      */
149 149
     protected function tryLoadingDoctrineMetadata($className)
150 150
     {
151
-        if (!$manager = $this->registry->getManagerForClass($className)) {
151
+        if ( ! $manager = $this->registry->getManagerForClass($className)) {
152 152
             return null;
153 153
         }
154 154
 
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
      */
165 165
     protected function normalizeFieldType($type)
166 166
     {
167
-        if (!isset($this->fieldMapping[$type])) {
167
+        if ( ! isset($this->fieldMapping[$type])) {
168 168
             return;
169 169
         }
170 170
 
Please login to merge, or discard this patch.
src/JMS/Serializer/Metadata/Driver/DoctrineTypeDriver.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,8 +30,8 @@  discard block
 block discarded – undo
30 30
 {
31 31
     protected function setDiscriminator(DoctrineClassMetadata $doctrineMetadata, ClassMetadata $classMetadata)
32 32
     {
33
-        if (empty($classMetadata->discriminatorMap) && !$classMetadata->discriminatorDisabled
34
-            && !empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
33
+        if (empty($classMetadata->discriminatorMap) && ! $classMetadata->discriminatorDisabled
34
+            && ! empty($doctrineMetadata->discriminatorMap) && $doctrineMetadata->isRootEntity()
35 35
         ) {
36 36
             $classMetadata->setDiscriminator(
37 37
                 $doctrineMetadata->discriminatorColumn['name'],
@@ -55,11 +55,11 @@  discard block
 block discarded – undo
55 55
             // For inheritance schemes, we cannot add any type as we would only add the super-type of the hierarchy.
56 56
             // On serialization, this would lead to only the supertype being serialized, and properties of subtypes
57 57
             // being ignored.
58
-            if ($targetMetadata instanceof DoctrineClassMetadata && !$targetMetadata->isInheritanceTypeNone()) {
58
+            if ($targetMetadata instanceof DoctrineClassMetadata && ! $targetMetadata->isInheritanceTypeNone()) {
59 59
                 return;
60 60
             }
61 61
 
62
-            if (!$doctrineMetadata->isSingleValuedAssociation($propertyName)) {
62
+            if ( ! $doctrineMetadata->isSingleValuedAssociation($propertyName)) {
63 63
                 $targetEntity = "ArrayCollection<{$targetEntity}>";
64 64
             }
65 65
 
Please login to merge, or discard this patch.
src/JMS/Serializer/EventDispatcher/Subscriber/DoctrineProxySubscriber.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 
42 42
     public function __construct($skipVirtualTypeInit = false, $initializeExcluded = true)
43 43
     {
44
-        $this->skipVirtualTypeInit = (bool)$skipVirtualTypeInit;
45
-        $this->initializeExcluded = (bool)$initializeExcluded;
44
+        $this->skipVirtualTypeInit = (bool) $skipVirtualTypeInit;
45
+        $this->initializeExcluded = (bool) $initializeExcluded;
46 46
     }
47 47
 
48 48
     public function onPreSerialize(PreSerializeEvent $event)
@@ -53,13 +53,13 @@  discard block
 block discarded – undo
53 53
         // If the set type name is not an actual class, but a faked type for which a custom handler exists, we do not
54 54
         // modify it with this subscriber. Also, we forgo autoloading here as an instance of this type is already created,
55 55
         // so it must be loaded if its a real class.
56
-        $virtualType = !class_exists($type['name'], false);
56
+        $virtualType = ! class_exists($type['name'], false);
57 57
 
58 58
         if ($object instanceof PersistentCollection
59 59
             || $object instanceof MongoDBPersistentCollection
60 60
             || $object instanceof PHPCRPersistentCollection
61 61
         ) {
62
-            if (!$virtualType) {
62
+            if ( ! $virtualType) {
63 63
                 $event->setType('ArrayCollection');
64 64
             }
65 65
 
@@ -67,13 +67,13 @@  discard block
 block discarded – undo
67 67
         }
68 68
 
69 69
         if (($this->skipVirtualTypeInit && $virtualType) ||
70
-            (!$object instanceof Proxy && !$object instanceof ORMProxy)
70
+            ( ! $object instanceof Proxy && ! $object instanceof ORMProxy)
71 71
         ) {
72 72
             return;
73 73
         }
74 74
 
75 75
         // do not initialize the proxy if is going to be excluded by-class by some exclusion strategy
76
-        if ($this->initializeExcluded === false && !$virtualType) {
76
+        if ($this->initializeExcluded === false && ! $virtualType) {
77 77
             $context = $event->getContext();
78 78
             $exclusionStrategy = $context->getExclusionStrategy();
79 79
             if ($exclusionStrategy !== null && $exclusionStrategy->shouldSkipClass($context->getMetadataFactory()->getMetadataForClass(get_parent_class($object)), $context)) {
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 
84 84
         $object->__load();
85 85
 
86
-        if (!$virtualType) {
86
+        if ( ! $virtualType) {
87 87
             $event->setType(get_parent_class($object), $type['params']);
88 88
         }
89 89
     }
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
     {
93 93
         $type = $event->getType();
94 94
         // is a virtual type? then there is no need to change the event name
95
-        if (!class_exists($type['name'], false)) {
95
+        if ( ! class_exists($type['name'], false)) {
96 96
             return;
97 97
         }
98 98
 
Please login to merge, or discard this patch.
JMS/Serializer/EventDispatcher/Subscriber/SymfonyValidatorSubscriber.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 
50 50
         $validator = $this->validator;
51 51
         $context->attributes->get('validation_groups')->map(
52
-            function (array $groups) use ($event, $validator) {
52
+            function(array $groups) use ($event, $validator) {
53 53
                 $list = $validator->validate($event->getObject(), $groups);
54 54
 
55 55
                 if ($list->count() > 0) {
Please login to merge, or discard this patch.
src/JMS/Serializer/EventDispatcher/EventDispatcher.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 
36 36
     public static function getDefaultMethodName($eventName)
37 37
     {
38
-        return 'on' . str_replace(array('_', '.'), '', $eventName);
38
+        return 'on'.str_replace(array('_', '.'), '', $eventName);
39 39
     }
40 40
 
41 41
     /**
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
     public function addSubscriber(EventSubscriberInterface $subscriber)
59 59
     {
60 60
         foreach ($subscriber->getSubscribedEvents() as $eventData) {
61
-            if (!isset($eventData['event'])) {
61
+            if ( ! isset($eventData['event'])) {
62 62
                 throw new InvalidArgumentException(sprintf('Each event must have a "event" key.'));
63 63
             }
64 64
 
@@ -72,26 +72,26 @@  discard block
 block discarded – undo
72 72
 
73 73
     public function hasListeners($eventName, $class, $format)
74 74
     {
75
-        if (!isset($this->listeners[$eventName])) {
75
+        if ( ! isset($this->listeners[$eventName])) {
76 76
             return false;
77 77
         }
78 78
 
79 79
         $loweredClass = strtolower($class);
80
-        if (!isset($this->classListeners[$eventName][$loweredClass][$format])) {
80
+        if ( ! isset($this->classListeners[$eventName][$loweredClass][$format])) {
81 81
             $this->classListeners[$eventName][$loweredClass][$format] = $this->initializeListeners($eventName, $loweredClass, $format);
82 82
         }
83 83
 
84
-        return !!$this->classListeners[$eventName][$loweredClass][$format];
84
+        return ! ! $this->classListeners[$eventName][$loweredClass][$format];
85 85
     }
86 86
 
87 87
     public function dispatch($eventName, $class, $format, Event $event)
88 88
     {
89
-        if (!isset($this->listeners[$eventName])) {
89
+        if ( ! isset($this->listeners[$eventName])) {
90 90
             return;
91 91
         }
92 92
 
93 93
         $loweredClass = strtolower($class);
94
-        if (!isset($this->classListeners[$eventName][$loweredClass][$format])) {
94
+        if ( ! isset($this->classListeners[$eventName][$loweredClass][$format])) {
95 95
             $this->classListeners[$eventName][$loweredClass][$format] = $this->initializeListeners($eventName, $loweredClass, $format);
96 96
         }
97 97
 
Please login to merge, or discard this patch.