Passed
Pull Request — master (#1536)
by
unknown
04:05
created
src/Metadata/PropertyMetadata.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -157,22 +157,22 @@  discard block
 block discarded – undo
157 157
             $class = $this->getReflection()->getDeclaringClass();
158 158
 
159 159
             if (empty($getter)) {
160
-                if ($class->hasMethod('get' . $this->name) && $class->getMethod('get' . $this->name)->isPublic()) {
161
-                    $getter = 'get' . $this->name;
162
-                } elseif ($class->hasMethod('is' . $this->name) && $class->getMethod('is' . $this->name)->isPublic()) {
163
-                    $getter = 'is' . $this->name;
164
-                } elseif ($class->hasMethod('has' . $this->name) && $class->getMethod('has' . $this->name)->isPublic()) {
165
-                    $getter = 'has' . $this->name;
160
+                if ($class->hasMethod('get'.$this->name) && $class->getMethod('get'.$this->name)->isPublic()) {
161
+                    $getter = 'get'.$this->name;
162
+                } elseif ($class->hasMethod('is'.$this->name) && $class->getMethod('is'.$this->name)->isPublic()) {
163
+                    $getter = 'is'.$this->name;
164
+                } elseif ($class->hasMethod('has'.$this->name) && $class->getMethod('has'.$this->name)->isPublic()) {
165
+                    $getter = 'has'.$this->name;
166 166
                 } else {
167
-                    throw new InvalidMetadataException(sprintf('There is neither a public %s method, nor a public %s method, nor a public %s method in class %s. Please specify which public method should be used for retrieving the value of the property %s.', 'get' . ucfirst($this->name), 'is' . ucfirst($this->name), 'has' . ucfirst($this->name), $this->class, $this->name));
167
+                    throw new InvalidMetadataException(sprintf('There is neither a public %s method, nor a public %s method, nor a public %s method in class %s. Please specify which public method should be used for retrieving the value of the property %s.', 'get'.ucfirst($this->name), 'is'.ucfirst($this->name), 'has'.ucfirst($this->name), $this->class, $this->name));
168 168
                 }
169 169
             }
170 170
 
171
-            if (empty($setter) && !$this->readOnly) {
172
-                if ($class->hasMethod('set' . $this->name) && $class->getMethod('set' . $this->name)->isPublic()) {
173
-                    $setter = 'set' . $this->name;
171
+            if (empty($setter) && ! $this->readOnly) {
172
+                if ($class->hasMethod('set'.$this->name) && $class->getMethod('set'.$this->name)->isPublic()) {
173
+                    $setter = 'set'.$this->name;
174 174
                 } else {
175
-                    throw new InvalidMetadataException(sprintf('There is no public %s method in class %s. Please specify which public method should be used for setting the value of the property %s.', 'set' . ucfirst($this->name), $this->class, $this->name));
175
+                    throw new InvalidMetadataException(sprintf('There is no public %s method in class %s. Please specify which public method should be used for setting the value of the property %s.', 'set'.ucfirst($this->name), $this->class, $this->name));
176 176
                 }
177 177
             }
178 178
         }
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
         return is_array($type)
192 192
             && 'array' === $type['name']
193 193
             && isset($type['params'][0])
194
-            && !isset($type['params'][1]);
194
+            && ! isset($type['params'][1]);
195 195
     }
196 196
 
197 197
     public static function isCollectionMap(?array $type = null): bool
Please login to merge, or discard this patch.
src/Serializer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 
115 115
     private function getNavigator(int $direction): GraphNavigatorInterface
116 116
     {
117
-        if (!isset($this->graphNavigators[$direction])) {
117
+        if ( ! isset($this->graphNavigators[$direction])) {
118 118
             throw new RuntimeException(
119 119
                 sprintf(
120 120
                     'Can not find a graph navigator for the direction "%s".',
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
             ? $this->serializationVisitors
133 133
             : $this->deserializationVisitors;
134 134
 
135
-        if (!isset($factories[$format])) {
135
+        if ( ! isset($factories[$format])) {
136 136
             throw new UnsupportedFormatException(
137 137
                 sprintf(
138 138
                     'The format "%s" is not supported for %s.',
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
         $result = $this->visit($navigator, $visitor, $context, $data, 'json', $type);
197 197
         $result = $this->convertArrayObjects($result);
198 198
 
199
-        if (!\is_array($result)) {
199
+        if ( ! \is_array($result)) {
200 200
             throw new RuntimeException(sprintf(
201 201
                 'The input data of type "%s" did not convert to an array, but got a result of type "%s".',
202 202
                 \is_object($data) ? \get_class($data) : \gettype($data),
Please login to merge, or discard this patch.
src/Context.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
             $this->addExclusionStrategy(new VersionExclusionStrategy($this->attributes['version']));
80 80
         }
81 81
 
82
-        if (!empty($this->attributes['max_depth_checks'])) {
82
+        if ( ! empty($this->attributes['max_depth_checks'])) {
83 83
             $this->addExclusionStrategy(new DepthExclusionStrategy());
84 84
         }
85 85
 
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 
133 133
     final protected function assertMutable(): void
134 134
     {
135
-        if (!$this->initialized) {
135
+        if ( ! $this->initialized) {
136 136
             return;
137 137
         }
138 138
 
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
     {
209 209
         $metadata = $this->metadataStack->pop();
210 210
 
211
-        if (!$metadata instanceof PropertyMetadata) {
211
+        if ( ! $metadata instanceof PropertyMetadata) {
212 212
             throw new RuntimeException('Context metadataStack not working well');
213 213
         }
214 214
     }
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
     {
218 218
         $metadata = $this->metadataStack->pop();
219 219
 
220
-        if (!$metadata instanceof ClassMetadata) {
220
+        if ( ! $metadata instanceof ClassMetadata) {
221 221
             throw new RuntimeException('Context metadataStack not working well');
222 222
         }
223 223
     }
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
      */
233 233
     public function getCurrentPath(): array
234 234
     {
235
-        if (!$this->metadataStack) {
235
+        if ( ! $this->metadataStack) {
236 236
             return [];
237 237
         }
238 238
 
Please login to merge, or discard this patch.
src/AbstractVisitor.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
             throw new NonStringCastableTypeException($value);
57 57
         }
58 58
 
59
-        if (is_object($value) && !method_exists($value, '__toString')) {
59
+        if (is_object($value) && ! method_exists($value, '__toString')) {
60 60
             throw new NonStringCastableTypeException($value);
61 61
         }
62 62
     }
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
      */
70 70
     protected function assertValueCanBeCastToInt($value): void
71 71
     {
72
-        if (is_object($value) && !$value instanceof \SimpleXMLElement) {
72
+        if (is_object($value) && ! $value instanceof \SimpleXMLElement) {
73 73
             throw new NonIntCastableTypeException($value);
74 74
         }
75 75
     }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
      */
83 83
     protected function assertValueCanCastToFloat($value): void
84 84
     {
85
-        if (is_object($value) && !$value instanceof \SimpleXMLElement) {
85
+        if (is_object($value) && ! $value instanceof \SimpleXMLElement) {
86 86
             throw new NonFloatCastableTypeException($value);
87 87
         }
88 88
     }
Please login to merge, or discard this patch.
src/GraphNavigator/DeserializationGraphNavigator.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -171,11 +171,11 @@  discard block
 block discarded – undo
171 171
                 $metadata = $this->metadataFactory->getMetadataForClass($type['name']);
172 172
                 \assert($metadata instanceof ClassMetadata);
173 173
 
174
-                if ($metadata->usingExpression && !$this->expressionExclusionStrategy) {
174
+                if ($metadata->usingExpression && ! $this->expressionExclusionStrategy) {
175 175
                     throw new ExpressionLanguageRequiredException(sprintf('To use conditional exclude/expose in %s you must configure the expression language.', $metadata->name));
176 176
                 }
177 177
 
178
-                if (!empty($metadata->discriminatorMap) && $type['name'] === $metadata->discriminatorBaseClass) {
178
+                if ( ! empty($metadata->discriminatorMap) && $type['name'] === $metadata->discriminatorBaseClass) {
179 179
                     $metadata = $this->resolveMetadata($data, $metadata);
180 180
                 }
181 181
 
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
     {
235 235
         $typeValue = $this->visitor->visitDiscriminatorMapProperty($data, $metadata);
236 236
 
237
-        if (!isset($metadata->discriminatorMap[$typeValue])) {
237
+        if ( ! isset($metadata->discriminatorMap[$typeValue])) {
238 238
             throw new LogicException(sprintf(
239 239
                 'The type value "%s" does not exist in the discriminator map of class "%s". Available types: %s',
240 240
                 $typeValue,
Please login to merge, or discard this patch.
src/Metadata/Driver/DoctrinePHPCRTypeDriver.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
             && ($fieldType = $this->normalizeFieldType($typeOfFiled))
30 30
         ) {
31 31
             $field = $doctrineMetadata->getFieldMapping($propertyName);
32
-            if (!empty($field['multivalue'])) {
32
+            if ( ! empty($field['multivalue'])) {
33 33
                 $fieldType = 'array';
34 34
             }
35 35
 
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
                 return;
46 46
             }
47 47
 
48
-            if (!$doctrineMetadata->isSingleValuedAssociation($propertyName)) {
48
+            if ( ! $doctrineMetadata->isSingleValuedAssociation($propertyName)) {
49 49
                 $targetEntity = sprintf('ArrayCollection<%s>', $targetEntity);
50 50
             }
51 51
 
Please login to merge, or discard this patch.
src/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
         \assert($classMetadata instanceof ClassMetadata);
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
 
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 
130 130
     protected function tryLoadingDoctrineMetadata(string $className): ?DoctrineClassMetadata
131 131
     {
132
-        if (!$manager = $this->registry->getManagerForClass($className)) {
132
+        if ( ! $manager = $this->registry->getManagerForClass($className)) {
133 133
             return null;
134 134
         }
135 135
 
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 
143 143
     protected function normalizeFieldType(string $type): ?string
144 144
     {
145
-        if (!isset($this->fieldMapping[$type])) {
145
+        if ( ! isset($this->fieldMapping[$type])) {
146 146
             return null;
147 147
         }
148 148
 
Please login to merge, or discard this patch.
src/SerializationContext.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      */
65 65
     public function startVisiting($object): void
66 66
     {
67
-        if (!\is_object($object)) {
67
+        if ( ! \is_object($object)) {
68 68
             return;
69 69
         }
70 70
 
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
      */
78 78
     public function stopVisiting($object): void
79 79
     {
80
-        if (!\is_object($object)) {
80
+        if ( ! \is_object($object)) {
81 81
             return;
82 82
         }
83 83
 
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
      */
95 95
     public function isVisiting($object): bool
96 96
     {
97
-        if (!\is_object($object)) {
97
+        if ( ! \is_object($object)) {
98 98
             return false;
99 99
         }
100 100
 
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
             $path[] = \get_class($obj);
109 109
         }
110 110
 
111
-        if (!$path) {
111
+        if ( ! $path) {
112 112
             return null;
113 113
         }
114 114
 
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 
128 128
     public function getObject(): ?object
129 129
     {
130
-        return !$this->visitingStack->isEmpty() ? $this->visitingStack->top() : null;
130
+        return ! $this->visitingStack->isEmpty() ? $this->visitingStack->top() : null;
131 131
     }
132 132
 
133 133
     public function getVisitingStack(): \SplStack
Please login to merge, or discard this patch.
src/Naming/CamelCaseNamingStrategy.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
 
32 32
     public function translateName(PropertyMetadata $property): string
33 33
     {
34
-        $name = preg_replace('/[A-Z]+/', $this->separator . '\\0', $property->name);
34
+        $name = preg_replace('/[A-Z]+/', $this->separator.'\\0', $property->name);
35 35
 
36 36
         if ($this->lowerCase) {
37 37
             return strtolower($name);
Please login to merge, or discard this patch.