Completed
Pull Request — master (#1136)
by Ivan
12:10
created
src/Metadata/Driver/YamlDriver.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
      */
71 71
     public function getAllClassNames(): array
72 72
     {
73
-        if (!$this->locator instanceof AdvancedFileLocatorInterface) {
73
+        if ( ! $this->locator instanceof AdvancedFileLocatorInterface) {
74 74
             throw new RuntimeException(
75 75
                 sprintf(
76 76
                     'Locator "%s" must be an instance of "AdvancedFileLocatorInterface".',
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     {
94 94
         $config = Yaml::parse(file_get_contents($file));
95 95
 
96
-        if (!isset($config[$name = $class->name])) {
96
+        if ( ! isset($config[$name = $class->name])) {
97 97
             throw new InvalidMetadataException(
98 98
                 sprintf('Expected metadata for class %s to be defined in %s.', $class->name, $file)
99 99
             );
@@ -125,9 +125,9 @@  discard block
 block discarded – undo
125 125
                     );
126 126
                     unset($propertySettings['exp']);
127 127
                 } else {
128
-                    if (!$class->hasMethod($methodName)) {
128
+                    if ( ! $class->hasMethod($methodName)) {
129 129
                         throw new InvalidMetadataException(
130
-                            'The method ' . $methodName . ' not found in class ' . $class->name
130
+                            'The method '.$methodName.' not found in class '.$class->name
131 131
                         );
132 132
                     }
133 133
                     $virtualPropertyMetadata = new VirtualPropertyMetadata($name, $methodName);
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
             }
139 139
         }
140 140
 
141
-        if (!$excludeAll) {
141
+        if ( ! $excludeAll) {
142 142
             foreach ($class->getProperties() as $property) {
143 143
                 $fromParent = false;
144 144
                 $pName = $property->getName();
@@ -148,13 +148,13 @@  discard block
 block discarded – undo
148 148
                     $fromParent = true;
149 149
                 }
150 150
 
151
-                if (!$fromParent && $property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
151
+                if ( ! $fromParent && $property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
152 152
                     continue;
153 153
                 }
154 154
 
155 155
                 $pName = $property->getName();
156 156
                 $propertiesMetadata[] = new PropertyMetadata($name, $pName);
157
-                $propertiesData[] =  !empty($config['properties']) && true === array_key_exists($pName, $config['properties'])
157
+                $propertiesData[] = ! empty($config['properties']) && true === array_key_exists($pName, $config['properties'])
158 158
                     ? (array) $config['properties'][$pName]
159 159
                     : null;
160 160
             }
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
                     || isset($propertiesData[$propertyKey]);
167 167
 
168 168
                 $pConfig = $propertiesData[$propertyKey];
169
-                if (!empty($pConfig)) {
169
+                if ( ! empty($pConfig)) {
170 170
                     if (isset($pConfig['exclude'])) {
171 171
                         $isExclude = (bool) $pConfig['exclude'];
172 172
                     }
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
                     }
197 197
 
198 198
                     if (isset($pConfig['expose_if'])) {
199
-                        $pMetadata->excludeIf = $this->parseExpression('!(' . $pConfig['expose_if'] . ')');
199
+                        $pMetadata->excludeIf = $this->parseExpression('!('.$pConfig['expose_if'].')');
200 200
                     }
201 201
 
202 202
                     if (isset($pConfig['serialized_name'])) {
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
                     }
305 305
                 }
306 306
 
307
-                if (!$pMetadata->serializedName) {
307
+                if ( ! $pMetadata->serializedName) {
308 308
                     $pMetadata->serializedName = $this->namingStrategy->translateName($pMetadata);
309 309
                 }
310 310
 
@@ -313,11 +313,11 @@  discard block
 block discarded – undo
313 313
                     $metadata->isMap = $metadata->isMap || PropertyMetadata::isCollectionMap($pMetadata->type);
314 314
                 }
315 315
 
316
-                if (!empty($pConfig) && !empty($pConfig['name'])) {
316
+                if ( ! empty($pConfig) && ! empty($pConfig['name'])) {
317 317
                     $pMetadata->name = (string) $pConfig['name'];
318 318
                 }
319 319
 
320
-                if ((ExclusionPolicy::NONE === $exclusionPolicy && !$isExclude)
320
+                if ((ExclusionPolicy::NONE === $exclusionPolicy && ! $isExclude)
321 321
                     || (ExclusionPolicy::ALL === $exclusionPolicy && $isExpose)
322 322
                 ) {
323 323
                     $metadata->addPropertyMetadata($pMetadata);
@@ -360,7 +360,7 @@  discard block
 block discarded – undo
360 360
 
361 361
     private function addClassProperties(ClassMetadata $metadata, array $config): void
362 362
     {
363
-        if (isset($config['custom_accessor_order']) && !isset($config['accessor_order'])) {
363
+        if (isset($config['custom_accessor_order']) && ! isset($config['accessor_order'])) {
364 364
             $config['accessor_order'] = 'custom';
365 365
         }
366 366
 
@@ -390,11 +390,11 @@  discard block
 block discarded – undo
390 390
             if (isset($config['discriminator']['disabled']) && true === $config['discriminator']['disabled']) {
391 391
                 $metadata->discriminatorDisabled = true;
392 392
             } else {
393
-                if (!isset($config['discriminator']['field_name'])) {
393
+                if ( ! isset($config['discriminator']['field_name'])) {
394 394
                     throw new InvalidMetadataException('The "field_name" attribute must be set for discriminators.');
395 395
                 }
396 396
 
397
-                if (!isset($config['discriminator']['map']) || !is_array($config['discriminator']['map'])) {
397
+                if ( ! isset($config['discriminator']['map']) || ! is_array($config['discriminator']['map'])) {
398 398
                     throw new InvalidMetadataException(
399 399
                         'The "map" attribute must be set, and be an array for discriminators.'
400 400
                     );
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
     {
429 429
         if (is_string($config)) {
430 430
             $config = [$config];
431
-        } elseif (!is_array($config)) {
431
+        } elseif ( ! is_array($config)) {
432 432
             throw new InvalidMetadataException(
433 433
                 sprintf(
434 434
                     'callback methods expects a string, or an array of strings that represent method names, but got %s.',
@@ -439,7 +439,7 @@  discard block
 block discarded – undo
439 439
 
440 440
         $methods = [];
441 441
         foreach ($config as $name) {
442
-            if (!$class->hasMethod($name)) {
442
+            if ( ! $class->hasMethod($name)) {
443 443
                 throw new InvalidMetadataException(
444 444
                     sprintf('The method %s does not exist in class %s.', $name, $class->name)
445 445
                 );
Please login to merge, or discard this patch.